Single price grid component using Flexbox

Community feedback

• 

  T

  ApplePieGiraffe•30,545

  @ApplePieGiraffePosted about 4 years ago

  Hey, Ananda Zulvansyah Putra! 👋

  Nice job on this challenge! The desktop layout of your solution looks pretty good! 👏

  I suggest,

  • Making sure the design doesn't revert to the desktop layout when the screen width decreases past 375px. Your solution should look good on all screens (even super tiny ones).
  • Adding a hover state to the button of the card component would be a nice touch, I think.

  Keep coding (and happy coding, too)! 😁

  1

  ---

  

  Ananda Zulvansyah Putra•160

  @azurasanPosted about 4 years ago

  @ApplePieGiraffe thank you, i'll fix it

  0
• 

  T

  Grace•31,270

  @grace-snowPosted about 4 years ago

  Ps from a quick glance at the html, I think you'll want to look into semantics a bit more. Heading levels should go in order. Also try the list elements for the why us section. And try to use classes for all styling. Good luck!

  1

  ---

  

  Ananda Zulvansyah Putra•160

  @azurasanPosted about 4 years ago

  @grace-snow thank you sir :), i'll try it later

  0
• 

  T

  Grace•31,270

  @grace-snowPosted about 4 years ago

  Hey, I won't give feedback on this yet until you've added mobile styles, as I'm viewing on my phone. Keep going with it if you can, nearly there 😉

  1

  ---

  

  Ananda Zulvansyah Putra•160

  @azurasanPosted about 4 years ago

  @grace-snow i already added mobile styles for (375px until 600px), i'll adding some styles for <=375px.thank you for your constructive advice .sorry my english is terrible

  0

  

  T

  Grace•31,270

  @grace-snowPosted about 4 years ago

  @azurasan you may as well just remove 375px as your minimum and not have one. All mobile styles can be the same then up until 600px when desktop layout kicks in. Good luck

  1

  

  Ananda Zulvansyah Putra•160

  @azurasanPosted about 4 years ago

  @grace-snow i'll do it. Thank you for your advice

  0