Skip to content

OUT-2027: Close dropdown popover when clicking away - #76

Merged
SandipBajracharya merged 1 commit into
previewfrom
OUT-2027
Jul 21, 2025
Merged

OUT-2027: Close dropdown popover when clicking away#76
SandipBajracharya merged 1 commit into
previewfrom
OUT-2027

Conversation

@SandipBajracharya

@SandipBajracharya SandipBajracharya commented Jul 18, 2025

Copy link
Copy Markdown
Collaborator

Changes

  • Created a hook that triggers when clicked outside the popover

Testing Criteria

Loom

- created a hook that triggers when clicked outside the popover
@linear

linear Bot commented Jul 18, 2025

Copy link
Copy Markdown

@vercel

vercel Bot commented Jul 18, 2025

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
quickbooks-sync ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 21, 2025 11:08am
quickbooks-sync (dev) ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 21, 2025 11:08am

@SandipBajracharya SandipBajracharya changed the title improvement(OUT-2027): Close dropdown popover when clicking away OUT-2027: Close dropdown popover when clicking away Jul 18, 2025
}
}

document.addEventListener('mousedown', handleClickOutside)

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

IMO mouse down events are a bit inconsistent on mobile devices. Have you checked this on mobile devices?

@SandipBajracharya SandipBajracharya Jul 21, 2025

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Verified. It is working on the mobile devices.

@SandipBajracharya
SandipBajracharya merged commit 89bc2e5 into preview Jul 21, 2025
1 check passed
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