Design comparison
Solution retrospective
My difficulty was how to get the all values from buttons, but after doing a little search simple doing in JavaScript: querySelectorAll(), i obtained all values from different buttons. After that i figured out i had to do a little loop with forEach() method.
Community feedback
- @correlucasPosted about 2 years ago
👾Hello @pereira1185, Congratulations on completing this challenge!
Great code and great solution! I’ve few suggestions for you that you can consider adding to your code:
1.Your component is perfect, but is not responsive yet, this is due the
fixed width
you've applied to the container.Look bothwidth
andmax-width
the main difference between these properties is that the first(width) is fixed and the second(max-width) isflexible
, for example, a component withwidth: 320px
will not grow or shrink because the size will be ever the same, but a container withmax-width: 320px
ormin-width: 320px
can grow or contract depending of the property you've set for the container. So if you want a responsive block element, never usewidth
choose ormin-width
ormax-width
.2.Add the website favicon inserting the svg image inside the
<head>
.<link rel="icon" type="image/x-icon" href="./images/favicon-32x32.png">
✌️ I hope this helps you and happy coding!
Marked as helpful0
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