Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Modal] Pressing the ESC key won't close all open modals anymore #118

Merged
merged 1 commit into from
Sep 13, 2018

Conversation

prudho
Copy link
Contributor

@prudho prudho commented Sep 13, 2018

Description

In case of nested modals, pressing the ESC key will only close the last opened modal.

Preview

Normal ESC preview

Closed Issues

Semantic-Org/Semantic-UI#5768

@prudho prudho added type/feat Any feature requests or improvements lang/javascript Anything involving JavaScript state/awaiting-reviews Pull requests which are waiting for reviews labels Sep 13, 2018
@prudho prudho added this to the 2.5.1 milestone Sep 13, 2018
@prudho prudho self-assigned this Sep 13, 2018
Copy link
Member

@ColinFrick ColinFrick left a comment

Choose a reason for hiding this comment

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

LGTM

Still believe this is only part of a larger refactoring of the modal module.

@y0hami y0hami merged commit 67b75f1 into fomantic:beta Sep 13, 2018
@y0hami y0hami removed the state/awaiting-reviews Pull requests which are waiting for reviews label Sep 18, 2018
@y0hami y0hami mentioned this pull request Sep 20, 2018
@prudho prudho deleted the fix-5768 branch April 11, 2019 13:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lang/javascript Anything involving JavaScript type/feat Any feature requests or improvements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants