Order Summary with React

Community feedback

• 

  P

  Ken•4,915

  @kens-visualsPosted about 3 years ago

  Hey @marcoariasm 👋🏻

  I have some quick tips to help you fix the accessibility and HTML issues.

  • In your markup, <div class="card">...</div> should be <main class="card">...</main> and .
  • Next, font-weight: 500px is not a proper font-weight value, so all you need to do is just remove px, like so, ``font-weight: 500`. These will fix the accessibility issues. Don't forget to generate a new repot once you fix the issues.
  • For the music icon, add aria-hidden="true”, because it's for decoration. You can read more about aria-hidden here.
  • Also, I suggest adding transition: all 0.2s; to the button and the links, this will make :hover smoother.

  I hope this was helpful 👨🏻‍💻 other than that, you did a great job for your second project, well done. Cheers 👾

  0