Single Price Grid Component

Community feedback

• 

  T

  Matt Studdert•13,611

  @mattstuddertPosted over 4 years ago

  Nice work on this challenge, Matas! Here are a few pointers after looking at your code:

  • You've wrapped the "per month" text in a h2, whereas I'd say that it's not a heading. You could wrap that whole line in the paragraph tag and then use a span inside to separate the words to style them differently.
  • In the "Why Us" area you've used a paragraph with break tags. I'd describe this as a list of features/benefits, so a ul with li elements inside would make sense.
  • Another recommendation would be to through it one more time matching your solution up to the design to try and get your project as close to the design as possible. You can see from the design comparison that the colours and shadows are slightly off. Accuracy is a key part of being a front-end developer, so it's a skill that's well worth taking some time to build.

  I hope these tips help. Keep up the great work!

  0