Solution retrospective
Added focus styles on top of the required hover style.
Used "clickable parent" technique for the card.
What challenges did you encounter, and how did you overcome them?I wasn't sure on the responsive image with fixed height. I've looked online and asked ChatGPT for help, in the end only thing I missed was width and height set to 100% on the image itself.
What specific areas of your project would you like help with?The screenshot comparison here on Frondend Mentor shows the fonts differently. When tested in Safari, Chrome, Edge, Firefox and Brave the fonts show up same as in the design file.
Please log in to post a comment
Log in with GitHubCommunity feedback
- @ArtemGolovko
The solution has no visual defects, which is good.
This might be a nitpick, I think that the solution is overengineered. And I think it could be made simpler.
Marked as helpful - @trucanh21
You forgot to add CSS for the h1:hover selector so that when users click on it, the color changes. You should add the color change effect and include transition: color 0.3s; in the h1 tag to create a smooth color transition when hovered.
Marked as helpful
Join 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