Responsive Scoreboard Using Vite + React
Design comparison
Community feedback
- @JosielLimaPosted 3 months ago
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 helpful0
Please log in to post a comment
Log in with GitHubJoin 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