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

Convert hui-divider-row to TypeScript/LitElement #1896

Merged
merged 4 commits into from
Oct 28, 2018

Conversation

iantrich
Copy link
Member

No description provided.

@ghost ghost assigned iantrich Oct 28, 2018
@ghost ghost added the in progress label Oct 28, 2018
@iantrich
Copy link
Member Author

iantrich commented Oct 28, 2018

This is currently broken. It doesn't like how I am applying the style to the div using JS, takes it as an object

}

return html`
${this._createDivider()}
Copy link
Member

Choose a reason for hiding this comment

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

Why extract into a new method?

Copy link
Member Author

Choose a reason for hiding this comment

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

I couldn't figure out how to apply the passed into styles to the div without using setProperty 🤷‍♂️

Copy link
Member

Choose a reason for hiding this comment

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

So then you can still just copy the content of createDivider into render and then do:

return html`${el}`;

@balloob balloob merged commit b8752c4 into home-assistant:dev Oct 28, 2018
@ghost ghost removed the in progress label Oct 28, 2018
@iantrich iantrich deleted the ts-divider-row branch November 16, 2018 19:19
@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.

4 participants