Single price grid component using CSS flexbox & grid

Community feedback

• 

  P

  Fluffy Kas•7,715

  @FluffyKasPosted over 2 years ago

  Heyo,

  I'm not working as a professional yet but I'm actively looking for jobs and sass/scss is something I see very frequently in job requirements. It doesn't take long to learn and you just use as many of its features as you wish.

  Your solution looks great btw, I only have some small suggestions really:

  1. Don't forget to set the font-family of your button. If you don't set it or don't reset it somehow, it's just going to use the browser's default font.

  2. Careful with your heading levels. It's better to have only one <h1> heading, and work your way down from there to create a clear document outline (much like with a book that has a main title and chapter titles, etc).

  3. In the why us section you actually want to use a <ul> witl an <li> for each item and then style it with css. Don't use <br> for this, that has a very specific purpose.

  Marked as helpful

  2

  ---

  

  Alexis Lienard•150

  @Al-lienPosted over 2 years ago

  Hello @FluffyKas ! Thanks a lot for giving me your feedback, i really appreciate it.

  How do you suggest I prioritize my code ? Do you think I should set "Join our community" as <h1> and "Why us" as <h3> ?

  I'm going to learn sass then and I will soon update my solution with all your advice.

  Have a great day !

  0