Design comparison
Solution retrospective
Any feedback and best practices advice would be appreciated!
Community feedback
- @zoleee98Posted over 2 years ago
i think its perfect, but i one thing.
as you can see the eye image is transparent like the hover background, and it shouldnt be like that
just make an after pseudo only for the eye image and it wont be transparent
i hope it helps
Marked as helpful1@kkrataPosted over 2 years ago@zoleee98 Thank you! I didn't know how to remove opacity only from the background image, but creating another pseudo element worked
1 - @shashreesamuelPosted over 2 years ago
Hey good job completing this challenge.
Keep up the good work
Your solution looks great however I think that the divider line needs less opacity using
opacity
.In terms of accessibility issues simply wrap all your content within main tags
I hope this helps
Cheers Happy coding š
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