<Swing Dev> [13] Review Request: 8061381 [macosx] Accelerators does not spelled for JMenuItems by Voice Over
Krishna Addepalli
krishna.addepalli at oracle.com
Mon Feb 25 09:43:16 UTC 2019
+1
Sorry for the delay.
Thanks,
Krishna
> On 18-Feb-2019, at 8:01 AM, Sergey Bylokhov <Sergey.Bylokhov at oracle.com> wrote:
>
> Hi, Krishna.
>
> This bug is covered by the SwingSet2, where it was found, running under VoiceOver, similar to other bugs like JDK-7124301, JDK-8061359.
>
> On 30/01/2019 22:52, Krishna Addepalli wrote:
>> Hi Sergey,
>> The fix looks good. However, I think, its good that if you add a test case for this fix. The reproducer provided in the bug is good enough(ofcourse, with some addition of pass fail criteria as for manual tests).
>> Thanks,
>> Krishna
>>> On 31-Jan-2019, at 4:09 AM, Phil Race <philip.race at oracle.com> wrote:
>>>
>>> +1
>>>
>>> -phil.
>>>
>>> On 1/24/19 9:16 AM, Sergey Bylokhov wrote:
>>>> Hello.
>>>> Please review the fix for jdk 13.
>>>>
>>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8061381
>>>> Fix: http://cr.openjdk.java.net/~serb/8061381/webrev.00
>>>>
>>>> This fix adds the support of swing menu accelerators for Voice Over on macOS
>>>> (note that swing menu is used when the "apple.laf.useScreenMenuBar" is false).
>>>> If the property "apple.laf.useScreenMenuBar" is true we use the native menu which
>>>> is already supported by the VoiceOver.
>>>>
>>>> The fix will convert the accelerator to the string which is passed to the VO, the logic
>>>> which is used to access the KeyStroke and convert it to the string is similar to what
>>>> we use on windows in the accessbridge.
>>>>
>>>
>
>
> --
> Best regards, Sergey.
More information about the swing-dev
mailing list