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

Make edit card only close with cancel or save #2798

Merged
merged 1 commit into from
Feb 19, 2019
Merged

Make edit card only close with cancel or save #2798

merged 1 commit into from
Feb 19, 2019

Conversation

timmo001
Copy link
Contributor

Description

Stops dialog from closing when accidentally clicking outside of the dialog. This makes sure either cancel or save (or refresh of course) be the only way to close the dialog.

Checklist

  • Code is tested and works on my devices
  • All other functionality works as it should

@balloob
Copy link
Member

balloob commented Feb 19, 2019

Like it!

@balloob
Copy link
Member

balloob commented Feb 19, 2019

I was experimenting with it for other cards, but they didn't had a cancel button. We might want to add more cancel buttons to any dialog that is a form, and make them all modal.

@balloob balloob merged commit 90c09e9 into home-assistant:dev Feb 19, 2019
@ghost ghost removed the in progress label Feb 19, 2019
@balloob balloob mentioned this pull request Feb 19, 2019
@timmo001 timmo001 deleted the edit-card-close-when-closed branch February 19, 2019 22:50
@github-actions github-actions bot locked and limited conversation to collaborators Jul 7, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants