HI, It would be useful to have some feedback on this please. I feel most of it is ok but I had a few problems with the buttons including hover.
@shashreesamuel
Posted
Hey good job completing this challenge.
Keep up the good work
Your solution looks great however I think the individual cards can be a bit wider.
In terms of accessibility issues simply wrap all the content between main tags
I hope this helps
Cheers Happy coding 👍
Marked as helpful
@Dany-GitHub
Posted
You did great👌😊 Here are some tips to help you u can set body{ height: 100vh;,display: flex; justify-content: center; align-items: center; } to center everything instead of using margin to center the main element, for the button u can add border: none; to remove those border things , add some padding to make it wider , button { padding: 1rem 2rem } , then for hover u can do this button:hover { background color: (use the color depend on the background of the card ); border: 4px solid white; border-radius: 35px; } change the radius to make it half circle happy coding keep going ✌😊
Marked as helpful
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
