danimart1991 / pvpc-hourly-pricing-card

Home Assistant Lovelace custom card to use with Spain electricity hourly pricing (PVPC) integration
Apache License 2.0
89 stars 21 forks source link

Error color icono tararjeta #19

Closed clavi69 closed 4 years ago

clavi69 commented 4 years ago

Con la ultima actualización de home assistant a la 0.112.0 el icono de la tarjeta que cambia de verde a rojo para indicar si se esta en periodo barato o caro de la luz ha dejando de funcionar y ahora esta siempre en negro Captura de pantalla 2020-07-02 a las 23 05 40

McGiverGim commented 4 years ago

Iba a comentar lo mismo... Screenshot_20200702-230745

McGiverGim commented 4 years ago

Si ayuda, examinando el código, parece que no se está rellenando el "fill" del svg que dibuja el círculo:

image

Según el código entiendo que debe rellenarse según la tarifa: https://github.com/danimart1991/pvpc-hourly-pricing-card/blob/191e5b8b1e993cc429cc2b5bd27a9d096a9faef9/dist/pvpc-hourly-pricing-card.js#L213-L218

No sé que habrá cambiado en esta versión para que esto deje de funcionar.

McGiverGim commented 4 years ago

Depurando un poco, el style, que contiene el getComputedStyle(document.body);, no contiene los estilos definidos en la tabla tariffPeriodIconColors. He visto que son colores típicos de Material Design. ¿Tal vez hayan cambiado por parte de Google o por parte de Home Assistant?

McGiverGim commented 4 years ago

Creo que esto está relacionado: https://github.com/home-assistant/frontend/issues/6254

McGiverGim commented 4 years ago

Según ese link, hay que cambiar en:

const tariffPeriodIconColors = {
  error: '--error-color',
  normal: '--google-yellow-500',
  peak: '--google-red-500',
  valley: '--google-green-500',
  'super-valley': '--google-blue-500'
};

Lo siguiente:

Por lo tanto lo mismo te toca buscar otros colores en general más adecuados para todo.

danimart1991 commented 4 years ago

Muchas gracias por avisar del error y por la investigación, me ha facilitado mucho arreglarlo.

McGiverGim commented 4 years ago

No sé si estará relacionado, pero los textos de los switch en el editor tampoco se ven, ni con la versión que ya tiene este fix: Screenshot_20200704-000009

danimart1991 commented 4 years ago

He creado una nueva Issue para tenerlo en cuenta. Muchas gracias.