Result summarize component

Community feedback

• 

  Haiko•200

  @Haiko-EPosted over 1 year ago

  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