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

Expenses chart component solution using React

#react
Akanksha 80

@akankshav108

Desktop design screenshot for the Expenses chart component coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
2junior
View challenge

Design comparison


SolutionDesign

Solution retrospective


All advices are welcome.

Community feedback

Prince Roy 1,210

@iprinceroyy

Posted

Great @akankshav108, I would suggest some points here:-

  1. There should be one level heading tag like h1 to avoid accessibility issues, if there's no requirement of the heading element, then set its font size to 0. h1{ font-size: 0; }
  2. There should be one main landmark where goes the content like this ➡️ <body><main>your content..</main></body> to avoid accessibility issues.

Hope this adds to your learning, Happy coding :)

Marked as helpful

0

Akanksha 80

@akankshav108

Posted

@iprinceroyy Thank you so much for your suggestions. Just these 2 points removed all 12 of my Accessibility issues.

0
Prince Roy 1,210

@iprinceroyy

Posted

@akankshav108 Cool! Glad to help you.

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