@mkboris
Posted
Nice work Harvee-king, here are a few things to review
- All content should be wrapped within landmarks. Wrap a
main
tag around the .container and afooter
for the attribution. - 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. This will help reset a list of default browser styles.
- Font-size, media queries should be written in
rem
not px
Marked as helpful