Skip to content

Improve digitizing cursor contrast and lock tools during tracking - #39

Merged
veillette merged 1 commit into
mainfrom
claude/fix-tracking-cursor-contrast-0UFrT
Feb 23, 2026
Merged

Improve digitizing cursor contrast and lock tools during tracking#39
veillette merged 1 commit into
mainfrom
claude/fix-tracking-cursor-contrast-0UFrT

Conversation

@veillette

Copy link
Copy Markdown
Collaborator

Apply the same dual-layer shadow technique used by CalibrationToolNode and
CoordinateSystemNode to the digitizing crosshair cursor. A dark shadow stroke
(4 px) renders beneath the white cursor stroke (1.5 px), giving good contrast
on both light and dark video backgrounds instead of relying on a single color
that only works on one background type.

Also prevent accidental interference with the calibration tool and coordinate
system while a track is being manually digitized: both nodes dim to 35% opacity
and become non-interactive (pickable: false) when activeTrackIdProperty is set,
restoring to full opacity and interactivity once digitizing ends.

https://claude.ai/code/session_01VAVzhrFTMNh4duuDe4v2uh

Apply the same dual-layer shadow technique used by CalibrationToolNode and
CoordinateSystemNode to the digitizing crosshair cursor. A dark shadow stroke
(4 px) renders beneath the white cursor stroke (1.5 px), giving good contrast
on both light and dark video backgrounds instead of relying on a single color
that only works on one background type.

Also prevent accidental interference with the calibration tool and coordinate
system while a track is being manually digitized: both nodes dim to 35% opacity
and become non-interactive (pickable: false) when activeTrackIdProperty is set,
restoring to full opacity and interactivity once digitizing ends.

https://claude.ai/code/session_01VAVzhrFTMNh4duuDe4v2uh
@veillette
veillette merged commit f146721 into main Feb 23, 2026
1 check passed
@veillette
veillette deleted the claude/fix-tracking-cursor-contrast-0UFrT branch February 23, 2026 09:20
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.

2 participants