Selina Lavery

Took a while but i think i am finally ready to stop tinkering with this. Building this app gave me so much practice in using API's, Postman, useEffect and useLocalStorage. Feedback welcome, thanks. You will need to allow location sharing for it to work: https://codesandbox.io/p/sandbox/weather-app-ff9mnn

New comment Mar 3

Selina Lavery

1 like • Mar 3

@Martynas Astravas Good catch, I should get rid of the <a> tag there. thanks!

LaurieSue Peterson

Feb 26 •

Code Review

Findtrend Desktop Corrections

Hi, hope everyone is doing great. If y'all have the time, I would greatly appreciate any feedback you have to offer on this project. I made some improvements (I hope). Just wanting to make sure everything is all good before I start the media queries portion of the project. Thank you so much🙂. @Cristian Florea https://codepen.io/PinkBeeMe/pen/NPRPNyq

New comment Mar 2

Selina Lavery

2 likes • Feb 26

This looks really good. Just a couple of pointers - when creating dummy links in the nav at the top, you can put a # in the href="#", just so it is not empty and means it wont throw an error if clicked. One more think I noticed is the monthly yearly toggle, instead of using an image, try to create the toggle using divs, one with a white BG and the other green circle sitting correctly on top. Good luck on the next media queries section 😎

Selina Lavery

0 likes • Feb 27

@Martynas Astravas everyday is a learning day 😁