<Swing Dev> [8] Review request for JDK-7132383: [macosx] bug6596966.java should be adapted for Mac
Alexander Scherbatiy
alexandr.scherbatiy at oracle.com
Thu Feb 7 13:18:54 UTC 2013
On 2/6/2013 3:15 PM, vera akulova wrote:
> Hello,
>
> as we discussed offline it is better to use
> SwingUtilities2.getSystemMnemonicKeyMask(), I suggest the following fix:
>
> http://cr.openjdk.java.net/~kshefov/7132383/webrev.01/
>
> Could you please review?
The fix looks good for me.
I would suggest to rename the getKeyCodesByKeyMask method to
getKeyCodesFromKeyMask and
add getSystemMnemonicKeyCodes() method to Util class because this is
often used in the tests.
Thanks,
Alexandr.
>
> Thanks,
> Vera
>
> On 11.12.2012 18:44, vera akulova wrote:
>> Hello,
>>
>> Please review a fix for the issue:
>> JDK-7132383: [macosx] bug6596966.java should be adapted for Mac
>>
>> The webrev is http://cr.openjdk.java.net/~kshefov/7132383/webrev.00/
>>
>> Ctrl-Alt-Mnemonic is used now for Mac instead of Alt-Mnemonic.
>> Fixed test works fine on windows, linux, solaris and macos.
>>
>> Thanks,
>> Vera.
More information about the swing-dev
mailing list