Single Price Grid Component HTML CSS SASS

Community feedback

• 

  P

  Fluffy Kas•7,715

  @FluffyKasPosted over 2 years ago

  Hey there,

  The colours seem fine to me ^^ Overall your component looks really good, responsiveness seems fine as well. I'm only here to point out some smaller things:

  1. Look up what br element does. You shouldn't really use it for spacing things out. For this, you could use some padding or put a flexbox on things (like the list in the why us section).

  2. In the Why Us section, it would be better to swap your "text-light" div to a ul, and then use an li for each item.

  3. You could add cursor: pointer and perhaps a nice little hover on your button.

  The rest seems great, well done!

  Marked as helpful

  1

  ---

  

  TomekSwitecki•160

  @TomekSwiteckiPosted over 2 years ago

  @FluffyKas Thanks for great feedback! It's a great idea to use list. I have never thought about using it this way. Brilliant idea! Thank you so much!

  0
• 

  Hexer•3,660

  @EmmanuelHexerPosted over 2 years ago

  Great job overall man.

  1
• 

  Naveen Gumaste•10,480

  @NaveenGumastePosted over 2 years ago

  Hay ! TomekSwitecki Good Job on challenge

  1