@garcialexco
Posted
Hello, congrats on completing this solution!
While I was working on this project, the biggest challenge I had was making it responsive as it was my first responsive project as well. Seems 'em',, 'rem' percentages or using anything scalable for your sizes makes it more accessible, and you definitely got that down pat. Your media queries look excellent!
Grid is a lot of fun, you definitely got the hang of it! The only thing I can see is that you have a couple redundant uses of 'display: grid' on lines 35 and 58 of your CSS. Since you already set it in your body, the rest of the elements should inherit it.
Great job! Here, take this interesting post on Reddit regarding CSS flex for your future coding adventures! https://www.reddit.com/r/webdev/comments/10ffo7a/css_flex_for_speed_learners/
Happy coding and great job on your solution!
Marked as helpful
@JackoWill
Posted
@garcialexco Hey, thank you!
Awesome thanks for the tips I will fix that up right away! I will give that post a read also.
Kind regards, Jacko