Single price grid component using CSS Grid

Community feedback

• 

  Mojtaba Mosavi•3,760

  @MojtabaMosaviPosted almost 3 years ago

  1- A litle note about naming classes, it always a good practice to try to be descriptive and logical as possible which is not so easy all the time. In small project like this you can get aways with it but once you start working larger projects is should be something that you really put thought into.

      <section class="product__community></section>
      <section class="product__cta></section>
      <section class="product__features></section>
  
  

  Keep coding :=)

  Marked as helpful

  0

  ---

  

  Vencer Torres•250

  @vencertorresPosted almost 3 years ago

  @MojtabaMosavi Thank you for pointing that out. Naming classes is one of the things that I've been trying to improve. BEM is something I'm looking into and I'm reading a lot about it.

  0

  

  Mojtaba Mosavi•3,760

  @MojtabaMosaviPosted almost 3 years ago

  @vencertorres Your on right track.

  0
• 

  Aakash Verma•6,710

  @skyv26Posted almost 3 years ago

  Hi! Vencer, You made it really good and responsive too. Just add margin at the both side of layout horizontally (in your media)

  I checked design in mobile view so there is not enough space to separate layout from mobile viewport (I mean it look cool if have some space instead of sticks with mobile left and right handside scree)

  I hope you understand.

  Good Luck

  Marked as helpful

  0

  ---

  

  Vencer Torres•250

  @vencertorresPosted almost 3 years ago

  @skyv26 Thank you. I'll keep that in mind.

  0
• 

  AliAbuhumra•230

  @aliabuhumraPosted almost 3 years ago

  This card is perfect

  0