Bug 128409 - [CommonNavigator] Why is TextActionHandler duplicated in org.eclipse.ui.navigator?
Summary: [CommonNavigator] Why is TextActionHandler duplicated in org.eclipse.ui.navig...
Status: RESOLVED WONTFIX
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.2   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Michael D. Elder CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on: 124512
Blocks:
  Show dependency tree
 
Reported: 2006-02-17 09:37 EST by Douglas Pollock CLA
Modified: 2006-04-07 11:11 EDT (History)
1 user (show)

See Also:

Attachments
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 Douglas Pollock CLA 2006-02-17 09:37:56 EST 
Why is TextActionHandler duplicated in org.eclipse.ui.navigator?
Comment 1 Nick Edgar CLA 2006-02-17 12:55:54 EST 
TextActionHandler exists in the IDE, which the base CN framework can't depend on.
See also bug 124512.

Does CN's copy of TextActionHandler need to be exposed as API though?
Comment 2 Douglas Pollock CLA 2006-02-17 13:04:53 EST 
Yes, Bug 124512 covers what I'm thinking of: a separate plug-in.
Comment 3 Michael D. Elder CLA 2006-04-07 11:11:33 EDT 
The TextActionHandler in CN has a few minor enhancements (such as the Accelerator action text in the menu item (upcoming)) and some slight changes around when fill action bars is invoked (the CN must do this on selection change instead of just view activation).

The version in this layer is not exposed as API and there are not intentions of doing so. I am open to changing where the TextActionHandler used by the CN comes from, with the minor enhancements made taken into account.

Closing as WONTFIX for now. I suggest the discussion be held again in the next release in the context of bug 124512.