Design comparison
Solution retrospective
second attempt at this challenge. this time is used flexbox and made it more responsive. still couldn't figure out how to change the social media icons color. changing the fill didn't seem to affect it in anyway
Community feedback
- @Floor20Posted over 2 years ago
Good work! Vanza Setia gave a good answer on the icons. Also, there are lots of free external resources you can use for icons that are come pre-filled, such as Ion Icon.
Also, I noticed your solution seems to have the two sides crashing into each other. Instead of using two separate divs for each section, try wrapping them both in one container div and using flexbox to separate them, this should be easier to work with.
1 - @vanzasetiaPosted over 2 years ago
Hi there! 👋
Good effort on this challenge! 👍
The best way to change the color of the social media icons or any SVG is by using inline SVG and in the stylesheet, you can do the following.
a:hover svg path { fill: #333; }
One thing that I would like to mention is the register button should be an interactive element, in this case, an anchor tag instead of a
div
. Users of assistive technology would not be able to know that there's a link or a button ifdiv
is used for interactive elements.For your information, anchor tags are for navigation - moving to different pages or content on the same screen, while the
button
elements are for actions like opening a modal, submitting a form, toggling element, etc.That's it! Hope you find this useful! 😁
1
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