Local Weather App <good|could be better| terrible>?

Code Feedback
1

Here’s a link to my just completed Local Weather App:


Your feedback is welcome
.Thanks!

1 Like
2

Looks nice, but the date is way off…

1 Like
3

Date seems ok to me.

Looks like someone didn’t account for some countries displaying month first in shorthand data format :wink:

1 Like
4

Thanks,I’ll see to it now.

5

how do you think I should handle or detect date format in other countries

6

I generally agree that you shouldn’t parse the date and time yourself (relevant video: https://www.youtube.com/watch?v=-5wpm-gesOY).

I would use external library like Moment JS:

You should add https://cdnjs.cloudflare.com/ajax/libs/moment.js/2.15.1/moment.min.js to your external JS libraries (same place where you added jQuery library), and replace
day = processDate(day);
with
day = moment().format("dddd, MMMM Do YYYY, h:mm:ss a");
(you can see how to format date/time here).

All date/time parsing code will be replaced with one line :wink:

1 Like
7

Thanks a lot, jenovs. That was really helpful.