Design comparison
Solution retrospective
did it from scratch, thanks for your reviews.
Community feedback
- @BhekiAccionPosted over 2 years ago
Looks great, well done. Something to note, the original card seems to have a shadow of some sort, or as if the card is nested inside another div, kindly take a closer look at that.
On accessibility, kindly make sure that all your img tags have an alt text as well. An alt text can be anything really, short description of the image, or you can say Just an image if you want.
All the best.
Marked as helpful0@SupafeiPosted over 2 years ago@BhekiAccion Thanks for the advice , I appreciate. I'm going to look for that shadow! cheers
0 - Account deleted
Hello there! 👋
Congratulations on finishing your challenge! 🎉
I have some feedback on this solution:
-
you can add the image and background color on hover with pseudo elements
:before & ::after
if you are not sure what i am talking about you can learn from my solution -
check the HTML report to fix accessiblity issues
i hope this is helpful and goodluck!
Marked as helpful0 -
Please log in to post a comment
Log in with GitHubJoin our Discord community
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