Submitted over 2 years ago
Social proof section using css grid and flex-box.
@Cachilox
Design comparison
SolutionDesign
Solution retrospective
My first challenge in frontend mentor, I wanted to know if I meet the requirements and to know more about the best practices.
Community feedback
- @tesla-ambassadorPosted over 2 years ago
Hey Mariano! Congratulations on completing your first challenge on frontend mentor. It's a really great milestone, kudos! Your solution is really great and I love how responsive it is, what you did with the ratings as the screen width reduces was really great.
Here's a few pointers to help you resolve some of your accessibility issues:
- In order to resolve some of your accessibility issues, you might wanna use landmarks in your html code, these help browsers easily navigate your code. So you might consider wrapping your divs in landmarks like
<main>
or<header>
or<footer>
you need to do this according to how your page is structured. Incase you want to know more about landmarks, follow this link. - Your heading tags should only increase by one ie when you have just used a
<h1>
tag, your next heading tag should be<h2>
and after that<h3>
and it continues like that all the way to<h6>
. So they have to be in a sequential order. These will help resolve most of your accessbility issues
Happy coding and keep up the good work!π
Marked as helpful2 - In order to resolve some of your accessibility issues, you might wanna use landmarks in your html code, these help browsers easily navigate your code. So you might consider wrapping your divs in landmarks like
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