On Fri, 23 Jan 2026 05:11:37 GMT, Prasanta Sadhukhan <[email protected]> 
wrote:

> Arrow icon of JMenuItem in JMenu which is used to invoke the submenu overlaps 
> with the menutext if the text is long.
> 
> Fix is made to add a gap for arrow icon rect too similar to menu text and 
> accelerator rects

Marked as reviewed by kizune (Reviewer).

With the fix it is better than without the fix - that is for sure, but the 
overall layout looks strange and majority of the arrow icon is being hidden 
behind the submenu's popup which is wrong. I am approving it as a stopgap 
solution but the overall we might want to re-visit how we calculate layouts of 
menu items to make it look more like a OS native menus.

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

PR Review: https://git.openjdk.org/jdk/pull/29375#pullrequestreview-3723843212
PR Comment: https://git.openjdk.org/jdk/pull/29375#issuecomment-3818883539

Reply via email to