Bug 25558 - [Editor Mgmt] Dropdown menus on forward/back nav buttons
Summary: [Editor Mgmt] Dropdown menus on forward/back nav buttons
Status: RESOLVED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 2.1   Edit
Hardware: All All
: P2 major (vote)
Target Milestone: 2.1 M4   Edit
Assignee: Lynne Kues CLA
QA Contact:
URL:
Whiteboard:
Keywords: usability
Depends on:
Blocks:
 
Reported: 2002-10-30 17:00 EST by Ron Baldwin CLA
Modified: 2002-11-20 18:06 EST (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ron Baldwin CLA 2002-10-30 17:00:24 EST
It would be nice to have dropdown menus containing the navigation history
attached to the forward/back toolbar buttons (similar to IE or Mozilla).
Comment 1 Nick Edgar CLA 2002-11-08 10:59:41 EST
Should limit drop-down length to 10.

Can use editor input's name as the text to show (editor input is available from 
NavigationLocation.input).  This relies on the implementation details of 
NavigationLocation though, rather than just what's declared in 
INavigationLocation (the input is just an Object).
It would be better if INavigationLocation had get/setText() to allow this text 
to be specified independently of the input, so that more detail could be 
shown.  E.g. JDT could say "MyClass.someMethod()" or "MyClass.java - someMethod
()" instead of just "MyClass.java".  NavigationLocation would set this to be 
the editor input's name by default, but subclasses or other callers could 
override (make get/setText public).

Should also use this text in the tooltip for the buttons: "Back to 
MyClass.java".
This would have to be updated on each navigation.
Comment 2 Lynne Kues CLA 2002-11-15 12:10:08 EST
Navigation history entries may not have all fields initialized.  The entry is 
only fully restored when an attempt to go to that location occurs, so will have 
to change the mememento saving/restoring to include name of entry for display 
purposes.
Comment 3 Lynne Kues CLA 2002-11-20 17:16:38 EST
Opened [Bug 26813] for necessary TextSelectionNavigationLocation changes.
Comment 4 Lynne Kues CLA 2002-11-20 17:19:13 EST
Only the getText() API will be added to INavigationLocation.  There is no need 
for setText().  Implementors of INavigationLocation can override getText().  
Comment 5 Lynne Kues CLA 2002-11-20 18:02:22 EST
As part of the fix for this feature, any "old" navigation history needs to be 
wiped out.  This is because locations are not restored until the location 
actually becomes active, so we will not have labels to display in the history 
list.
Comment 6 Lynne Kues CLA 2002-11-20 18:06:23 EST
In build > 20021120.