<Swing Dev> [9] Review Request for 8129940: JRadioButton does not honor non-standard FocusTraversalKeys
Alexander Zvegintsev
alexander.zvegintsev at oracle.com
Thu Jul 2 13:10:32 UTC 2015
+1
Thanks,
Alexander.
On 07/02/2015 04:01 PM, Alexander Scherbatiy wrote:
> On 7/2/2015 3: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:
>
> The fix looks good for me.
>
> Assuming that the new version is:
> http://cr.openjdk.java.net/~ssadetsky/8129940/webrev.01/
>
> Thanks,
> Alexandr.
>
>>
>> --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