bookmark-landing-page-master

Hey Kelvin, looks good, seems to be ok with responsiveness and email validates as well. A few things I noted here are the navbar and the hover on the buttons.

• The nav only has 5 items I think it is simple enough to accomplish in one set, rather than creating a separate ul for mobile and desktop. The links on both versions are also missing their anchor tags.

• Hovering on the buttons causes shifting of other elements because it is adding the border. I'd put the border as transparent beforehand, and just change the color for hover. This way, the border is already occupying space initially, and would not cause a jump when a user hovers.

Hope this helps :-)

Good job Kelvin, I love the responsiveness 💯

• add some padding to header , footer
• make those cards in download extension section have some hover state

Other than that the solution seems fine to me 👌