Insure Landing Page

Community feedback

• 

  Sahand Masoleh•1,210

  @sahand-masolehPosted over 2 years ago

  Hello there! great job! I just noticed a few things:

  • I can't open the nav menu mobile view, nothing happens when I click the button.
  • Set a height for your hero image. Right now it can go from very tiny to bigger than the view port at 700px width. Set a height with the object-fit: cover property so that it wouldn't have unpredictable behavior.
  • Right now you're loading all images and hiding/unhiding them with css based on view size, this negates the point of having multiple resolutions of one image. Use the picture tag or the img tag with srcset to only load the image that is needed based on a media query.

  0