Design comparison
SolutionDesign
Solution retrospective
I'm happy to hear your feedback.
Community feedback
- @yasssuzPosted over 3 years ago
Hello Asmaa,
Amazing job, but here a few things:
- the button looks good, but you should definitely add a
cursor: pointer
to make sure the user knows that it is clickable. - I would as well add a cool animation on the button to make the transition between colors more smooth, here the code:
transition: color 0.3s ease, background-color 0.3s ease
- in the tablet version, it looks a little bit buggy, try to change the media query from 800px to 1025px.
happy coding and to support me please upvote my comment :)
1@AsmaaMKPosted over 3 years ago@Galielo-App Thanks a lot for your feedback, I'm going to modify this points.
1 - the button looks good, but you should definitely add a
- @coding404lifePosted over 3 years ago
Hello Asmaa it looks amazing keep going and learning :) just one simple note the button looks a bit wider than the image and more height you could give it something like
Height:3.2em;
andwidth:12em
other than that it looks amazing :) you doing great job0
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