single-price-grid-component

Community feedback

• 

  P

  tediko•6,680

  @tedikoPosted almost 4 years ago

  Hello, M. Neehanth Reddy! 👋

  Good job, your solution looks good and responds well! I would suggest:

  • Instead of using grid just on .subscription-box i would consider set your .price-container to be your grid container. This way if you ever want to change the arrangments of layout it would be simple and easy. Nevertheless, your solution is also good.
  • Read about semantic. Semantic elements lead to more consistent code, they are easier to read and improve accessibility.
  • Try to stick more with design. I mean font-sizes in this case.

  Good luck with that, have fun coding! 💪

  1

  ---

  

  M. Neehanth Reddy•185

  @neehanthreddy01Posted almost 4 years ago

  Yeah sure, thank you very much @tediko

  0
• 

  T

  ApplePieGiraffe•30,545

  @ApplePieGiraffePosted almost 4 years ago

  Hey, M. Neehanth Reddy! 👋

  Nice job on this challenge! I think your solution looks good and scales well! 👍

  Just one small suggestion—I think adding a hover state to the button would be a nice touch! 😉

  Keep coding (and happy coding, too)! 😁

  0

  ---

  

  M. Neehanth Reddy•185

  @neehanthreddy01Posted almost 4 years ago

  @ApplePieGiraffe Yeah sure, thank you 😊

  0