On Tue, 10 Jun 2025 21:22:47 GMT, Damon Nguyen <dngu...@openjdk.org> wrote:

>> This change is to restore the original intent of the test by updating the 
>> instructions to check that the type of Cursor is preserved when clicked and 
>> dragged. Now the test correctly has instructions to check that an I-beam 
>> cursor stays an I-beam until released over a List with its cursor being 
>> updated to a Hand cursor.
>> 
>> There is a bug where this does not correctly update in macOS (found in 
>> [JDK-7177297](https://bugs.openjdk.org/browse/JDK-7177297)). So, this test 
>> needs to be problem-listed.
>> 
>> I have confirmed that preserving the cursor image when dragging is native 
>> behavior across macOS, Windows, and Ubuntu. And I have checked that the test 
>> passes on both Windows and Ubuntu, while macOS fails and immediately updates 
>> the cursor as it leaves the TextArea.
>
> Damon Nguyen has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Update PL num. Clarify instructions.

test/jdk/java/awt/Cursor/CursorDragTest/ListDragCursor.java line 52:

> 50:                 pass the test. This test fails if the cursor updates
> 51:                 when pointing over the different components before
> 52:                 releasing the left mouse button.

Suggestion:

                The mouse cursor appears as an I-beam cursor
                and stays the same while dragging across the components.
                Once you reach the list release the left mouse button, at that
                moment the cursor should change to a Hand Cursor.
                If the above is true press pass.
                
                 The test fails if the cursor updates while dragging
                 over the different components before releasing
                 the left mouse button.

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

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

Reply via email to