Hello @davidsonaguiar,
Congrats on finishing another project! There is a few thing that you can change:
- Your image isn't decorative. Provide
alt
attribute with description for the non-sighted users. - Why do you select both
body
andhtml
elements in your CSS? There is no need to apply style rules to both of them, so stick only withbody {}
selector. - Block elements (like html , body , div , p ) have width 100% automatically so
width
is not required forbody
element, remove that property from your css rules. - Since you used
justify-content
andalign-items
properties in yourbody
to align your flex items both horizontally and vertically there is no need to addmargin: auto
to yourmain
element which is now your flex item. Don't Repeat Yourself. - Headings are signposts that guide readers through component. Therefore, they should indicate what a section or a paragraph is about. Keep your
<h1>
heading which is title of your component but change<h2>
to<p>
element. - You forgot to wrap your list items content within anchor tags
<a>
so they can point to external website.
Good luck!
Marked as helpful
0
Davidson Aguiar• 320
@davidsonaguiar
Posted
@tediko Dude, thank you very much for the tips, I'll make the changes
0