Could I get a one more review of this? Thanks and regards, Shashi
-----Original Message----- From: Sergey Bylokhov Sent: Friday, October 5, 2018 4:47 AM To: Shashidhara Veerabhadraiah <shashidhara.veerabhadra...@oracle.com>; awt-...@openjdk.java.net; swing-dev@openjdk.java.net Subject: Re: <Swing Dev> [12] JDK-8061359: [macosx] Checkbox toggles on Space press but does not spoken by Voice Over 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.