Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Social Proof Section

P

@myrojoylee

Desktop design screenshot for the Social proof section coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

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

Loai Esam 670

@LoaiEsam37

Posted

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 reason

Marked as helpful

0

Please log in to post a comment

Log in with GitHub
Discord logo

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