search

ThingPulse / esp8266-weather-station-color

ESP8266 Weather Station in Color using ILI9341 TFT 240x320 display
https://thingpulse.com/product/esp8266-wifi-color-display-kit-2-4/
MIT License
569 stars 222 forks source link

Temperature not displayed in "Current Conditions" page #139

Closed JMe4801 closed 3 years ago

JMe4801 commented 3 years ago

8<------------------------ BUG REPORT ----------------------------------------- Latest version of esp8266-weather-station-color.ino available

in line 503 you have : 

drawLabelValue(0, "Temperature:", currentWeather.temp + degreeSign);

i suggest you replace with : 

drawLabelValue(0, "Temperature:", String(currentWeather.temp,1) + (degreeSign));

Apologies if i do not follow all the rules of githup, first time for me i use it ;-).

Regards.

Jacques

8<------------------------ END BUG REPORT -------------------------------------

marcelstoer commented 3 years ago

Thanks, the original code indeed does look oddly wrong. I'll need to test this myself first, though.