Design comparison
Solution retrospective
The most diffcult to my was the hover over the image with the icon
Community feedback
- @Catalyst497Posted over 2 years ago
Hey Jose.
I've seen your project solution and I must say, you did a good job regarding accuracy and neatness. Couldn't look at your code though. GitHub is telling me it is not found.
A few improvements you could make though... You see that icon that displays when you hover over the image, you could actually make it transition. How? You can have the two divs stacked on top of one another. Upper for the icon and lower for the image. You can then make the upper one to have an opacity of 0 on default and of 1 when hovered on, then add a transition property for the opacity for whatever duration of your choice. And voila!
You have a few issues on accessibilty, I guess you should use landmarks such as <header></header>, <div id="root"></div> in your code. Would help a bit with the accessibility.
I am also a junior developer like you and would love to see future projects that you build. This is my solution to this particular challenge: https://www.frontendmentor.io/solutions/nft-preview-card-AMDO5woVyU
Have a nice time coding.
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