Responsive Order Summary Component Using Flexbox

Community feedback

• 

  Boots 😺•1,590

  @adityaphasuPosted over 1 year ago

  Hi!

  Nice solution!

  Some points to improve your code and design:

  • Try removing background-size: cover; from the body so that the background image doesn't cover the whole page.
  • when setting font size try using relative units like rem instead of using absolute units like px as they are better for scalable layouts. You can read more about it here!
  • Instead of using a div as a container you should use the <main> tag which is generally good for accessibility purposes, it provides a clear and meaningful structure to your HTML and it helps search engines understand the purpose and the main content of the page.You can do it like this:

  <main class="container">
    //rest of the code
  </main>
  

  Overall good work!

  Keep up the hard work and good luck!😊

  Marked as helpful

  1