On Thu, 15 Jan 2026 00:39:24 GMT, Damon Nguyen <[email protected]> wrote:

> Although the if/else branches look strange, it works. I don't see a clear way 
> to clean up and condense the conditionals so lgtm unless someone else has an 
> idea.

I haven't looked at the updated code yet.

I wonder why the logic in `MenuItemLayoutHelper.java` isn't enough, why 
adjustments are needed.

As I posted in [my previous 
comment](https://github.com/openjdk/jdk/pull/28889#discussion_r2694197701), I'm 
not against this approach as long as it works and fixes the layout, yet I'd 
like to clean up the code if possible… in a follow-up issue.

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

PR Comment: https://git.openjdk.org/jdk/pull/28889#issuecomment-3754543967

Reply via email to