-
Notifications
You must be signed in to change notification settings - Fork 94
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
Meatballs #734
Comments
@Colin-Stone Yeah why not. makes sense. At least it will be cleaner |
Makes sense to me. What particular items would you like to see in the meatballs menu @Colin-Stone ? |
@jelaplan do you think that is something worth a UX design session to discuss what should go there? |
@VitGottwald sorry that I didn't see this. Zach pointe me to it. I don't monitor GitHub too closely. I think an icon menu is an attractive option for the button bar so we can add multiple related options. |
This is not a feature in VSCode yet, but I've requested it here: microsoft/vscode#99345 If we get 20 upvotes in 30 days they will add it to their backlog! So please go vote if you have time ^^ |
@katelynienaber Just to clarify: With the meatballs menu, is the design supposed to be that everything currently in the right-click context menu also goes into the meatballs menu? Or is the meatballs menu supposed to be for a separate set of items? |
@lauren-li I think we should bring it up in standup. What would make sense to me is to move all commands into the meatball menu, unless they are frequently used. But that's just me :) |
WRT hiding the icons, I would be ok but I don't think that is very popular. The usability testing we did revealed that people didn't notice the icons at first. But they are convenient once you know they are there. Because people didn't notice them and tended to go to the context menu when looking for something, I'd like to see the icon actions also on the menu as well as on icons. An advantage of the meatball menu is that it offers an affordance so if we put the menu items in there then it provides a ready way to find things. |
Update, this should be possible now. VSCode has added submenus |
After reviewing the Profiles PR's we may also want to consider more menu options including the use of the meatball drop-down menus.
The example above is from the GitHub plugin but shows two useful menus at the highest level including a refreshAll button and the MeatBalls button which presents a dropdown menu. I did think adding delete profile and possibly other generic menus may be useful here as it doesn't require the user to know about the command palette.
@crawr @jellypuno what do you think as a future issue?.
The text was updated successfully, but these errors were encountered: