Dragging mouse to select text in script editor may randomly change selection range and trigger drag about godot HOT 5 CLOSED

Bisected #92424 as the first bad commit.

from godot.

I can't reproduce (Godot v4.3.beta (475248d) - Windows 10.0.22631 - Vulkan (Forward+)).
But it looks like TextEdit is receiving mouse unpressed and pressed notifications when you didn't actually release the mouse button.
Does this also affect actions like panning or selecting in the scene view, or using ui sliders, or is it only the script editor?

from godot.

Bisected #92424 as the first bad commit.

Thanks for bisecting. I suspect it is because the patch uses GetAsyncKeyState instead of GetKeyState. Hypothetically, if one press the mouse button while moving the cursor, the window could be receiving these messages (pseudo code):

1. MouseMove, buttons=None, xy=(100, 100)
2. MousePress(Left), buttons=LeftMouse, xy=(101, 101)
3. MouseMove, buttons=LeftMouse, xy=(102, 102)

When the application processes the first message, it is possible that the user has already pressed the mouse button so GetAsyncKeyState(VK_LBUTTON) may indicate the mouse button is pressed, different from what the message represents. More importantly it would indicate the mouse button being held down before the MousePress message had been processed. If the control relies on the MousePress message to initialize states related to text selection, it could explain the bug. (This is just a hypothesis -- I haven't checked what is happening within Godot.)

I think it really should use GetKeyState instead.

I can't reproduce [...]. But it looks like TextEdit is receiving mouse unpressed and pressed notifications when you didn't actually release the mouse button. [...]

In my capture, when the selection stops changing I definitely released the left mouse button. If my hypothesis above is correct, then the key point is that the mouse must be moving when pressing the mouse button to reproduce the bug.

If you really need to verify, I can make a capture with an OSD showing the mouse buttons (though I don't currently have a tool for this so I'll need to hunt for one).

from godot.

Can confirm in v4.3.beta.custom_build [475248d]
Windows 10.0.19045

from godot.

I misunderstood earlier, I can reproduce it now while moving the mouse.

Adding logpoints show that a mouse motion event with the left mouse button down is being sent before the mouse pressed event.

Created a PR #93105 that uses GetKeyState as suggested.

from godot.