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

feat(select-card) #885

Merged
merged 15 commits into from
Jan 10, 2023
Merged

feat(select-card) #885

merged 15 commits into from
Jan 10, 2023

Conversation

PaoloTK
Copy link
Contributor

@PaoloTK PaoloTK commented Dec 5, 2022

Description

This PR adds a select card to control select and input_select entities.

select-light

Related Issue

#873

This PR fixes or closes issue: fixes 873

Motivation and Context

SSIA

How Has This Been Tested

Demo instance and own instance.

Types of changes

  • 🐛 Bug fix (non-breaking change which fixes an issue)
  • 🚀 New feature (non-breaking change which adds functionality)
  • 🌎 Translation (addition or update a translation)
  • ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

Checklist

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have tested the change locally.
  • I followed the steps if I add a new language .

@briis
Copy link
Contributor

briis commented Dec 18, 2022

I would love this card. This is really missed in the different entity cards.

@piitaya
Copy link
Owner

piitaya commented Jan 10, 2023

What do you think about coloring the select too?
CleanShot 2023-01-10 at 10 25 11

Comment on lines 110 to 114
<div class="actions" ?rtl=${rtl}>
<mushroom-select-option-control
.hass=${this.hass}
.entity=${entity}
></mushroom-select-option-control>
Copy link
Owner

Choose a reason for hiding this comment

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

Suggested change
<div class="actions" ?rtl=${rtl}>
<mushroom-select-option-control
.hass=${this.hass}
.entity=${entity}
></mushroom-select-option-control>
<div class="actions" ?rtl=${rtl}>
<mushroom-select-option-control
.hass=${this.hass}
.entity=${entity}
></mushroom-select-option-control>
</div>

One missing div

@PaoloTK
Copy link
Contributor Author

PaoloTK commented Jan 10, 2023

What do you think about coloring the select too? CleanShot 2023-01-10 at 10 25 11

Cool idea, should it be a toggle or fixed?

@piitaya
Copy link
Owner

piitaya commented Jan 10, 2023

I have the code locally so I can push it. For lights and fan, the slider is colored with the icon color so we can have the same usage here.

@PaoloTK
Copy link
Contributor Author

PaoloTK commented Jan 10, 2023

I have the code locally so I can push it. For lights and fan, the slider is colored with the icon color so we can have the same usage here.

Sounds good to me!

@piitaya piitaya merged commit 3945c82 into piitaya:main Jan 10, 2023
@piitaya piitaya added the enhancement New feature or request label Jan 10, 2023
@PaoloTK PaoloTK deleted the select_card branch March 12, 2023 18:32
bryan-stewart pushed a commit to bryan-stewart/lovelace-mushroom that referenced this pull request Apr 5, 2023
* initial commit

* initial implementation

* fixed overflow bug

* bug fixes

* fixed current value not loading at first

* fix select overflowing to the right at some widths

* select height reduced to match other mushroom cards

* Update README.md

* Create select.md

* default entity in "add card" menu is now correct

* Added Select Card pictures

* fixed bug value not getting set for select entities

* Color select with icon color

Co-authored-by: PaoloTK <[email protected]>
Co-authored-by: Paul Bottein <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants