Skip to content

modal.js: Prevent unintentional closing#5501

Open
nilmerg wants to merge 1 commit intomainfrom
feature/prevent-unintentional-closing-of-a-modal-5307
Open

modal.js: Prevent unintentional closing#5501
nilmerg wants to merge 1 commit intomainfrom
feature/prevent-unintentional-closing-of-a-modal-5307

Conversation

@nilmerg
Copy link
Copy Markdown
Member

@nilmerg nilmerg commented May 6, 2026

This tracks the user's interactions by observing the events keydown, paste and change to detect changes to forms inside a modal. Upon any change, the modal cannot be closed anymore by pushing Escape or clicking outside the modal. Instead, the modal will wobble for a short period.

resolves #5307

requires

@nilmerg nilmerg added this to the 2.14.0 milestone May 6, 2026
@nilmerg nilmerg requested a review from sukhwinder33445 May 6, 2026 09:35
@nilmerg nilmerg self-assigned this May 6, 2026
@nilmerg nilmerg added the enhancement New feature or improvement label May 6, 2026
@cla-bot cla-bot Bot added the cla/signed label May 6, 2026
This tracks the user's interactions by observing the events
`keydown`, `paste` and `change` to detect changes to forms
inside a modal. Upon any change, the modal cannot be closed
anymore by pushing Escape or clicking outside the modal.
Instead, the modal will *wobble* for a short period.

resolves #5307
@nilmerg nilmerg force-pushed the feature/prevent-unintentional-closing-of-a-modal-5307 branch from faaa549 to 256caeb Compare May 6, 2026 10:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla/signed enhancement New feature or improvement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Prevent unintentional closing of a modal

1 participant