search

fufar / simple-clock-card

Simple clock card for Home assistant lovelace
49 stars 23 forks source link

Why 250 and not 1000? #1

Closed idoaflalo closed 3 years ago

idoaflalo commented 3 years ago

https://github.com/fufar/simple-clock-card/blob/ab8f49ad5075de337175b3bb6d1d9587002e5923/simple-clock-card.js#L53

fufar commented 3 years ago

why not, it cause problem?

idoaflalo commented 3 years ago

why not, it cause problem?

It's better to update the DOM as little as possible. If it was the only process on the page I would say it does not affect, but because lovelace can be heavy I think it is better to update every second (Which is the smallest unit the clock displays).

fufar commented 3 years ago

changed to 1000ms