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 summarize component

@gracebir

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


The only thing that was difficult is setting the background gradian

Community feedback

Haiko 200

@Haiko-E

Posted

Hi there,

if i may, i got some feedback points. Maybe its helpfull for you.

  • Does the solution include semantic HTML? i see your useing a lot of div's maybe you can replace them with some other sematic HTML like a ul with a li's maybe a main

  • Is it accessible, and what improvements could be made?I'm still learning about accesibility, so can't give you any advice about that

  • Does the layout look good on a range of screen sizes?Yes, only thing i found is the align items on the div directly under the root( make it a main) should be removed on mobile, it gives some white space on the top and bottom

  • Is the code well-structured, readable, and reusable?Yes

If you would to checkout mine, every feedback is welcome!

You can see my solution here: https://www.frontendmentor.io/solutions/resultssummarycomponent-pkgZZXVKsw

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