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

Alexey Ivanov aivanov at openjdk.org
Mon Aug 11 16:25:36 UTC 2025


On Mon, 11 Aug 2025 15:51:05 GMT, Phil Race <prr at openjdk.org> wrote:

>> 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 ?

Yes, you're right!

I was curious why my IDE showed `acceleratorSelectionForeground` in grey as if unused. It's used but never assigned.

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

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


More information about the client-libs-dev mailing list