RFR: 8348760: RadioButton is not shown if JRadioButtonMenuItem is rendered with ImageIcon in WindowsLookAndFeel [v4]
Prasanta Sadhukhan
psadhukhan at openjdk.org
Thu Jan 30 12:49:14 UTC 2025
On Thu, 30 Jan 2025 11:40:30 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:
>> Prasanta Sadhukhan has updated the pull request incrementally with one additional commit since the last revision:
>>
>> optimise
>
> src/java.desktop/windows/classes/com/sun/java/swing/plaf/windows/WindowsIconFactory.java line 891:
>
>> 889: Part part;
>> 890: if (type == JRadioButtonMenuItem.class) {
>> 891: part = Part.BP_RADIOBUTTON;
>
> The part `Part.BP_RADIOBUTTON` doesn't look right to me.
>
> The selected case above always uses the part `Part.MP_POPUPCHECK` — and it looks correct because this code paints a menu item (and radio or check menu items are usually in popup menus rather than on the menu bar).
You are right..It is not needed..I have removed..
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/23324#discussion_r1935552863
More information about the client-libs-dev
mailing list