@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 besection
butmain
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 yourbody
in CSS.
Keep coding :-)
Marked as helpful
@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.