Hi
This is touching all screen edges on mobile and the attribution is overlapping the card content. There should be space between the component and screen edge according to the design.
There are some big problems with the html but are easy to fix.
-
Anything with a hover style in the designs must be an interactive element. That's what having a hover style means, you expect it to be clickable. But you have not used interactive elements, so they are not clickable for keyboard or screenreaders etc
-
Text should never be in a div or span alone. Use a meaningful element like a paragraph tag
-
Look up how to write alt text and how to treat decorative images. All the alt attributes on this need rewriting
I hope that helps
Marked as helpful
@shravankumartalabhaktula
Posted
@grace-snow Hi Grace, thank you so much for your detailed feedback.
- "Anything with a hover style in the designs must be an interactive element." -- you mean <a></a> or <button></button> tags?
@shravankumartalabhaktula in this case yes those are the interactive elements for this challenge
Marked as helpful
