Gentle ping, any reviewers?
Regards,
Pankaj
On 30/07/20 3:34 PM, Pankaj Bansal wrote:
ping, any reviewers?
Regards,
Pankaj
On 26/07/20 4:24 PM, Pankaj Bansal wrote:
Hi All,
Please review the following fix for jdk16.
Bug : https://bugs.openjdk.java.net/browse/JDK-8249548
webrev: http://cr.openjdk.java.net/~pbansal/8249548/webrev00/
<http://cr.openjdk.java.net/%7Epbansal/8249548/webrev00/>
Issue: If few ToggleButtons are added to button group, the backward
focus traversal can get stuck in few scenarios while using
LayoutFocusTraversalPolicy. For exaple, if a TextField is added to a
frame and a Button Group is added with two buttons and the second
button in set selected, we can not move the focus from Button Group
to Text Field using "Shift+Tab". More details in JBS.
Cause: The LayoutFocusTraversalPolicy uses
SortingFocusTraversalPolicy always selects first button for focus
traversal in button group for Toggle Button. This can be changed by
Component if one of the button in the button group is set selected.
This results in the selected Button being selected again and again
for selection on pressing "Shift+Tab". This results in focus being
stuck at same position. This is a generic issue and it is observed in
all L&F for JToggleButton.
The JRadioButton does not have this issue as BasicRadioButtonUI and
AquaRadioButtonUI have code to handle this issue but the Synth L&F
has this issue for JRadioButton also. So, this issue can be observed
in both ToggleButton and RadioButton in nimbus L&F.
Fix: The fix is to add the code to handle this in BasicButtonUI
class, so that it is available for all L&F without copying it in
different L&F classes. The focus traversal works fine after this
change for all L&F for both ToggleButton and RadioButton. Along with
the current problem, this fix is also fixes issue like we cant change
the selected button and focus in ToggleButton group like RadioButton
group.
An automated test is added. I have ran the Mach5 with all tests and
this is not causing any failures. Links in JBS.
Regards
Pankaj