Design comparison
SolutionDesign
Solution retrospective
Please don't hesitate to leave me feedback. Constructive criticism is very welcomed.
Community feedback
- @IEdiongPosted almost 3 years ago
Hello 👋, great job on completing this challenge. Here are some things I noticed;
- fix the color of the
title
andowner name
#title { color: #ffffff; } .ownerName span { color: #ffffff; }
- add some padding at the bottom of the priceAndTime class
.priceAndTime { padding-bottom: 0.5em; #or you could add more }
- the color of the eye of the image that is to be displayed on hover of the image is meant to be white.
- Also, look into the accessibility and HTML issues raised here.
I hope this was helpful. Shalom.
1 - fix the color of the
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