Design comparison
Solution retrospective
I've just completed a front-end coding challenge from @frontendmentor! 🎉
You can see my solution here: https://www.frontendmentor.io/solutions/result-summary-component-using-html-and-css-flex-aL-_lEcDzc
Any suggestions on how I can improve are welcome!
Community feedback
- @beowulf1958Posted about 1 year ago
Congratulations on completing your first challenge! Colors are great, gradients are working, space between elements all good. You used CSS variables and the design is responsive. However, the desktop version is tall and narrow, so that the scores sit below the labels. You can easily fix that by changing the width on the desktop version to 750px.
Your HTML is clean and your CSS well organized. However, I did notice that you used heading tags (h3, h4, h5, h6) for styling and not for structure, which is a big no-no. You should give a class to those elements that need styling and use CSS to get the styles.
<p class="title">Summary</div>
.title { font-size: 1.17em }
I look forward to see what you do next.
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