Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
document
keeps gettingmousemove
events even outside the window, untilmouseup
. Who knew?!? Not me anyway... but now that I do, this quick change allows dragging to keep going outside the window. Tested manually in all my local browsers, IE9 - 11 and iOS on Browserstack, and of course the test suites still work (after removingmouseout
which is no longer part of the interaction).Two notes:
coverSlip
around, because that's an easy way to prevent selecting text as you drag around.setCapture
/releaseCapture
in IE, but a) that seems to be older versions than we target, and b) might have been mitigated bycoverSlip
anyway. I saw no difference if I included it in our code.cc @etpinard