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

Error while selecting tabs in quick succession with control #174

Closed
rayman89 opened this issue Nov 6, 2017 · 6 comments
Closed

Error while selecting tabs in quick succession with control #174

rayman89 opened this issue Nov 6, 2017 · 6 comments

Comments

@rayman89
Copy link

rayman89 commented Nov 6, 2017

When selecting tabs in quick succession with control the addon sometimes would select an other random tab or clear the selection.

@piroor
Copy link
Owner

piroor commented Nov 30, 2017

It is an intentional behavior. If you click non-active tab item, the active tab or last clicked tab is also automatically selected. This is similar to Windows Explorer's file selection behavior.

However I agree that it is confusable on the panel. On Tree Style Tab's sidebar, there is visually active tab and the behavior above is natural. On the other hand, MTH's panel has no visual feedback for active or last clicked item, so there is no hint to know the behavior.

@piroor
Copy link
Owner

piroor commented Nov 30, 2017

By recent commits, now the last clicked tab has a visual feedback. Moreover, initially the current tab is treated as the last clicked tab by default. If you see any odd selection for not a last clicked tab, please add a comment with detailed steps to reproduce.

@rayman89
Copy link
Author

I tested a bit more and noticed what the "issue" is. When someone clicks and moves the mouse too quickly sometimes what ends up doing is dragging the mouse while holding down the click and the dragging while holding control on your addon has an other functionality. So for example if I have 5 tabs and selected 1 2 and 3 and then went to select 4 but actually instead of control click did drag the mouse while holding control and click the addon restarts the selection and selects only tab 4. So there is no real issue people would just need to either do it slower or use the drag selection from the start.

You can close this if you want or even better add a 200 ~ 300ms delay for the drag selection to prevent the 2 kinds of selection from conflicting with each other.

@piroor
Copy link
Owner

piroor commented Nov 30, 2017

Ctrl-drag and shift-drag to modify selection is not implemented yet, sorry.

@piroor
Copy link
Owner

piroor commented Nov 30, 2017

I didn't implemented those actions because basically MTH was designed to work on actual tabs and ctrl/shift-drag on tabs can be used to other actions. So only long-press to start selection by dragging was available. Now you'll see the behavior when you use MTH with Tree Style Tab.

To be honest, current WebExtensions-based MTH was originally made just for Tree Style Tab, aimed to reintroduce behaviors of the combination of legacy TST + MTH. Other usecase including the toolbar button popup are just optional features. I added those features with minimum effort, just to release WE-MTH as a standalone addon. Thus the panel is actually not enough useful.

Due to such a background, I have very few motivation to improve the panel itself. Instead I hope to spend more time to improve UX around the combination of MTH + TST, and I won't improve the panel aggressively. If you really want to use the panel as the primary UI, contribution from people who actually need that is really needed.

Current popup panel is based on a common codes shared to the behavior above

@rayman89
Copy link
Author

Seems to be a misunderstanding in our communication. I'm not requesting being able to control-drag. Selection is ok the way it's now what I asked was a really unimportant change so don't worry you can close this.

BTW I would love to contribute to the code so the changes I suggested could be implemented but my knowledge of programing is almost 0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants