order summary component, my first challenge

Community feedback

• 

  Davide•1,725

  @Da-vi-dePosted about 3 years ago

  Hi Caio, it's a nice result as first challenge, well done!

  • Good use of semantic HTML except at the beginning after body tag, the container shouldn't be section but main instead. section is not semantic and it's not the right region landmark for containing big code blocks.

  • On mobile there's a horizontal scrollbar, you can get rid of it by adding overflow: hidden to your body in CSS.

  Keep coding :-)

  Marked as helpful

  1

  ---

  

  Caio Roman•20

  @CaioRomanPosted about 3 years ago

  @Da-vi-de hi Davide, thanks for the help, i didn't know the main tag, so it helped a lot, and i don't know css responsive so i didn't do anything for mobile.

  0