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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Convert config flow to Lit/TS #2814
Conversation
return html` | ||
<paper-dialog | ||
with-backdrop | ||
.opened=${true} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
always true?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, because people can click outside the dialog and that will set opened to false. We want the dialog to show as we will remove all DOM elements when there is nothing to show.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll have to finish reviewing tomorrow. I'm crazy tired and need sleep 馃槃
Co-Authored-By: balloob <paulus@home-assistant.io>
Co-Authored-By: balloob <paulus@home-assistant.io>
Was planning to add area selection to the config flow dialog, but realized we should first get it to Lit/TS before making any other changes, so here we are 馃憤