Skip to content

fix(Flyout): fix hover reliability across gap, safe area, and race conditions#598

Draft
egucciar wants to merge 1 commit intoreshaped-ui:canaryfrom
observeinc:flyout-fix-hover-edge-cases
Draft

fix(Flyout): fix hover reliability across gap, safe area, and race conditions#598
egucciar wants to merge 1 commit intoreshaped-ui:canaryfrom
observeinc:flyout-fix-hover-edge-cases

Conversation

@egucciar
Copy link
Copy Markdown
Contributor

@egucciar egucciar commented Mar 9, 2026

  • Adjust hover timing: instant open (0ms), 200ms leave grace period
  • Expand safe area polygon with trough rect and content rect checks
  • Add 50ms grace period to prevent single mousemove closing the menu
  • Remove disableTriggers/enableTriggers (broke adjacent hover menus)
  • Add openIntentRef to handle async render race condition
  • Clean up safe area on content mouse enter to prevent racing
  • Read safe area origin before cleanup to prevent data loss

Summary

Related Issue

Screenshots / Recordings

Notes for Reviewers

@egucciar egucciar force-pushed the flyout-fix-hover-edge-cases branch from c11453c to fd3bb1d Compare March 9, 2026 23:33
@egucciar egucciar changed the title [DRAFT] Flyout: fix hover reliability across gap, safe area, and race conditions fix(Flyout): fix hover reliability across gap, safe area, and race conditions Mar 10, 2026
…nditions

- Adjust hover timing: instant open (0ms), 200ms leave grace period
- Expand safe area polygon with trough rect and content rect checks
- Add 50ms grace period to prevent single mousemove closing the menu
- Remove disableTriggers/enableTriggers (broke adjacent hover menus)
- Add openIntentRef to handle async render race condition
- Clean up safe area on content mouse enter to prevent racing
- Read safe area origin before cleanup to prevent data loss
@egucciar egucciar force-pushed the flyout-fix-hover-edge-cases branch from fd3bb1d to 91c9497 Compare March 10, 2026 00:56
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.

1 participant