Skip to content

fix: prevent device long-press context menu on action tracker buttons#100

Merged
tytremblay merged 1 commit intomainfrom
ty/hold_fix
Mar 1, 2026
Merged

fix: prevent device long-press context menu on action tracker buttons#100
tytremblay merged 1 commit intomainfrom
ty/hold_fix

Conversation

@tytremblay
Copy link
Copy Markdown
Collaborator

  • Add onContextMenu preventDefault so OS context menu (Download/Share/Print) does not interrupt press-and-hold behavior on mobile
  • Add -webkit-touch-callout:none to suppress iOS callout on long press Fixes issue where holding >0.5s triggered device long-press instead of app hold, preventing release and data recording

Made-with: Cursor

- Add onContextMenu preventDefault so OS context menu (Download/Share/Print)
  does not interrupt press-and-hold behavior on mobile
- Add -webkit-touch-callout:none to suppress iOS callout on long press
Fixes issue where holding >0.5s triggered device long-press instead of
app hold, preventing release and data recording

Made-with: Cursor
@tytremblay tytremblay requested a review from luanzeba February 27, 2026 20:59
@github-actions
Copy link
Copy Markdown

PR Preview Action v1.8.1

QR code for preview link

🚀 View preview at
https://FRC2713.github.io/QRScout/pr-preview/pr-100/

Built to branch gh-pages at 2026-02-27 21:00 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

Copy link
Copy Markdown
Collaborator

@luanzeba luanzeba left a comment

Choose a reason for hiding this comment

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

Oh man, weird that this didn't trigger for me at all. I don't have an android device so just tested on iOS. It looks like this should fix it.

@tytremblay tytremblay merged commit 9edb6f3 into main Mar 1, 2026
1 check passed
@tytremblay tytremblay deleted the ty/hold_fix branch March 1, 2026 12:32
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