Local Weather App - Feedback Requested :)

Code Feedback
1

I used fcc weather api, bootstrap, jquery and coded in es6 javascript.

Any advice and suggestions please? :slight_smile:

EDIT: I’ve removed the ‘retard unit’ reference to Fehrenheit unit which wasn’t supposed to offend anyone and was there just as a try to generate some humour. ‘retard unit’ is just a humorous reference to metric unit system used on humor sites like 9gag.com.

2

Heya! I’m no stranger to a bit of humor injected into these projects. That being said, I think you should modify your ‘Change to Retarded’ button, especially if you at some point want to use this on a resume or portfolio or something. Maybe just pick a word that conveys the joke that other people won’t find offensive?

Also, if the window is small enough to see a vertical scroll bar, your background will shift down upon scrolling. Maybe it’s related to the ‘background-attachment’ property?

2 Likes
3

I totally understand, Okay I’ll change it right away. Thanks! :slight_smile:

4

I fixed everything as you suggested. :slight_smile:

5

I’ve added an explanation in the code that it is just a humorous online reference and is not supposed to offend anyone. :slight_smile:

6

hm it looks good but i have the same feedback about the “switch to retarded”.
I love humor but ok, it doesn’t feel like it. It’s a bit out of place i think.

Nice looking weather app!

7

Changed that to “switch to fehrenheit”. :slight_smile:

8

You are spelling Fahrenheit wrong.

9