order-summary-component

Community feedback

• 

  バレンタイン 😈•64,190

  @VCaramesPosted almost 2 years ago

  Hey there! 👋 Here are some suggestions to help improve your code:

  • Do not forget ⚠️ to check your FEM report, to see what is incorrect and update your code with it. This should be the first thing that should be done right after submitting the challenge.

  • Along with the blank alt tag, you also want to include the aria-hidden=“true” ⚠️ to the “illustration” and “music icon” to fully remove them from assistive technology.

  More Info:📚

  MDN Aria-Hidden

  • The “Cancel Order” was created with the incorrect element ❌. When the user clicks on the button they should directed to a different part of you site. The anchor tag will achieve this.

  More Info:📚

  MDN The Anchor element

  • Implement a "Mobile First" approach 📱 > 🖥

  Mobile devices are now the dominant 👑 way in which people browse the web, it is critical that your website/content looks perfect on all mobile devices.

  More Info: 📚

  Mobile First

  • For improved accessibility 📈 for your content, it is best practice ✅ to use em for media-queries. Using this unit gives users the ability to scale elements up and down, relative to a set value.

  If you have any questions or need further clarification, feel free to reach out to me.

  Happy Coding! 🎆🎊🪅

  Marked as helpful

  1