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

Responsive Scoreboard Using Vite + React

@MuhammadSamiAhmad

Desktop design screenshot for the Results summary component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

P

@JosielLima

Posted

Great job! Your code is really readable and well-organized, which makes it much easier to understand and maintain.

I have just two suggestions for improvement:

Use of H1 tags: I noticed an excessive use of the <h1> tag. This can negatively impact the semantics of the page, especially in terms of accessibility and SEO. It would be good to review the use of headers to maintain the appropriate hierarchy.

Using JSON for colors: The solution you implemented using JSON to store colors is innovative, but based on my experience, it’s not common to receive this type of information directly from the backend. A more traditional and perhaps more suitable approach would be to define these colors directly in the frontend, even if it slightly increases the verbosity of the code.

Overall, the work is excellent! These are just suggestions to further optimize the solution.

Marked as helpful

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