Order summary component with flexbox

Community feedback

• 

  Adriano•38,390

  @AdrianoEscarabotePosted about 2 years ago

  Hi lamy, how are you?

  I really liked the result of your project, but I have some tips that I think you will enjoy:

  • every Html document must contain the main tag, so we can identify the main content, to fix this, wrap all the content with the main tag. HTML5 landmark elements are used to improve navigation experience on your site for users of assistive technology.
  • to improve the design of the page, we can do this:

  body {
      background-size: 100% 50vmin;
  }
  

  The rest is great!

  I hope it helps... 👍

  1

  ---

  

  lamy•150

  @Lamy237Posted about 2 years ago

  Hi @AdrianoEscarabote!

  Thank you for your feedback. I added the main tag and updated the background-size and few other things. I actually didn't know about the unit vmin, so I learned something new today 😊.

  Thanks again... 🙏

  1