Design comparison
SolutionDesign
Community feedback
- @denieldenPosted almost 3 years ago
Hi Nelson, congrats great job!
I've looked at your solution and have some suggestions that I hope will help you:
- Give the flexbox and
heigth
properties to thebody
for center the card and removemargin
from.container
class. Note: Flexbox aligns to the size of the parent container. - You can use the
vh
measurement for the height... Viewport Height handles the sizing of an element in relation to the height of the browser window. - Through the
:hover
state it changes the color of thebutton
- After adding a little
transition
on the image hover effect
Hope this helps. Keep up the good work!
Marked as helpful1 - Give the flexbox and
- @nelsonuprety1Posted almost 3 years ago
Thank you deniel. I will definitely keep the things that you have mentioned in mind and implement it.
1@denieldenPosted almost 3 years ago@nelsonuprety1 You are welcome! And don't forget to mark my comments as helpful!
Thank you and 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