On Thu, 30 Oct 2025 02:11:24 GMT, Prasanta Sadhukhan <[email protected]> 
wrote:

>> I just tested the latest version of the fix with the slightly modified test 
>> case (i added two extra menu items with 
>> `menuItem.setHorizontalTextPosition(SwingConstants.LEFT);`
>> and
>> `menuItem.setHorizontalTextPosition(SwingConstants.RIGHT);`
>> and it does not look exactly correct. Here's the screenshot:
>> <img width="331" height="511" alt="image" 
>> src="https://github.com/user-attachments/assets/8494241e-aefd-4259-bcb2-8b3cf2360d7d";
>>  />
>
> Fixed..it now looks ok to me...Please check
> 
> <img width="391" height="614" alt="image" 
> src="https://github.com/user-attachments/assets/46661a18-947b-4483-969d-c7d05ace3ff0";
>  />

Yes, new version works correctly. I would suggest to add these menu items to 
the regression test but that is optional.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27968#discussion_r2479381292

Reply via email to