Add a context menu for the shader editor file list #95738
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
I think this can be merged along with #84063, to make the shader editor's usability better |
Calinou
approved these changes
Aug 25, 2024
|
Noted! 👌 I'll also do a PR for "Sort". It was originally going to be part of this, but I decided to stick with the bare essentials for the initial context menu. |
|
Needs rebase (probably due to #84063). |
akien-mga
changed the title
SomeRanDev
force-pushed
the
shader_editor_context_menu
branch
from
bcd2f55 to
34949d4
Compare
SomeRanDev
force-pushed
the
shader_editor_context_menu
branch
from
34949d4 to
8a087c7
Compare
|
Rebased and ready to go (after checks)! 👍 |
|
Thanks! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I frequently right-click on items in the shader editor expecting a context menu like the script editor, and it's very jarring when nothing happens. This adds a basic context menu with the same structure as the one from the script editor; it includes a couple new options like "Close Other Tabs", "Close All", and "Show in File System."
Relevant issue: #75061