On Tue, 17 Jun 2025 14:54:46 GMT, Alexey Ivanov <aiva...@openjdk.org> wrote:

>> Damon Nguyen has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   Rearrange and update test.
>
> test/jdk/java/awt/Cursor/CursorDragTest/ListDragCursor.java line 121:
> 
>> 119:         instructionsFrame.pack();
>> 120:         instructionsFrame.setLocation(300, 100);
>> 121:         instructionsFrame.setVisible(true);
> 
> I suggest displaying the frame in the centre of the screen
> 
> 
> instructionsFrame.setLocationRelativeTo(null);
> 
> 
> and then position the test frame to the right of the instruction frame. This 
> would align to the most common layout of manual tests and would adjust 
> positioning to the size of the current screen.
> 
> Create the instruction frame first to be able to use its location to position 
> the test frame.

Updated!

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25705#discussion_r2152716941

Reply via email to