Community
Participate
Working Groups
I20080805-1307 The hovers in the QuickDiff ruler, including the CVS Annotation hovers, are not accessible via keyboard / screen reader. We should add a command that opens the QuickDiff hover for the cursor line, e.g. "Show QuickDiff Description", like Edit > Show Tooltip Description. Could also add this to the ruler context menu (Ctrl+F10). See also bug 78244 and bug 236440.
Created attachment 135376 [details] Fix Adds a "Show Change Ruler Tooltip" command, which users can bind to a shortcut. Filed bug 275890 to share the code I had to copy and to make the command ID and constant API.
Nice work! Some issues I found: - I know the ATE does it the same (filed bug 276065) but in the new code - it would be better to only create the information presenter if need. - we should uninstall and dispose it when the editor is closed - widgetOffset2ModelOffset(...) can return -1; since we catch the case where the document might be 'null' we should also catch the -1 case - makeAnnotationHoverFocusable(...) returns a boolean which is not used - "comp" is a bad name for a ruler ;-) - also document private methods - widgetOffset2ModelOffset - added note to bug 270475 that we need to document that nothing happens when there's no change - sticky hover ==> focused hover +1 for RC1 assuming above items get addressed.
(In reply to comment #2) Fixed these issues and released to HEAD. > - sticky hover ==> focused hover I also fixed this for the existing 'Show Tooltip Description' action.
Verified in I20090514-2000.
