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

Single price grid component

Asma 80

@asmablad

Desktop design screenshot for the Single price grid component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

Ahmed El_Bald 1,020

@Ahmed-Elbald

Posted

Hi Asma,

Again, I do believe you are really talented when it comes to applying the UI. I like how it's responsive on all screen sizes. I just have some notes here that you might find useful:

  • Text like '30-day, hassle-free money back guarantee' should not be wrapped in an h2 element as it's not actually a page heading. Use a regular p element instead as headings imply the beginning of a new section.
  • Try to make your classes names more descriptive so that other developers can test your code more easily. For instance, you can replace your one, two and three classes with intro, cta (call to action) and outro or anything that is meaningful.
  • Again, you should've used a description list dl for the pricing.

Marked as helpful

0

Asma 80

@asmablad

Posted

Thank you so much for your response, i have now fixed the issue.

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