Design comparison
SolutionDesign
Solution retrospective
Nice easy little three column component. Helped a lot doing mobile first then making desktop responsive after and converting display to grid. Any comments or feedback is always appreciated.
Community feedback
- @dev-mksinghPosted over 2 years ago
Hey @asha12562, good effort. Though you need to improvise some of the responsive issue :
- When you'll resize and check the site you'll find that at 702px your buttons don't align properly.
you can do it by setting
height
property to a fixed value, if you are using flexbox. - The content is overflowing when you resize it.
Okay the last is very minor but important: kindly style the active state. for example:
button:hover { cursor:pointer; background-color: transparent; }
Hope it will be helpful to improve your code.
Marked as helpful0@asha12562Posted over 2 years ago@dev-mksingh I forgot the hover states! That's my bad. Thank you for the feedback, always appreciated
0 - When you'll resize and check the site you'll find that at 702px your buttons don't align properly.
you can do it by setting
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