09.08.2012 06:09, J.S. Ferguson wrote:
Seems to fix the menu visual problem, but now actions are firing twice when the
accelerator is used as described in bug 7160951.
http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7160951
Yes this bug on our radar.
On Aug 7, 2012, at 4:39 AM, Sergey Bylokhov wrote:
Hi Everyone,
Please review the fix.
When we translate calls from our swing menu components to awt peer we resets
information about shortcut in the setLabel().
This happens because of ScreenMenuItem.setAccelerator() method call peers
setLabel(..,..,..) directly and does not initialize ScreenMenuItem.shortcut
property.
But default implementation of ScreenMenuItem.setLabel() assumes that this
field(shortcut) will be initialized.
This works on jdk6 because it does not reset shortcut if null or empty shortcut
is provided.
As a solution we can use peers methods directly in both cases.
Bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7186371
Webrev can be found at: http://cr.openjdk.java.net/~serb/7186371/webrev.00
--
Best regards, Sergey.
--
Best regards, Sergey.
