@ApplePieGiraffe
Posted
Hey there, Manuel Salvador! π
Good work on this challenge! π
A couple of things I suggest are,
- Setting the
alt
text for the icons in the card component to be an empty string so that they will be ignored by screen readers. Thatβs because those elements aren't necessary or important to the content of the page and as a result doesnβt need to be read by screen readers. - Perhaps making the card component an
article
tag since it is a standalone component and already contains a heading inside to identify it. That'll make your markup slightly more semtantic! π - Adding a favicon to the site. There should be one in the starter files for this challenge that you can use.
Hope you find these suggestions helpful. π
Keep coding (and happy coding, too)! π
Marked as helpful