-
Notifications
You must be signed in to change notification settings - Fork 11
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
1-click "Tabs aside" #83
Comments
I don't think this should be the default behavior (new users, accidental click, ...) but I could see this being an option that can be enabled. |
Thats completely fine if its optional, just p.s.: if it works smooth enough, thats not an issues (it is not like deleting something) . |
The other thing is that the browser action context menu is limited to 6 entries, that would mean moving the quick restore menu items into a submenu. |
isn't it already too low anyway ? (so prompt to move the sessions) |
I 2nd irvnriir thought, that the Tabs aside entry should be the 1-click default. I used OneTab before where this is the default beahviour. There the entry limitation is solved by shifting every menu item in a single ("root") submenu. So you right click the icon > hover the single submenu entry > and then have 10 entries with one of them beeing a further submenu. Its less then a second for the first submenu to open and since its just were the mouse is anyway it doesn't bother (me) at all. It would be cool to have that in tabs-aside. PS: biggest downside of OneTab is no sync and no active sessions PS2: you've done a great job with this app! |
If I find the time for it I will add an option for this. But unfortunately I do not have a lot of time for Tabs Aside right now so that might take a while. |
LMB click on the toolbar item should { Save And Close / Close (if saved) } the tabs, while the current menu there -- should be in the Context Menu (before/after the current Sessions list) .
Before #87 is fixed, it should have the relevant indication on the icon .
The text was updated successfully, but these errors were encountered: