Design comparison
Solution retrospective
First attempt, and brand new to this. Appreciate some of the code will be messy and inefficient so any help in this regard would be appreciated.
Also struggled with the image active state. Changed the opacity but was unable to change the colour to cyan.
Thanks for your feedback in advance :)
Community feedback
- @atif-devPosted almost 2 years ago
Hi Tom, you did good. I suggest, you also implement first hover effect properly. When we hover over the image an eye icon appear, right?(eye icon is missing in your project). You can also learn about image overlay here:
https://www.w3schools.com/howto/howto_css_image_overlay.asp
Hope you get this Feedback Useful👍😇
Marked as helpful1@tplatt92Posted almost 2 years ago@atif-dev Thank you for the feedback, it took me a while to figure it out but i've finally made it work. :)
1
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