Bug 510371 - "Show revision information" menu entry action should support hide revision instead creating a sub entry inside "revisions"
Summary: "Show revision information" menu entry action should support hide revision in...
Status: CLOSED DUPLICATE of bug 454476
Alias: None
Product: EGit
Classification: Technology
Component: UI (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-01-12 10:11 EST by Albert Tregnaghi CLA
Modified: 2017-01-18 07:45 EST (History)
2 users (show)

See Also:

Attachments
Just a screenshot to understand text better (12.71 KB, image/png)
2017-01-12 10:11 EST, Albert Tregnaghi CLA 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Albert Tregnaghi CLA 2017-01-12 10:11:19 EST 
Created attachment 266277 [details]
Just a screenshot to understand text better

Currently you got the context menue at ruler column containing

"Show revision information"
("Revisions" is not shown)

When now clicking on "Show revision information" revisions are shown correctly. 

When you want to hide the revision information you cannot click again to "Show revision information" item but you must know that there has  another
menu entry appeared called "Revisions" now. "Revisions" has got a sub entry 
"Hide Revision information".

I feel this not very intuitive.

The "Show line numbers" entry in context menu does it better: Its a toggle menu entry with a "check" icon to show the state.

I would prefer to have it same way with "Show revision information": Mark as checked when revision information is viewed. When clicking again on it, turn the revision information off and hide the "check" icon.
Comment 1 Andrey Loskutov CLA 2017-01-12 10:20:58 EST 

*** This bug has been marked as a duplicate of bug 454476 ***
Comment 2 Leo Ufimtsev CLA 2017-01-17 17:53:47 EST 
@Albert, consider voting for the bug this was linked to. Might get more attention. (I submitted a duplicate also..)
Comment 3 Albert Tregnaghi CLA 2017-01-18 07:45:55 EST 
(In reply to Leo Ufimtsev from comment #2)
> @Albert, consider voting for the bug this was linked to. Might get more
> attention. (I submitted a duplicate also..)

@Leo - I voted as suggested.