Design comparison
Solution retrospective
Any feedbacks are welcome!
Thanks!!
Community feedback
- @dnksebastianPosted over 2 years ago
Hi @elyssontanaka
Your solution looks clean and it's very well done! One small thing you might want to consider is using 'transition' CSS property on your hoverable elements, just to make the color changes smooth. You can also use 'background-size: cover' on your body element in order to make it work on all resolutions.
Keep it up!
Marked as helpful0@elyssontanakaPosted over 2 years agoHi @dnksebastian, thank you for your feedback! I totally forgot to add the transition for the hover states, I'll fix that! About the background-size, when I use "cover" it gets very different from the design preview, I'm not very sure why.. so I didn't use it on the project, but I agree it's the best option.
Thanks again!
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