Skip to content

OUT-2107: Close the list by clicking on a QB item cell a second time - #92

Merged
SandipBajracharya merged 1 commit into
previewfrom
OUT-2107
Aug 4, 2025
Merged

OUT-2107: Close the list by clicking on a QB item cell a second time#92
SandipBajracharya merged 1 commit into
previewfrom
OUT-2107

Conversation

@SandipBajracharya

Copy link
Copy Markdown
Collaborator

Changes

  • update useClickOutside hook to take exlcude refs that exclude the close action on clicked. Those exclude ref would be of table cells.
  • click on any other area except for dropdow and active open cell closes the dropdown

Testing Criteria

Loom

@linear

linear Bot commented Jul 31, 2025

Copy link
Copy Markdown

@vercel

vercel Bot commented Jul 31, 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 31, 2025 0:30am

@arpandhakal arpandhakal left a comment

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.

LGTM

@SandipBajracharya
SandipBajracharya merged commit 66448f0 into preview Aug 4, 2025
3 checks 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