Skip to content

fix(photo-annotator): widen handle hit-area, raise click-vs-drag threshold#1522

Merged
steilerDev merged 1 commit into
betafrom
fix/annotator-text-drag-handle-hit-area
May 19, 2026
Merged

fix(photo-annotator): widen handle hit-area, raise click-vs-drag threshold#1522
steilerDev merged 1 commit into
betafrom
fix/annotator-text-drag-handle-hit-area

Conversation

@steilerDev
Copy link
Copy Markdown
Owner

Summary

User reported handles are hard to hit and text shapes disappear when starting a drag.

  • Bumped the resize-handle hit-test tolerance from 8 → 14 image pixels (corner, endpoint, cardinal and callout-tail handles).
  • Raised the click-vs-drag threshold from 3 → 5 px so a tiny jitter when starting a drag doesn't get misclassified as a "click again to edit" and open the inline input on top of the text shape.

Test plan

  • All PhotoAnnotator unit tests pass (16 suites)
  • Typecheck green
  • Manual: select a shape, grab a handle in one go — easier to land. Drag a text shape — it tracks the cursor without disappearing.

…shold

Resize handles were rendered ~6-8 image pixels across but only had an
8 px hit tolerance — fiddly to grab. Bumped the tolerance to 14 px for
all handle hit-tests (corner, endpoint, cardinal, tail).

For text shapes the user reported the element disappears on drag.
Raised the click-vs-drag threshold from 3 to 5 image pixels so a small
involuntary jitter when starting a drag is treated as a drag rather
than misclassified as a second click that opens the inline editor.

Co-Authored-By: Claude frontend-developer (Haiku 4.5) <noreply@anthropic.com>
@steilerDev steilerDev merged commit 4b0b128 into beta May 19, 2026
26 of 30 checks passed
@steilerDev steilerDev deleted the fix/annotator-text-drag-handle-hit-area branch May 19, 2026 16:50
@github-actions
Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 2.6.0-beta.59 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant