Design comparison
SolutionDesign
Solution retrospective
This one was easy ;)
Community feedback
- @Sdann26Posted over 2 years ago
Hi Just!
Reviewing your layout you have done quite well, although looking at the design I would only tell you about two small changes.
- To the
.stats
and.user-city
classes removefont-weight: 700
as the font appears thin, not thick. - To the class
.user-name
addcolor: var(--Very-dark-desaturated-blue)
since the numbers also have blue color.
Other than that everything is pretty good. I don't know if you handle BEM, which is a naming scheme for your classes so if you can use it to name your classes it's a good way to work as a team in the future, have an order and make the CSS code more readable.
Without anything else to add I hope my comments are helpful.
0 - To the
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