Hi Robin,
Looks nice. We probably want this into preferences and I'd still like the tooltop as an option. I.e. when you turn off
the other panes the tooltip should come back. What I'd really want is the "Press F2"-version, like the javadoc
tooltips, but that requires some more work (I guess, maybe only a little is needed).
Showing the tooltips when you press F2 makes more sense to me. What I'm
confortable to do now is to add another option in the history menu
(after show revision comment) to show the tooltips. I can send a new
patch with this.
For the search bar, I would like to make it visible by pressing ctrl-f
when the history panel has the focus. So if I learn how to do it I can
also do the F2 for the tooltips (low priority though).
+ /* private */TextViewer revDetailTextViewer;
+ /* private */TextViewer revCommentTextViewer;
+ /* private */IAction toggleCommentWrapAction;
+ /* private */IAction toggleRevDetailAction;
+ /* private */IAction toggleRevCommentAction;
- private Table table;
+ /* private */Table table;
- private List<IFileRevision> fileRevisions;
+ /* private */List<IFileRevision> fileRevisions;
- protected boolean hintShowDiffNow;
+ /* private */boolean hintShowDiffNow;
Why? What's wrong plain private?
This is to improve performance and getting rid of the warning:
Read access to enclosing field GitHistoryPage.appliedPatches is emulated
by a synthetic accessor method. Increasing its visibility will improve
your performance
[]s,
Roger.
-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
