Design comparison
Community feedback
- @Yup03Posted almost 2 years ago
Great challenge @AdeMarq, And i think it's all good but the only thing that i may suggest is to add transition to your buttons to smoothly change properties when passing from "normal" to "hover" state :
button{transition: all 0.3s ease}
You can also add the cursor property to all your buttons in one place instead of putting
cursor:pointer
to each hover state of each button you could just dobutton{cursor:pointer}
and it will be applied to all your button elementThere is some padding that you can add especially to the top of the container when adding breakpoint :
@media (max-width: 930px){ .container{ padding-top:100px;/*You can also adjust the value to your need*/ }
That's in my opinion some improvement that can be made . Hope that you will find it helpful
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