@ApplePieGiraffe
Posted
Greetings, Jairo Atoche! 👋
Great job on this challenge! 👏 It's great that you're using Sass to add superpowers to your CSS! 😆 I hope you enjoyed using it!
Here are one or two things I'd like to suggest,
- Getting rid of the
header
tag in your HTML (since it is empty and therefore unnecessary in this challenge). - Using a tag such as the
span
orp
for the price in the card component (not a heading tag, since it is not a heading for anything). - Setting the
alt
text for the icon inside the button to an empty string so that it will be ignored by screen readers (since it is not nearly as important as the text in the button and does not need to be read out loud). 😉
Hope you find this helpful. 😊
Keep coding (and happy coding, too)! 😁
Marked as helpful