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

@Igwe0001

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


Hi everyone i just finished this challenge, i would really love to hear your thoughts on the challange, thank you.

Community feedback

P
Grace 27,650

@grace-snow

Posted

Hi, This is nicely responsive and looks pretty close to the design. I think the headings are slightly too small and thin viewing on mobile but that's all.

There are some minor improvements I'd make to the html:

  • the lower headings are all subheadings under 'join our community' so if the parent heading is a h3, they need to be h4s.
  • the price shouldn't be a heading, but again if it is, it has to be another level down, belonging to pricing. In this case that would be a h5.
  • in why us, if you find yourself putting lots of line breaks in a paragraph, and it doesn't read like a paragraph, it should be a list. (So swap for one ul and lis for each item)

I hope that's helpful, upvote if it is ☺

2

@Igwe0001

Posted

@grace-snow Hi thanks for your input, made some changes to it, thanks for helping again.

0

@Lazizbek1997

Posted

Good job man👍

0

@Igwe0001

Posted

@Lazizbek1997 thank you

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