Design comparison
Solution retrospective
Hello, everyone ✌️
This challenge take me much more time than I expected initially, all because of the image that is, for me, pretty complicate to work with yet. I really think there are much more to improve in that code, particularly on the Javascript.
Any Feedback Is Greatly Appreciated!
Community feedback
- @dmuriPosted over 1 year ago
Hello, nice work!
Actually the javascript is not needed in this project. There is a css pseudo class called active that has the behavior you are looking for. There are two very simple changes in the style.scss file you can make that will allow you to remove the script.js all together.
.on{ color: white; background-color: $softMagenta; box-shadow: none; }
Change
.on
tobutton:activate
.social-actived{ border: 1px solid $softMagenta; }
Change
.social-activated
to.s:activate
Happy coding!
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