Four cards feature section using CSS Grid System

Solution retrospective
So glad to work on this third challenge. Any comment is highly welcomed from you guys. Thank you!
Please log in to post a comment
Log in with GitHubCommunity feedback
- @grace-snow
Hi,
This looks really nice, well done 👍
Just a few suggestions for you, but all pretty minor
- instead of section, I think you want to use 'header' on the top block
- be careful not to jump heading levels. Your h4s should be h3s
- personally, I think the icons on these cards are pretty meaningless, so could have
alt="" aria-hidden="true"
so assistive technology knows not to announce them (they will have just heard the heading anyway)
If you wanted to stretch yourself a bit more, I suggest you try this again using css grid, as that's what this challenge was really designed for ☺
Best of luck with your code learning
Join our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord