Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

order summary component, my first challenge

@CaioRoman

Desktop design screenshot for the Order summary component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Hello, this is my first challenge, and i'm learning css and html, so any constructive criticism is welcome, p.s. sorry for my english, i'm brazilian and i'm still learning english so i'm using google translator

Community feedback

Davide 1,725

@Da-vi-de

Posted

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

@CaioRoman

Posted

@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

Please log in to post a comment

Log in with GitHub
Discord logo

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord