Skip to content

Show local cursor while dragging the viewport #1992

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
redneck-f25 wants to merge 16 commits into
base: master
Choose a base branch
from
Open

Show local cursor while dragging the viewport #1992

Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
6eb28b8
Make local cursor accessible/configurable
redneck-f25 Sep 15, 2025
355dd27
Make RFB::dragViewport a private Variable with getter and setter
redneck-f25 Sep 15, 2025
0f48f31
Show local (grab/grabbing) cursor while dragging the viewport
redneck-f25 Sep 15, 2025
5782915
Do not compare to 'mousedown' on every 'mousemove'
redneck-f25 Sep 15, 2025
97c0ad4
Add view_drag parameter to enable dragging when connected
redneck-f25 Sep 15, 2025
abc7d63
Fix id of hidden checkbox noVNC_setting_view_drag
redneck-f25 Sep 17, 2025
254fff9
Revert "Show local (grab/grabbing) cursor while dragging the viewport"
redneck-f25 Sep 17, 2025
59e40b4
Revert "Make local cursor accessible/configurable"
redneck-f25 Sep 17, 2025
b1745d5
Drop a note about isTouchDevice and useFallback
redneck-f25 Sep 17, 2025
5a9a81c
Add __FORCE_NO_TOUCH_DEVICE__ to not use fallback
redneck-f25 Sep 17, 2025
4dd16ad
Always initalize canvas.style in case of showing local cursors
redneck-f25 Sep 17, 2025
b5b7ebe
Extend Cursor-API to show local cursors
redneck-f25 Sep 17, 2025
3b79d45
Add members for local cursors
redneck-f25 Sep 17, 2025
68a2db4
Refresh, attach, detach cursor on API-calls and mouse events
redneck-f25 Sep 17, 2025
4fc3fc9
Implement local cursors
redneck-f25 Sep 17, 2025
702302f
Add default local cursor set to UI
redneck-f25 Sep 17, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 5 additions & 1 deletion core/util/cursor.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -106,7 +106,7 @@ export default class Cursor {
}

clear() {
this._target.style.cursor = 'none';
this.setLocalCursor('none');
this._canvas.width = 0;
this._canvas.height = 0;
this._position.x = this._position.x + this._hotSpot.x;
Expand All @@ -115,6 +115,10 @@ export default class Cursor {
this._hotSpot.y = 0;
}

setLocalCursor(cursor) {
Copy link
Copy Markdown
Member

@CendioOssman CendioOssman Sep 16, 2025

Choose a reason for hiding this comment

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

I don't think this fits well with how this class normally operates. I think you'll need to extend the change() API. And probably provide a fallback image for whe we cannot rely on a browser rendered cursor.

Copy link
Copy Markdown
Author

@redneck-f25 redneck-f25 Sep 17, 2025

Choose a reason for hiding this comment

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

Thanks for the hint.

Reverted with 59e40b4
Proposal b5b7ebe

this._target.style.cursor = cursor;
}

// Mouse events might be emulated, this allows
// moving the cursor in such cases
move(clientX, clientY) {
Expand Down