FCC WeatherApp Review

FCC WeatherApp Review
0

#1

hey guys,

so my FCC Weather App is now ready for review! Would really appreciate feedback regarding responsive-design, and just making sure all the functionality is in-tact:

Thanks in advance!


#2

Looks really good. I didn’t really look too much at the code.

  1. I would change the button hover to use transform: scale()
.changeLoc:hover {
/*     padding: 1.2rem; */
  transform: scale(1.1);
}
  1. I would give .container a max-width of say 1200px

  2. On small screen sizes, you might want to lower the font size a bit more and make the button smaller.

  3. I would refactor the EventListener callbacks into stand-alone functions.

Good job.


#3

appreciate the feedback! thank you!