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

Alexey Ivanov aivanov at openjdk.org
Mon Aug 11 11:53:26 UTC 2025


On Mon, 11 Aug 2025 11:47:36 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:

>> src/java.desktop/share/classes/javax/swing/plaf/basic/BasicMenuItemUI.java line 723:
>> 
>>> 721:                         acceleratorSelectionForeground);
>>> 722:         SwingUtilities3.setAcceleratorForeground(acceleratorForeground);
>>> 723:         SwingUtilities3.paintAccText(g, lh, lr);
>> 
>> This is a really weird way… Pass the colors explicitly as parameters to the `SwingUtilities3.paintAccText` method.
>
> Again, the comment marked as resolved…
> 
> You have changed the implementation in `BasicMenuItemUI` but left it unchanged in other methods.

It's so confusing! This particular comment hasn't been resolved: you still pass parameters to `SwingUtilities3` via global static variables!

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

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


More information about the client-libs-dev mailing list