@mkboris
Posted
Nice work cacesasa, here are some suggestions for improvement
- All content should be wrapped within landmarks. Wrap a
main
tag around the .social_wrapper and afooter
for the attribution. - Consider using a modern CSS reset at the start of the styles in every project. Like this one Modern CSS Reset.
- Font-size should be written in rem not px. This article explains it better Why font-size must NEVER be in pixels.
- Media queries should be defined in
rem
not px
Marked as helpful
@cacesasa
Posted
@mkboris Thank you for the comments. I implemented the changes that you recommended and created a boiler-plate for my CSS reset.