Design comparison
Solution retrospective
Here is my effort with the CSS grid knowledge that I have! Thank you to all who take the time to give me feedback! I do read everything, I just do not get to reply to everyone. But I do read it!
I started this project a while back and did not understand enough grid concepts to finish it. I came back to it today and applied what I've learned so far. I have not applied aria labels which is the comment I always get on these. I just keep forgetting because I am so focused on every other thing, lol.
Community feedback
- @LoaiEsam37Posted over 1 year ago
add role attr main to
<div class="social-proof-card">
like this<div class="social-proof-card" role="main">
or you can just do it like this<main class="social-proof-card">
so that it becomes easier for screen readers to know where is the main content and you can also use something like <nav> , <footer> and <sidebar> tags for the same reasonMarked as helpful0
Please log in to post a comment
Log in with GitHubJoin 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