Aman Singh Bhogal• 1,010
@asbhogal
Posted
Hi Emanuel,
Great work! The design matches the mockups well, your logic is clean and it's great to see grid
used for this. Just a few pointers I wanted to mention:
- Always have your
font-size
inrem
for accessibility purposes. This article by freeCodeCamp explains it in detail Link - Locally host your Google Fonts for privacy and performance reasons. Here's a good practical video showing how to do this Link
- At viewport widths <=
330px
, the parent container is touching the browser boundary. You could add some padding or margin just to give it some white/breathing space
Hope this helps!
1