tobyweston / temperature-machine

Data logger for multiple DS18B20 temperature sensors on one or more machines
Apache License 2.0
67 stars 22 forks source link

Long term graphs title #40

Closed Quaxo76 closed 6 years ago

Quaxo76 commented 6 years ago

I just updated to the latest available version. I noticed that when I click on one of the long range graphs, and it enlarges, at the top left of the screen I get the "title" of that graph (i.e. 24-hour or 1 week). If then I change to another graph using the left/right arrows on screen, the graph does change, but the title does not. It should change to constantly reflect what graph is being shown...

Cristian

tobyweston commented 6 years ago

Good spot. Will investigate

tobyweston commented 6 years ago

Fixed by https://github.com/tobyweston/temperature-machine-ui/commit/7ea30cfc4bf1db2ed18edd616b87d3b58b6a19e2

Quaxo76 commented 6 years ago

Thank you, it works now. Could I also suggest to also add the descriptor to the actual graph? Where it says "temperature" on the top center of the graph, it could say "temperature 24 hours", "temperature 7 days" etc. This would be good especially since I'm storing the past graphs on the sd...

tobyweston commented 6 years ago

Fixed the second part and labeled the charts better.