Boris• 2,870
@mkboris
Posted
Nice work adrianalbino, here are a few things to review
- All content should be wrapped within landmarks. Wrap a
main
tag around the .container - Every page should have at least one heading typically an
h1
element to provide a clear structure. - To improve the semantic meaning of these social media links, you should use the
a
(anchor) tag instead ofp
. It's not good practice to wrap every element indiv
because divs aren't semantic, using an unordered listul
to group the social media links is a better approach for both semantics and accessibility. - Use a separate file for your css
Marked as helpful
0