Skip to content

Fix mouse ghost gap (delay cursor show) bug on Windows client - #8790

Merged
nbolton merged 6 commits into
masterfrom
ghost-gap
Jul 28, 2025
Merged

Fix mouse ghost gap (delay cursor show) bug on Windows client#8790
nbolton merged 6 commits into
masterfrom
ghost-gap

Conversation

@nbolton

@nbolton nbolton commented Jul 28, 2025

Copy link
Copy Markdown
Member

Fixes: #8788 (may help with #7948 and #6035)

Previously we were toggling the 'MouseKeys' Windows accessibility feature on and off, but this was causing many bugs; cursor show delay while Windows brings up the mouse stack(?) and often forever lost mouse cursor when toggling off the 'MouseKeys' feature. After a few days of hacking away at this, it seems safest to just leave it on.

nbolton added 5 commits July 28, 2025 16:49
The `ShowCursor` function does not directly show/hide the cursor; it increments or decrements the internal display counter. When that counter reaches a certain positive/negative number, then the cursor is shown or hidden at a particular threshold.
sithlord48
sithlord48 previously approved these changes Jul 28, 2025

@sithlord48 sithlord48 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good only one minor suggestion.

Previously we were toggling the 'MouseKeys' Windows accessibility feature on and off, but this was causing many bugs; cursor show delay while Windows brings up the mouse stack(?) and often forever lost mouse cursor when toggling off the 'MouseKeys' feature. After a few days of hacking away at this, it seems safest to just leave it on.
@nbolton
nbolton requested a review from sithlord48 July 28, 2025 19:05
@nbolton
nbolton enabled auto-merge (rebase) July 28, 2025 19:05
@nbolton
nbolton merged commit 51e467d into master Jul 28, 2025
31 checks passed
@nbolton
nbolton deleted the ghost-gap branch July 28, 2025 20:45
@sithlord48 sithlord48 mentioned this pull request Jul 29, 2025
12 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Cursor takes a few seconds to show on Windows client

2 participants