On Sun, 10 Aug 2025 12:58:58 GMT, Prasanta Sadhukhan <psadhuk...@openjdk.org> wrote:
> Hi all, > > This pull request contains a backport of commit > [e29346db](https://github.com/openjdk/jdk/commit/e29346dbd6328dcadc347a70d8c06ce141efef02) > from the [openjdk/jdk](https://git.openjdk.org/jdk) repository. > > The commit being backported was authored by Prasanta Sadhukhan on 7 Aug > 2025 and was reviewed by Phil Race, Alexander Zuev and Abhishek Kumar. > > > The issue is a manual JCK test fails due to lack of demarcation between > selected and unselected JRadioButtonMenuItem and JCheckboxMenuItem whereby > JDK does not draw bullet/checkmark in WIndows 11 if imgage icon is present as > the (position) offset to draw them is same as the image icon position. > Fix is made to ensure the bullet/checkmark is drawn followed by image icon > followed by menu text so that user can distinguish between selected and > deselected radiobutton/checkbox menuitem > > CI testing and manual conformance test along with SwingSet2 demo testing is ok src/java.desktop/share/classes/com/sun/java/swing/SwingUtilities3.java line 258: > 256: public static void setAcceleratorSelectionForeground(Color > acceleratorSelectionFg) { > 257: acceleratorForeground = acceleratorSelectionFg; > 258: } I just pointed this out on the main bug but this looks like a copy/paste error. It should be acceleratorSelectionForeground. To Alexei's point, why can't we pass these as parameters ? ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/26717#discussion_r2267225024