Skip to content

fix(frontend): dismiss batch claim drawer with Escape and backdrop - #1607

Open
Heazzy500 wants to merge 1 commit into
LabsCrypt:mainfrom
Heazzy500:fix/issue-1484
Open

Heazzy500 wants to merge 1 commit into
LabsCrypt:mainfrom
Heazzy500:fix/issue-1484

Conversation

@Heazzy500

Copy link
Copy Markdown
Contributor

Adds standard modal dismissal behavior to BatchClaimDrawer: Escape closes the drawer and clicking the backdrop closes it without submitting claims. Includes interaction coverage for both paths.

Closes #1484

@Heazzy500

Copy link
Copy Markdown
Contributor Author

Kindly review and close the issue

@Heazzy500

Copy link
Copy Markdown
Contributor Author

@Caneryy @LawalRahman @Oluwatomilola
Pls kindly review and close the issue

@Heazzy500

Copy link
Copy Markdown
Contributor Author

@LabsCrypt I synced this PR with the latest main to address the stale dependency and contract CI state. The PR is mergeable now; please review the updated checks and merge when they are green.

@Heazzy500

Copy link
Copy Markdown
Contributor Author

@Caneryy @LawalRahman @Oluwatomilola I synced this PR with current main and fixed the lint failure by typing the fixture as Stream instead of using any. The focused ESLint check and drawer tests pass (4/4), and Frontend CI is green on the new head. Please review and merge once the remaining checks finish.

@Heazzy500
Heazzy500 force-pushed the fix/issue-1484 branch 2 times, most recently from f4a8698 to 5c149ce Compare October 15, 2026 01:02

This branch has not been deployed

No deployments
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.

[Frontend] Support Escape key and click-outside backdrop dismissal for BatchClaimDrawer

2 participants