Design comparison
Solution retrospective
I'm proud of my code being readable and maintainable with a minimum number of lines of code.
What challenges did you encounter, and how did you overcome them?Nothing to report
What specific areas of your project would you like help with?Nothing to report
Community feedback
- @yas-avocadPosted 7 months ago
Hey!
-
You should move the
transition: ease-in-out 0.3s
to.social-links
rather than have it in.social-links a:active
. When you have the transition underactive
, the transition only applies to the first half of the activity, so that's why the color snaps back when you move the mouse away. -
Also, instead of having it
ease-in-out
, you should write the code like this:
transition: color .15s, background-color .15s;
This way, the text color and background color transition together at the same time.
-
For style, your
border-radius
can be a little less. Try using 10-12px for the container rather than 20px. -
Also for style, you can increase the padding around the main container. Maybe try 38px-42px. This will give the contents more room to breath and get you closer to the targeted design.
:)
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