@VCarames
Posted
Hey there! 👋 Here are some suggestions to help improve your code:
-
Increase 📈 the
width
of the component to better match the FEM example. -
Along with the blank
Alt Tag
, you also want to include thearia-hidden=“true”
to the “illustration*”and “music icon” to fully remove them from assistive technology. -
The only heading in this challenge is the ”Order Summary". Every other text should wrapped in a
Paragraph
element. -
Your CSS Reset is extremely bare and being underutilized. To fully maximize your CSS reset, you want to add more to it.
Here are few CSS Resets that you can look at and use to create your own or just copy and paste one that is already prebuilt.
https://www.joshwcomeau.com/css/custom-css-reset/
https://meyerweb.com/eric/tools/css/reset/
http://html5doctor.com/html-5-reset-stylesheet/
If you have any questions or need further clarification, feel free to reach out to me.
Happy Coding! 🍂🦃