Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Result summary component using HTML and CSS Flex

Henry 40

@stan545

Desktop design screenshot for the Results summary component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

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

P
beowulf1958 1,170

@beowulf1958

Posted

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 helpful

0

Henry 40

@stan545

Posted

@beowulf1958 Thanks, will do that ASAP.

0

Please log in to post a comment

Log in with GitHub
Discord logo

Join 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