@FelipinDR
Posted
You should not add an alt tag to the icons because they're just icons, also you should use "aria-hidden:true" to hide them from accessibility Good job finishing this, good luck with the next challenges
Marked as helpful
@DammyD
Posted
@FelipinDR I'll make sure to put this feedback to use in my next challenge. Thank you.
@suhaib52
Posted
You did it in a very good manner. Your code is well structured and readable.
It would be nice if you start using semantic html tags, these tags would reduce the accessibility issues.
You can also use :root to define the colors and then call them by using var() in your stylesheet. When you use the colors as a variable than your css code will look more precise.
Marked as helpful
@DammyD
Posted
@suhaib52 I really appreciate your feedback. I promise to put this feedback to use in my next challenge.
Thank you.
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord
