RFR: 8348760: RadioButton is not shown if JRadioButtonMenuItem is rendered with ImageIcon in WindowsLookAndFeel [v33]
Alexey Ivanov
aivanov at openjdk.org
Fri Jul 11 21:00:47 UTC 2025
On Fri, 11 Jul 2025 18:05:57 GMT, Phil Race <prr at openjdk.org> wrote:
>> Prasanta Sadhukhan has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Adjust offset for varying size imageicon
>
> test/jdk/javax/swing/JMenuItem/TestRadioAndCheckMenuItemWithIcon.java line 31:
>
>> 29: * JRadioButtonMenuItem and JCheckboxMenuItem
>> 30: * is rendered with ImageIcon in WindowsLookAndFeel
>> 31: * @requires (os.family == "windows")
>
> Can't we run this with all L&Fs on all platforms ?
> What about it would be so different in some other case ?
>
> Also that last update for different sized icons isn't being tested - so can we vary the icon size in this test ?
Should we run this test for all L&Fs?
So far, Windows L&F was unique in the way that it combined both check marks / bullets and icons. If we implement it the new way so that all the check marks / bullets and menu icons align, the menu behaviour in Windows L&F will be the same as that in Metal and Nimbus at least.
`javax/swing/JMenuItem/RightLeftOrientation.java` tests a similar scenario. If the behaviour of Windows L&F is the same as in other L&F, we may just add this bug id to an existing test.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/23324#discussion_r2201835865
More information about the client-libs-dev
mailing list