Bug 539453 - "Copy commit id" menu item on history view without selection
Summary: "Copy commit id" menu item on history view without selection
Status: RESOLVED FIXED
Alias: None
Product: EGit
Classification: Technology
Component: UI (show other bugs)
Version: 5.2   Edit
Hardware: All All
: P3 trivial (vote)
Target Milestone: 5.2   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-09-26 03:37 EDT by Michael Keppler CLA
Modified: 2018-09-29 03:57 EDT (History)
2 users (show)

See Also:


Attachments
screenshot (2.89 KB, image/png)
2018-09-26 03:37 EDT, Michael Keppler CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Keppler CLA 2018-09-26 03:37:07 EDT
Created attachment 275986 [details]
screenshot

1. Open the history view
2. Do not select any commit in the view
3. Right click on the column headers.

The context menu contains a "copy commit id" entry. It should not be visible, since there is no selection. If the menu item is used, there is no content in the clipboard.
Comment 1 Matthias Sohn CLA 2018-09-26 04:44:39 EDT
I think the action should be shown but disabled if there is no selection
Comment 2 Eclipse Genie CLA 2018-09-26 13:00:33 EDT
New Gerrit change created: https://git.eclipse.org/r/130034
Comment 3 Thomas Wolf CLA 2018-09-28 04:21:13 EDT
(In reply to Matthias Sohn from comment #1)
> I think the action should be shown but disabled if there is no selection

Why should it be shown?
Comment 4 Matthias Sohn CLA 2018-09-28 09:02:28 EDT
you are right, since there is only a single action which doesn't make sense when there is no selection, no context menu should be shown at all.

I was thinking about a menu where some entries cannot be used due to some property of the selected item, in this case it helps users to always show all the actions to be expected on this type of item and disable the ones which cannot be used for the selected item since it makes users aware of the available actions for this item type. But in this case there is only a single action, and when no item is selected there is no such thing like an item type so what I said earlier doesn't make sense.
Comment 5 Eclipse Genie CLA 2018-09-29 03:56:29 EDT
Gerrit change https://git.eclipse.org/r/130034 was merged to [master].
Commit: http://git.eclipse.org/c/egit/egit.git/commit/?id=139b78cda2797f1024b4c35a8b8ac70aa6bb0406