Mobile First Testimonial Grid Template Using HTML and CSS
Design comparison
Solution retrospective
Kindly leave a feedback. Do you think I aced this challenge?
Community feedback
- @Brivan-26Posted almost 3 years ago
You did a great job! I think it's better to make sure that the whole content doesn't overflow the100vh of the body because you have a little bit of scroll. I checked your solution code and I noticed that you didn't set <h1> element, where it says on my accessibility report that I need to have a level one heading "Page should contain a level-one heading". How could you fix this?
0@phaevourkhrisPosted almost 3 years ago@Brivan-26 Thanks for the feedback.
I dunno why you're getting a heading issue in your accessibility report, but I didn't get any. You could include a h1 then and see if it solves the issue.
0
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