Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

BUG: Mouse Move Event Error: Uncaught (in promise) DOMException #1110

Closed
KalebLester opened this issue Mar 17, 2023 · 2 comments
Closed

BUG: Mouse Move Event Error: Uncaught (in promise) DOMException #1110

KalebLester opened this issue Mar 17, 2023 · 2 comments

Comments

@KalebLester
Copy link

This issue was discovered when on MangaDex. After hovering over a clickable item once, it is no longer clickable. I tested this on multiple browser and in isolated environments with only 10ten enabled. The issue occurs on Chromium browser and Safari; I could not recreate this issue in Firefox. I am unaware of any other sites this issue affects.

The issue occurs on the menubar and the "Popular New Titles" section on the main page (you have to move the mouse over the entire element). The issue does not occur in the settings page.

I've recorded the issue on Microsoft Edge and I took a screenshot of the console in Safari. The video showcases how the error is logged in the console after every element is hovered.

Screenshot.2023-03-17.at.10.39.02.AM.mov

SCR-20230317-eou


The full error from Chromium is:
Uncaught (in promise) DOMException: Failed to execute 'setEnd' on 'Range': The offset 1 is larger than the node's length (0). at adjustForRangeBoundary (chrome-extension://pnmaklegiibbioifkmfkgpfnmdehdfan/10ten-ja-content.js:4634:22) at rawCaretPositionFromPoint (chrome-extension://pnmaklegiibbioifkmfkgpfnmdehdfan/10ten-ja-content.js:4587:21) at caretPositionFromPoint (chrome-extension://pnmaklegiibbioifkmfkgpfnmdehdfan/10ten-ja-content.js:4521:20) at getTextFromCoveringLink (chrome-extension://pnmaklegiibbioifkmfkgpfnmdehdfan/10ten-ja-content.js:4937:24) at getTextAtPoint (chrome-extension://pnmaklegiibbioifkmfkgpfnmdehdfan/10ten-ja-content.js:4469:24) at ContentHandler.tryToUpdatePopup (chrome-extension://pnmaklegiibbioifkmfkgpfnmdehdfan/10ten-ja-content.js:11213:29) at ContentHandler.onMouseMove (chrome-extension://pnmaklegiibbioifkmfkgpfnmdehdfan/10ten-ja-content.js:10465:19)


Extension version: 1.13.5
Note: I'm using the Chromium version on Edge; however, I do not believe this to be significant as the issue is present on Safari too.

@birtles
Copy link
Member

birtles commented Mar 20, 2023

Thanks for the excellent bug report. Sorry for the delay in getting back to you. I've dug into it and I believe I found the problem. It's in a code path not used by Firefox (since Firefox has better APIs for text lookup) so it's not surprising that it doesn't reproduce in Firefox.

Also, a side note, there is also a version for Edge in the Edge extensions store. It's almost identical to the Chrome version except for a few parts where the address of Chrome-specific pages has been updated for Edge.

@birtles
Copy link
Member

birtles commented Mar 20, 2023

Should be fixed by a7c0ce1

@birtles birtles closed this as completed Mar 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants