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.

Bug 48779

Summary: + is used in Keyboard Shortcuts dialog instead of -
Product: platform Reporter: Patrick Keegan <pkeegan>
Component: Dialogs&WizardsAssignee: Stanislav Aubrecht <saubrecht>
Severity: blocker CC: jhoffman
Priority: P3 Keywords: UI
Version: 4.x   
Hardware: All   
OS: All   
Issue Type: TASK Exception Reporter:
Bug Depends on:    
Bug Blocks: 58412    
Attachments: a proposed patch

Description Patrick Keegan 2004-09-09 14:54:52 UTC
The Sun convention for expressing keyboard
shortcuts is to use hyphens (-) between the keys,
not plus signs. E.g. Ctrl+K should be Ctrl-K
Comment 1 Marian Mirilovic 2004-09-09 15:03:44 UTC
Kayboard Shortcuts dialog ...
Comment 2 Jiri Rechtacek 2004-09-09 18:09:02 UTC
Note: see javadoc of java.awt.event.KeyEvent
* Returns a String describing the modifier key(s), such as "Shift",
* or "Ctrl+Shift".  These strings can be localized by changing the 
* file.
* @return string a text description of the combination of modifier
*                keys that were held down during the event
public static String getKeyModifiersText(int modifiers) {

Should we do find&replace in returned string from this jdk's method?
Comment 3 Jiri Rechtacek 2004-09-09 18:25:28 UTC
Created attachment 17533 [details]
a proposed patch
Comment 4 Jiri Rechtacek 2004-09-09 18:27:09 UTC
The attached patch contains code which Search&Replace all occurrences
of '+' in returned values. No problem to integrate it.
Comment 5 Patrick Keegan 2004-09-09 18:29:11 UTC
Interesting, Jirka. Thanks for the info. Does the hack you propose
have any performance implications or other dangers?

Jeff, should we file a bug against AWT?

Jirka, please don't integrate the patch yet.
Comment 6 Jiri Rechtacek 2004-09-10 08:54:32 UTC
Ad performance: I don't afraid of any performance regression if the
patch will integrated. The set of shortcuts with some modifiers is
limited, moreover the strings are short and not often recalculated.
Comment 7 Jiri Rechtacek 2004-10-22 09:39:19 UTC
No progress here. I change to PATCH type, I'm ready to integrate if
Comment 8 Patrick Keegan 2004-10-22 12:46:11 UTC
I'm switching this to promo E because I'm uncomfortable making a UI
change so long after UI freeze, especially when it's a hack of
something that really should be fixed in the JDK.
Comment 9 Quality Engineering 2008-12-23 14:32:41 UTC
This issue had *1 votes* before move to platform component
Comment 10 Stanislav Aubrecht 2012-04-26 07:37:36 UTC
Native applications on MS Windows use "+" chars in keyboard shortcuts, closing.