RFR: 8348760: RadioButton is not shown if JRadioButtonMenuItem is rendered with ImageIcon in WindowsLookAndFeel [v38]

Phil Race prr at openjdk.org
Mon Aug 11 15:54:36 UTC 2025


On Mon, 4 Aug 2025 05:31:46 GMT, Prasanta Sadhukhan <psadhukhan at openjdk.org> wrote:

>> When JRadioButtonMenuItem is called with imageIcon, then only imageIcon is shown without radiobutton in WIndowsLookAndFeel as there was no provision of drawing the radiobutton alongside icon.
>> If icon is not there, the radiobutton is drawn. Added provision of drawing the radiobutton windows Skin even when imageIcon is present.
>
> Prasanta Sadhukhan has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 42 commits:
> 
>  - Merge branch 'master' of https://git.openjdk.java.net/jdk into JDK-8348760
>  - Fix not needed background effect for Windows 10
>  - Remove Windows version check
>  - Merge master
>  - bullet/checkmark placement for varying imageicon size fix, test update
>  - Adjust offset for varying size imageicon
>  - squish fix
>  - Alignment fix
>  - MenuItem with icon fix
>  - Merge branch 'master' of https://git.openjdk.java.net/jdk into JDK-8348760
>  - ... and 32 more: https://git.openjdk.org/jdk/compare/57553ca1...5f51ecdc

src/java.desktop/share/classes/com/sun/java/swing/SwingUtilities3.java line 251:

> 249: 
> 250:     public static void setAcceleratorSelectionForeground(Color acceleratorSelectionFg) {
> 251:         acceleratorForeground = acceleratorSelectionFg;

I just noticed this "oops". This should be acceleratorSelectionForeground.
Do we not have a test that would spot this ?

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/23324#discussion_r2267210782


More information about the client-libs-dev mailing list