<Swing Dev> [9] Review Request for 8129940: JRadioButton does not honor non-standard FocusTraversalKeys
Semyon Sadetsky
semyon.sadetsky at oracle.com
Thu Jul 2 12:53:03 UTC 2015
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