<Swing Dev> [9] Review Request for 8129940: JRadioButton does not honor non-standard FocusTraversalKeys
Yuri Nesterenko
yuri.nesterenko at oracle.com
Thu Jul 2 12:59:59 UTC 2015
Hi friends,
as to bug8075609.java, cound you look at my request
<AWT Dev> [9] Review Request for 8130125: [TEST_BUG] add @modules to the
several client tests unaffected by the automated bulk update
there it is fixed, I guess!
-yan
On 07/02/2015 03:53 PM, Semyon Sadetsky wrote:
> Hi Alexander,
>
> 1. It seems bug8075609.java is wrong . Tab key should not move focus to
> the next option in the group.
> 2. I have changed the if-statement to the shortcut you've proposed:
>
> --Semyon
>
> On 7/2/2015 3:13 PM, Alexander Scherbatiy wrote:
>>
>> - Could you look at the test
>> javax/swing/JRadioButton/8075609/bug8075609.java
>> It fails on my Windows system even without your fix
>> (RuntimeException: Focus is not on textField as Expected).
>> If it does not relate to the fix area we can create a separate
>> issue to it.
>>
>> - The 'if' statement on the line 609 can be simplified to return
>> keys != null && keys.contains(stroke)
>>
>> Thanks,
>> Alexandr.
>>
>> On 6/26/2015 4:22 PM, Semyon Sadetsky wrote:
>>> Hello,
>>>
>>> Please review fix for JDK9:
>>>
>>> bug: https://bugs.openjdk.java.net/browse/JDK-8129940
>>> webrev: http://cr.openjdk.java.net/~ssadetsky/8129940/webrev.00/
>>>
>>> It is a regression from 8033699. In this fix focus traversal keys
>>> were hard-coded in JRadioButton to be TAB and SHIFT+TAB only. But
>>> those keys can be amended by user as well. Solution : use the generic
>>> logic to identify focus traversal keys.
>>>
>>> --Semyon
>>
>
More information about the swing-dev
mailing list