Design comparison
Solution retrospective
always welcome feedback for improvement.
Community feedback
- @MohamedAridahPosted over 2 years ago
Hello @justcoder42022, you did good job for this challenge.
However i have some notes:
-
change
height: 100vh
of thebody
to bemin-height: 100vh
. this will allow content to take more height of the body if it needs to. -
use
transition
property for smoother hover effect. -
use
overflow: hidden
for.wrapper
. by doing this you don't need to setborder-radius
for the image. -
there is extra space on the top and bottom of the
main
element. try to know it's source and remove it. -
you can see My solution for this challenge it may be helpful for you..!
I hope this wasn't too long for you, hoping also it was useful😍.
Goodbye and have a nice day.
Keep coding🚀
Marked as helpful1 -
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