<Swing Dev> [8] Review request for JDK-7132383: [macosx] bug6596966.java should be adapted for Mac
vera akulova
vera.akulova at oracle.com
Thu Feb 7 14:32:55 UTC 2013
Hi Alexander,
thanks for your reply,
please review the new version of fix:
http://cr.openjdk.java.net/~kshefov/7132383/webrev.02/
Vera
On 07.02.2013 17:18, Alexander Scherbatiy wrote:
> 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