Register icons for git stash and nb shelve actions #8373
Conversation
mbien
added
UI
| // TODO git unstash in shelve action? | ||
|
|
||
| @NbBundle.Messages({ | ||
| "CTL_UnstashMenu.name=&Git Unstash", | ||
| "CTL_UnstashMenu.name.popup=Git Unstash", | ||
| "# {0} - stash index", "# {1} - stash name", "CTL_UnstashAction.name={0} - {1}" | ||
| }) | ||
| private static class UnshelveMenu { | ||
| private static class UnstashMenu { |
There was a problem hiding this comment.
this might indicate that things were copied around. Git unstash should not be in the shelve action.
this lead also to this:
which is not the patch storage, but the git unstash action (unshelve is actually registered a few lines before that)
but this would be for other PRs.
mbien
force-pushed
the
stash-and-shelve-action-icons
branch
2 times, most recently
from
4c5e954 to
a667b48
Compare
So that they can be added to the toolbar.
mbien
force-pushed
the
stash-and-shelve-action-icons
branch
from
a667b48 to
e78d951
Compare
| // TODO pick/create better icon | ||
| @StaticResource | ||
| private static final String ICON_RESOURCE = "org/netbeans/modules/git/resources/icons/get_clean.png"; //NOI18N |
There was a problem hiding this comment.
@eirikbakke having icons for "git stash" and "shelve changes" would be great I think. Possibly the clean icon combined with the diff icon or something like that? (not for NB 26, maybe some day later)
There was a problem hiding this comment.
Sure, I can look at that the next time I'm doing icon stuff.
Where does the word "shelve" come from? It sounds synonymous with "stash", but it's not an actual git command. Looking at this StackOverflow question, it's an IntelliJ term that seems to confuse users. Maybe it should be renamed in the NetBeans UI?
Git stash via jgit does also only support repo wide "stash push", while shelve works on the selection.
What is the "selection" referred to here?
There was a problem hiding this comment.
Shelve is used in Subversion, Perforce and Mercurial (with some extensions). They are used for similar purpose as git stash
There was a problem hiding this comment.
shelve moves changes into patch files and stores those locally, this should work on any VCS. Stash is just git stash. Shelve will only shelve the selected files. Stash will stash the whole repo since I don't think jgit supports file stashing yet (git does for quite a while).
There was a problem hiding this comment.
Thanks, maybe I understand now. Perhaps "Stash to Patch File" might be a better name?
There was a problem hiding this comment.
Here's the icon I drew for Shelve, if you think it makes sense:
For stash, we could use ide/git/src/org/netbeans/modules/git/resources/icons/stashes.png, which I have redrawn in SVG as follows:
(The latter is part of #8424 )
I can install those two icons in a separate PR later.
There was a problem hiding this comment.
Did this PR actually add the Stash and Shelve actions to a toolbar somewhere? Which toolbar is it, and how do I access it? (Can't see the context from the very cropped screenshot on top.)
There was a problem hiding this comment.
it only registered the icons for the actions, this is the precondition to add something to the toolbar. User has to do this manually via right click -> customize toolbar.
That is why it wasn't super important for me to pick the right icons. It is just for testing purposes mostly and to get an idea how useful shelve still is. We should update them to the new icons for NB 27 though.
There was a problem hiding this comment.
Ah, now I see. Thanks! One annoying thing about the main toolbar is that icons shown there must be prepared in both 16x16 and 24x24 versions. But I can do it for this action. (One would think that the SVG could just be scaled up 150%, but it doesn't look good, as borders end up with a non-standard thickness, and horizontal/vertical lines get pushed off the pixel grid.)
matthiasblaesing
left a comment
matthiasblaesing
left a comment
There was a problem hiding this comment.
While I agree, that wording/nesting/workflow might be improved, the target of this PR is clearly spelled out and improves situation for people needing the actions.
Updated icons would be great, but that is also independent from this PR and also independent from the wording.
Lets get this in, it is an improvement.
4 participants
So that they can be added to the tool bar.
The action registration in the context menu is a bit of a mess right now. Git stash via jgit does also only support repo wide "stash push", while shelve works on the selection. This isn't communicated anywhere in the UI though, so I updated the text for the shelve action to "Shelve selected Changes".
The way to get to the shelve action is via the stash window - which is super weird (one is global the other is selection based). Custom toolbar makes this easier.