Design comparison
Solution retrospective
Couldn't position the "eye" on hover perfectly. Need suggestions.
Community feedback
- @zoleee98Posted over 2 years ago
hi great job, but unfortunately u cant add before/after pseudo elements on images.
here is my tip:
- put that image in a div, and use before+after on that div
- i used before pseudo to make the background with div:hover::before{}
- and then div:hover::after{} for the eye image, just like this: content: url("images/icon-view.svg");
and dont forget that you can make a flexbox from before/after pseudos to center the eye image.
hope that helps;)
Marked as helpful0 - @shashreesamuelPosted over 2 years ago
Hey good job completing this challenge
Keep up the good work
Your solution looks great however I think your card needs to be scaled up a little bit using
transform: scale()
.In terms of accessibility issues simply wrap all your content between main tags
I hope this helps
Cheers Happy coding š
Marked as helpful0@sayamalviPosted over 2 years ago@TheCoderGuru Hey thanks man ! I did wrap my content inside the main tag ..
0
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