Skip to content

Consolidate button#2203

Closed
iantrich wants to merge 8 commits intohome-assistant:devfrom
iantrich:consolidate-button
Closed

Consolidate button#2203
iantrich wants to merge 8 commits intohome-assistant:devfrom
iantrich:consolidate-button

Conversation

@iantrich
Copy link
Copy Markdown
Member

@iantrich iantrich commented Dec 6, 2018

This PR updates the service-button element to a button element, with all the same functionality as the entity-button card. The entity-button card has been renamed to button card and just wraps the button element now as the setting of an entity is now optional.

If accepted, I plan on updating my outstanding documentation PR related to tap/hold actions: home-assistant/home-assistant.io#7751

home-assistant/ui-schema#211

@ghost ghost assigned iantrich Dec 6, 2018
@ghost ghost added the in progress label Dec 6, 2018
Comment thread src/common/entity/compute_state_domain.ts Outdated
@balloob
Copy link
Copy Markdown
Member

balloob commented Dec 6, 2018

Still need to do a bit more testing

So wise.

Comment thread src/common/entity/compute_state_name.ts Outdated
@balloob
Copy link
Copy Markdown
Member

balloob commented Dec 6, 2018

Can you include some description what this PR does?

@iantrich iantrich added the wip label Dec 6, 2018
@iantrich iantrich removed the wip label Dec 7, 2018
@iantrich
Copy link
Copy Markdown
Member Author

iantrich commented Dec 7, 2018

Done testing. Side note, after this gets merged, need to go and update the tap/hold gallery demos

@iantrich iantrich added this to the 0.84 milestone Dec 7, 2018
@balloob balloob removed this from the 0.84 milestone Dec 10, 2018
@iantrich iantrich closed this Feb 6, 2019
@iantrich iantrich deleted the consolidate-button branch October 30, 2019 03:18
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 6, 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.

3 participants