Responsive Order Summary page

Community feedback

• 

  Daniel 🛸•44,310

  @danielmrz-devPosted 11 months ago

  Hello @CaseyLawver!

  You did a very good job there!

  I have just one very simple suggestion for improvement:

  • Since the button is a clickable element, it's nice to add cursor: pointer to it. It gives the user one more visual indication that the element is clickable.

  I hope it helps!

  Other than that, you did a great job!

  0
• 

  Andrés Gutiérrez Ramírez•2,470

  @AGutierrezRPosted 11 months ago

  Hello there @CaseyLawver 👋. Good job on completing the challenge!

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

  CSS and Styling:

  • Implement CSS custom properties to define and utilize project colors more easily.
  • The body should not have its height limited. Instead of using height: 100vh, use min-height: 100vh.

  Accessibility and Semantic HTML:

  • The icons/illustration images are decorative, so their alt text must be empty: alt="".

  I hope you find this helpful 😁. Most importantly, your submitted solution is fantastic!

  Happy coding!

  0