@mkboris
Posted
Hi jaysonalfie great job completing your first challenge, here are a few things to review
- All content should be wrapped within landmarks. Wrap a
main
tag around the .main- 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. - Font-size should be written in rem not px. This article explains it better Why font-size must NEVER be in pixels.
- Consider using a modern CSS reset at the start of the styles in every project. Like this one Modern CSS Reset.
Hope this helps
Marked as helpful
@jaysonalfie
Posted
@mkboris Thank you I will be sure to check on all that you have shared