[PR #6570] fix(web): Close popover on Escape press #6609

Closed
opened 2026-02-28 21:30:37 -06:00 by GiteaMirror · 0 comments
Owner

Original Pull Request: https://github.com/actualbudget/actual/pull/6570

State: closed
Merged: Yes


Summary

Fixes a bug where the filter popover cannot be closed using the Escape key when an Autocomplete component is inside the popover.

Details

The issue occurs because e.stopPropagation() prevents the Escape key event from reaching the popover. As a result, the user must manually close the popover.

Fix

Call stopPropagation() only when the dropdown is open; otherwise, allow the event to propagate to the parent popover.

Fixes #3039

**Original Pull Request:** https://github.com/actualbudget/actual/pull/6570 **State:** closed **Merged:** Yes --- ### Summary Fixes a bug where the filter popover cannot be closed using the Escape key when an Autocomplete component is inside the popover. ### Details The issue occurs because e.stopPropagation() prevents the Escape key event from reaching the popover. As a result, the user must manually close the popover. ### Fix Call stopPropagation() only when the dropdown is open; otherwise, allow the event to propagate to the parent popover. Fixes #3039
GiteaMirror added the pull-request label 2026-02-28 21:30:37 -06:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/actual#6609