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

drag detection #3

Closed
flypenguin opened this issue Feb 9, 2018 · 3 comments
Closed

drag detection #3

flypenguin opened this issue Feb 9, 2018 · 3 comments

Comments

@flypenguin
Copy link

Love the extension. One killer thing though: I usually drag-and-drop text really often. If I do this, the tabs change from pinned to unpinned all the time, cause I seem to drag too slowly. I also don't want to increase the click wait time, cause that is annoying.

STEPS TO REPRODUCE:

  • click-drag text slowly

WHAT HAPPENS:

  • tab changes from pinned to unpinned or vice versa

WHAT SHOULD HAPPEN:

  • extension should detect considerable mouse movement, and NOT change pinned status. that should only happen when the mouse is not moved.

Does that sound sensible?

@jbhoot
Copy link
Owner

jbhoot commented Feb 9, 2018

Yes, I noticed this bug today. I am testing a simple fix currently, however, with a caveat - the add-on ignores any sort of mouse movements (mousemove event). So, long press wouldn't work if the mouse moves even a bit (which is quite likely to happen on a touchpad).

Anyway, I am updating the add-on on AMO with this fix, banking on the rationale, that fewer people would be affected by the 'even a slight movement not allowed' bug than 'tab pins unwittingly during text selection or drag-and-drop'.

I will patch another version by the end of the weekend that allows for a slight mouse movement.

Let me know if you have a better idea.

@jbhoot
Copy link
Owner

jbhoot commented Feb 9, 2018

v3.1 pushed on AMO. Now, the add-on ignores mousemove event (entirely). I have created issue #4 to deal with allowing marginal mouse movement during long click/tap. Drop a comment in case you find a need to keep the current issue open.

@jbhoot jbhoot closed this as completed Feb 9, 2018
@flypenguin
Copy link
Author

flypenguin commented Feb 9, 2018 via email

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