Responsive Landing page using CSS grid.

Community feedback

• 

  S MD suleman•3,530

  @sulemaan7070Posted almost 2 years ago

  hey @richard9809😄, good job on completing the challenge💯... here are a few tips and best practices to make your site better!!

  1.I see there is no color change on many elements you can probably go to the design-folder and find the active state image. you can add that styles on hover.

  2.Add cursor:pointer on both hamburger menu and the close-icon.

  3.use picture element as a best practice for responsively displaying the images..

  <picture>
    <source media="(min-width: 650px)" srcset="img_food.jpg">
    <source media="(min-width: 465px)" srcset="img_car.jpg">
    <img src="img_girl.jpg">
  </picture>
  

  more about picture element here happy coding💯💯

  0