Design comparison
SolutionDesign
Solution retrospective
thank's for your comments
Community feedback
- @mattstuddertPosted almost 5 years ago
Nice work, Nehal! Here are a few pointers after taking a look at your code:
- You've got
h2
andh4
elements, but noh1
orh3
. I'd always recommend starting with ah1
as your main heading. Then I would use the other levels to create the rest of the content hierarchy. But be careful not to unnecessarily skip heading levels, as this can cause accessibility issues. - You're using a paragraph with
br
tags in the Why Us area. This is a list of features/benefits, so I'd recommend using aul
withli
elements for each item.
I hope that helps. Keep up the great work!
1@NehalGNafadyPosted almost 5 years ago@mattstuddert You are right, thanks for your comment. i'll put in my head.
0 - You've got
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