Axis title does not show preferred unit of measure

mstenta / atmospi

Atmospheric monitoring app for logging and graphing temperatures and humidities over time using a Raspberry Pi and DS18B20, DHT11, DHT22, and AM2302 sensor(s).

31 stars 9 forks source link

Axis title does not show preferred unit of measure #26

Closed ultima-originem closed 9 years ago

ultima-originem commented 9 years ago

In v0.2.3 the axis title always shows degrees Fahrenheit and does not get the preferred UoM from default_settings.py

mstenta commented 9 years ago

Do you mean in v0.2.3? Because that's the version that added temperature units to default_settings.py. Before that it was always Fahrenheit.

ultima-originem commented 9 years ago

Corrected; my bad :-)

The problem was present is the version downloaded today.

mstenta commented 9 years ago

Oh ok hmm... maybe I just missed one of the replacements? I'll take a quick look...

mstenta commented 9 years ago

Ah ha! Found it... line 76 of Atmospi/static/atmospi.js should be:

text: 'Temperature (°' + settings['t_unit'] + ')'

I will push a fix...

mstenta commented 9 years ago

Fixed in the master branch!

ultima-originem commented 9 years ago

Works! Thanks!