Looks fine.

On 01/10/2018 21:54, shashidhara.veerabhadra...@oracle.com wrote:
Hi Sergey, Thanks for your review and here is the updated webrev:

http://cr.openjdk.java.net/~sveerabhadra/8061359/webrev.01/

Thanks and regards,

Shashi


On 02/10/18 1:14 AM, Sergey Bylokhov wrote:
Hi, Shashi.
It looks like this line is not necessary;
"if(ptr != 0) {"
The prt is checked at the beginning of this method.

On 01/10/2018 02:54, Shashidhara Veerabhadraiah wrote:
Hi All, Please review a fix for an accessibility bug.

Bug: https://bugs.openjdk.java.net/browse/JDK-8061359

Webrev: http://cr.openjdk.java.net/~sveerabhadra/8061359/webrev.00/

Problem: The checkbox component state changes thro' keyboard space key were not getting voiced over. The problem was that those events(state property changes) were not processed and hence just ignored. Now added conditions to send changes for the state changes associated with checkbox component.

To test this, please use the swingset2 demo(or any swing application demo) and go to any checkbox control(JCheckBox) and select and deselect it by pressing space key. One should hear the component state, component name and component. Please make sure you have turned on the voice over on mac os(Command + F5).

Currently I am not sure to add any manual test to test this out. Please let me know if such a test needs to be added.

Thanks and regards,
Shashi






--
Best regards, Sergey.

Reply via email to