Local Weather App : Feedback

Hello there!

I hope you all are doing well! I have just completed my local weather app and i would love to hear from you.
Any kind of feedback is welcomed and appreciated.

Here is my project link.

Works as advertised.

Maybe add a limit to the number size? I went to the website and I got this 64.02199999999999 in degrees.

Tried to fix it myself but your variable names are not very readable.
In your next project try to use full names when doing variables instead of just a few characters.
The code below insures that at least for fahrenheit there’s only 2 numbers behind decimal.

fahrenheit = (9*tmp)/5 + 32;
$('.tmp').text(Math.round(fahrenheit * 100) / 100).toFixed(2);

Otherwise. Keep it up! :+1:

1 Like

Thank for your precious feedback @ItsRoyal. I have modified my Celsius to Fahrenheit conversion to limit the decimal count to 2 as you advised.
Also, from next time i will use more readable variables names as you advised. :relaxed::relaxed:

1 Like