-
Notifications
You must be signed in to change notification settings - Fork 2
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
FR: Add a Sidebar tab with buttons #9
Comments
I think that makes sense, and I'm certainly not opposed to sidebars and buttons. |
🥳 |
A simple icon could be this one? https://fonts.google.com/icons?selected=Material%20Icons%20Outlined%3Amouse%3A |
also, clicking the buttons triggers the |
@chrisgrieser checkout 0.0.30 and let me know what you think!
|
I would avoid any functionality with repeated right clicks, since some mice aren't well suited to do that. (e.g. mine :P ) Since the select-all-command is something only used once anyway, you would mostly use the the sidebar for select next-commands, for which you want to run the command repeatedly. So I would suggest to solve the right-click-issue by simply assigning select-next to the left click, I think? |
Haha sure, I can swap those around |
Otherwise? I think the side view is looking fine |
Haha I prefer what you've done there! So unset the background color, and only show it on hover, right? |
@chrisgrieser I've swapped around right and left click functionality. Also changed styling to similar to what you've got |
<3 |
Gonna consider this closed then :) |
This is a very unusual request, coming from a theme dev who pretty much eliminates all the buttons he can find, but I think this plugin is one of the rare cases where buttons really make sense from a UI perspective. Here is why:
This means, one would need an alternative method of quickly and repeatedly calling a command while also having an overview of what commands there are created by this plugin. The only thing that came to my mind is basically a bunch of buttons, in a sidebar, which you can click repeatedly.
The text was updated successfully, but these errors were encountered: