On Thu, 17 Sep 2026 21:16:07 GMT, Ziad El Midaoui <[email protected]> 
wrote:

>> tests/manual/printing/PrintOrientTest.java line 109:
>> 
>>> 107:         Button print = new Button("Print");
>>> 108:         print.setLayoutX(80);
>>> 109:         print.setLayoutY(300);
>> 
>> Setting a manual layout is almost always a bad idea, usually just using a 
>> box and alignment is better
>
> This PR is for instructions changes only and the only reason why I changed 
> the layout was because the button was hiding some instructions. I will keep 
> this change for another PR.

Can you file a follow-up cleanup bug? Or expand 
[JDK-8392582](https://bugs.openjdk.org/browse/JDK-8392582) to cover it?

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

PR Review Comment: https://git.openjdk.org/jfx/pull/2315#discussion_r4047077979

Reply via email to