Add icons for many context menu options
Added many icons for various context menu options:
- Editor Scene Tab Menu:
- Show in FileSystem
- Play This Scene
- Inspector Dock Menu:
- Resource Show in FileSystem
- Script Editor File Menu:
- Show in FileSystem
- Script Editor Scripts List Menu:
- Save
- Close
- Copy Script Path
- Show in FileSystem
- Move Up
- Move Down
- Sort
- Shader Editor Scripts List Menu:
- Show in FileSystem
I wonder if "Move Up" and "Move Down" are used frequently enough to warrant an icon.
I wonder if "Move Up" and "Move Down" are used frequently enough to warrant an icon.
That's fair! I'm mostly following the existing conventions of the Scene dock's node context menu:
I think having icons for all entries in context menus would help a lot with accessibility; I find that the Scene dock's menu does this well.
Most context menus either have icons for all items (with possible exceptions) or none.
Scene menu has only 2 icons:
It should not have icons at all, see #89747 for justification. Although technically it's a context menu, so not sure.
Script menu has only half of the icons:
It's probably fine to have them, but you should add icons to more items. Though it's difficult to justify having icons for 2 save buttons and 4 close buttons I guess.
Shader menu has single icon.
It's the same case as script menu. They should be consistent.
Script editor top menu should have no icons. Top menus just have no icons.
The only non-questionable change is the one in the inspector.