On Tue, 10 Jun 2025 00:42:21 GMT, Harshitha Onkar <hon...@openjdk.org> wrote:

>> 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:                 dragging is complete.
> 
> The new instructions doesn't seem to hold true when I tested on windows.
> Additionally the instructions require rephrasing for clarity?

> In this PassFailJFrame it looks like it "works" correctly for 1 second, and 
> then the cursor updates anyway afterwards. I'm going to try to look into why 
> this is, but is this what you are seeing too @honkar-jdk ?

Looks like there are 3 scenarios here and based on how fast or slow the cursor 
is moved and cursor update.

1) Move the cursor without mouse drag (i.e without clicking left mouse btn). 
The cursor update is immediate.

2) Move the cursor quickly across the components with mouse drag (i.e with left 
mouse btn pressed) The cursor updates when you are midway into the components 
which implies cursor update is slow but it ultimately shows the correct cursor.

3) Move the cursor slowly across the components with mouse drag (i.e with left 
mouse btn pressed) cursor does get updated on entering the components as 
expected similar to case 1.

I'm probably guessing the mouse drag action creates a delay in how soon the 
cursor is updated.

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

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

Reply via email to