Skip to content
Submitted 10 months ago

Results Summary Component

Jack-OC•250
@Jack-OC
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

I decided to try dynamically populating the scores using the JSON data in the file provided. It does appear to work, but the code looks a bit messy. I would be thankful for any suggestions.

Community feedback

  • Daniel 🛸•44,310
    @danielmrz-devPosted 10 months ago

    Hello @Jack-OC!

    You did a very good job there!

    I have a very simple suggestion for improvement:

    • Since the button is a clickable element, it's nice to add cursor: pointer to it.

    📌 This gives the user a visual indication that the element is clickable, as obvious as it may look.

    I hope it helps!

    Other than that, you did a great job!

    Marked as helpful
    0
    Jack-OC•250
    @Jack-OCPosted 10 months ago

    Thank you for the feedback. I genuinely thought I’d put that in there, I must have deleted by accident. It's always something small you forget.

    2

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