@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 ali's
maybe amain
-
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