Responsive Price Grid Component using CSS Grid

Community feedback

• 

  Bryan Li•3,530

  @Zy8712Posted about 1 year ago

  Your site looks pretty good. Code is pretty organized and readable. Good use of proper naming conventions for your classes. One thing I'd slightly modify is the breaking point of your box for the switching between mobile and desktop format. Another thing would be the spacing and box size for the mobile layout. When the screen is a certain size the box looks sorta awkward as theres a bunch of empty space on the right side of the boxes. Looking at the original design for the mobile format of the site you can see that the three boxes are supposed to be around the same size.

  Aside from that your code is very well organized and readable. Nice work 👍

  1

  ---

  

  Abdulhamid Adeniji•160

  @Alpha437Posted about 1 year ago

  @Zy8712 Thanks for this insightful feedback. I will work on that

  0

  

  Abdulhamid Adeniji•160

  @Alpha437Posted about 1 year ago

  @Zy8712 I just made some adjustments... Do you mind checking it out?

  0

  

  Bryan Li•3,530

  @Zy8712Posted about 1 year ago

  @Alpha437 Looks great! I see less shifting on the container and better spacing on the mobile version. Good job!!!

  1

  

  Abdulhamid Adeniji•160

  @Alpha437Posted about 1 year ago

  @Zy8712 Thanks for the feedback, I really appreciate it.

  0