artem-sedykh / mini-climate-card

Minimalistic climate card for Home Assistant Lovelace UI
MIT License
281 stars 19 forks source link

Hide secondary_info option and align CSS #95

Closed ThomDietrich closed 2 years ago

ThomDietrich commented 2 years ago

Hello,

Contrary to other elements (like toggle or hvac_mode) we can't hide the secondary info. Could you add that feature? If it's already possible, the README does not clearly show that.

secondary_info:
      hide: true

Thanks and cheers!

regevbr commented 2 years ago

released in v2.3.0

ThomDietrich commented 2 years ago

You my friend are amazing! 😃

ThomDietrich commented 2 years ago

One issue remains :( The first line text isn't centered correctly, neither is the left icon looking at it.

image

regevbr commented 2 years ago

CSS sadly is not in my wheelhouse :-( A PR will be more than welcome...

ThomDietrich commented 2 years ago

Mine neither. I won't be able to help here much.

This change seems to fix the icon misalignment:

.ctl-wrap {
  margin-top: auto;
}

Please try and merge if it also works for you.


This change seems to fix the text misalignment:

.entity__info__name_wrap {
  display: flex;
  justify-content: center;
}
.entity__info__name {
  margin-bottom: auto;
  margin-top: auto;
}

But only for me. As soon as I try to add the secondary_info back it's a mess.

regevbr commented 2 years ago

We will have to wait for the owner @artem-sedykh to have a look at it

ThomDietrich commented 2 years ago

Okay let's see. You wanna reopen the issue?

regevbr commented 2 years ago

you should open a new one please