This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.
Summary: | "Window -> Select Document in -> Select Node in Projects" ?! | ||
---|---|---|---|
Product: | projects | Reporter: | Jesse Glick <jglick> |
Component: | Generic Projects UI | Assignee: | Jiri Rechtacek <jrechtacek> |
Status: | CLOSED FIXED | ||
Severity: | blocker | CC: | jrojcek, jtulach, mmirilovic, phrebejk |
Priority: | P3 | Keywords: | REGRESSION |
Version: | 4.x | ||
Hardware: | All | ||
OS: | All | ||
Issue Type: | DEFECT | Exception Reporter: | |
Bug Depends on: | |||
Bug Blocks: | 41535 | ||
Attachments: | proposed patch |
Description
Jesse Glick
2004-09-16 21:13:43 UTC
Please rollback the fix for the issue #48778 which caused this. Rather than rolling back the fix of issue #48778, suggest you just change the Presenter.Menu to use a different label than the action name. Created attachment 17778 [details]
proposed patch
In the patch, better to either call super.getMenuPresenter, or explicitly use Actions.connect; "bare" JMenuItem semantics misses some subtler things handled by Actions we might as well preserve, e.g. automatic mnemonic handling. Hmm, using Actions.connect() should be the correct way. However if I use it then there seems to be no way of having different name for the Presenter.Menu and the Action, which is what we need to achieve. That's bad. I have to implement it myself (with empty icon, Mneonics.setText() etc.). Any ideas? the attached patch has been applied. Checking in src/org/netbeans/modules/project/ui/actions/Bundle.properties; /cvs/projects/projectui/src/org/netbeans/modules/project/ui/actions/Bundle.properties,v <-- Bundle.properties new revision: 1.18; previous revision: 1.17 done Checking in src/org/netbeans/modules/project/ui/actions/SelectNodeAction.java; /cvs/projects/projectui/src/org/netbeans/modules/project/ui/actions/SelectNodeAction.java,v <-- SelectNodeAction.java new revision: 1.4; previous revision: 1.3 done This issue was solved long time ago. Because nobody has reopened it neither added comments, we are verifying/closing it now. If you are still able to reproduce the problem, please reopen. Thanks in advance. |