On Fri, 11 Jul 2025 18:05:57 GMT, Phil Race <p...@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

Reply via email to