@mkboris
Posted
Nice work @Divinitry, here are a few things to review
- All content should be wrapped within landmarks. Wrap a
main
tag around card. and afooter
for the attribution - Using an unordered list
ul
to group the social media links is a better approach for both semantics and accessibility. - It's best practice linking Google fonts directly in the HTML
head
section rather than directly in your CSS file as it enables asynchronous downloading, improving page load times. - Consider using a modern CSS reset at the start of the styles in every project. Like this one Modern CSS Reset.
Marked as helpful
@Divinitry
Posted
@mkboris
oh wow, thank you very much for the feedback, will put these to good use!