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

html,css, bootstrap

@NehalGNafady

Desktop design screenshot for the Single price grid component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


thank's for your comments

Community feedback

P
Matt Studdert 13,611

@mattstuddert

Posted

Nice work, Nehal! Here are a few pointers after taking a look at your code:

  • You've got h2 and h4 elements, but no h1 or h3. I'd always recommend starting with a h1 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 a ul with li elements for each item.

I hope that helps. Keep up the great work!

1

@NehalGNafady

Posted

@mattstuddert You are right, thanks for your comment. i'll put in my head.

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