Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Huddle | SASS + JS

@WillamiFerreira


Design comparison


SolutionDesign

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

@dmuri

Posted

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 to button: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 GitHub
Discord logo

Join 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