Product-Cart design using flex

Community feedback

• 

  Andre Ferreira•450

  @Namonaki0Posted over 1 year ago

  Hi TANVEER, well done for completing the challenge.

  A few points I would like to mention:

  Instead of having a div with a class of main you could have the <main> tag and make that the main wrapper.

  You can center it in the middle of the page with the below:

  main {
    display: grid;
    height: 100vh;
    place-content: center;
  }
  

  I can see that you haven't included any media queries which would make your design responsive. I would use the mobile-first approach and then work my way up by using a media-query with a min-width for a chosen breakpoint, most likely 1440px as specified.

  Just a few points to consider. I hope this helps.

  Marked as helpful

  0