Design comparison
SolutionDesign
Solution retrospective
Hey, Frontend Mentor Community!
It would be awesome to get any feedback on how I could improve my code.
Community feedback
- @smcglone-lowerPosted over 2 years ago
You could use CSS variables to set up the colors.
Also, you could use CSS variables to set the background colors of the cards like so:
<div class="cards"> <div class="card orange">...</div> <div class="card light-green">...</div> <div class="card dark-green">...</div> </div>
.orange { background-color: hsl(...); } .light-green { background-color: hsl(...); } .dark-green { background-color: hsl(...); } .card { background-color: var(--bg-color); }
Marked as helpful0@Artur-NeriPosted over 2 years ago@smcglone-lower, thanks! I will give CSS variables a go on my next project, then.
0 - @MaxiTeddyPosted over 2 years ago
Just increase your line-height. Nice job
0@Artur-NeriPosted over 2 years ago@MaxiTeddy Thanks, man! I did'nt even realize that.
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