single-price-grid component-master

Community feedback

• 

  Daniel 🛸•44,310

  @danielmrz-devPosted 12 months ago

  Hello @yassawambessaw24!

  Your project looks great! Beautiful and responsive!

  I just have one suggestion:

  • Change the color of that 3rd section to #49BEBD and your project will look even closer to the original design.

  I hope it helps!

  Other than that, you did a great job!

  Marked as helpful

  0
• 

  Melvin Aguilar 🧑🏻‍💻•61,140

  @MelvinAguilarPosted 12 months ago

  Hello there 👋. Good job on completing the challenge !

  I have other suggestions about your code that might interest you.

  • 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>).

  • You should use the cursor: pointer property to indicate that the element like a button or a link is clickable.

  • Wrap the page's whole main content in the <main> tag.

  I hope you find it useful! 😄 Above all, the solution you submitted is great!

  Happy coding!

  0