MrJay's Solution to Single price grid component

Community feedback

• 

  Melvin Aguilar 🧑🏻‍💻•61,140

  @MelvinAguilarPosted about 1 year ago

  Hello there 👋. Good job on completing the challenge !

  Here are some suggestions to further improve your solution:

  • Use <ul> and <li> for the content under "Why Us": Since the content is a list of items, it's more semantically correct to use an unordered list (<ul>) and list items (<li>) instead of just using paragraph (<p>).

  • Consider using min-h-screen (which translates to min-height: 100vh) rather than combining breakpoints with heights like lg:h-[100vh] h-auto. This approach is more straightforward and ensures that there are no unexpected issues on smaller screens. It simplifies your code while achieving the same result.

  I hope you find it useful! 😄

  Happy coding!

  Marked as helpful

  1

  ---

  

  Jere•150

  @chungvuiloongPosted about 1 year ago

  Hi @MelvinAguilar,

  Thank you for your feedback and your support 😊.

  Amazing feedback!

  I tried both solutions and it worked great!

  Hope to see more of your feedback as it is very helpful.

  Have you considered being a mentor?

  Happy Hacking!

  0