Skip to content
Submitted over 1 year ago

Dynamic Results summary component

Zosima•220
@SantiagoPonce
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Results summary component
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

Hi! 😀

After a while without doing any exercise I wanted to get the most out of it. I was really pleased with the results and the dynamism added

  • You can enter your score
  • The page will inform you how well you did compared to the other (fictitious) users loaded in a json file

(although the documentation and js functions need to be improved) Another aspect to improve could be to make the "Great" qualifier dynamic (which can be too optimistic 🥳 even if our rating is very low)

Any suggestion is welcome :) Have a good day!

Community feedback

  • Finney•3,030
    @Finney06Posted over 1 year ago

    Hello there 👋. Good job on completing the challenge !

    Here are some suggestions regarding your code that may be of interest to you.

    HTML 🏷️:

    To clear the Accessibility report:

    • Always avoid skipping heading levels; Starting with <h1> and working your way down the heading levels (<h2>, <h3>, etc.) helps ensure that your document has a clear and consistent hierarchy.

    I hope you find it helpful!😏 Above all, the solution you submitted is 👌. 🎉Happy coding!

    Marked as helpful
    1
    Zosima•220
    @SantiagoPoncePosted over 1 year ago

    @Finney06 Hi! Thanks for your comment, I was thinking of putting an h1 with an sr-only class but I think that because I wanted to upload it as soon as possible I didn't implement it 😅 I will

    0

Please log in to post a comment

Log in with GitHub

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