Submitted almost 3 years ago

Using JavaScript and CSS to dynimcally size charts from JSON file

@mshaynerush

A solution to the Expenses chart component challenge

Solution retrospective

If you can see a better way to add the elements to the page. I used a lot of divs, and while I don't think that's a terrible idea, I am wondering if I made this more complex than it had to be.

Also I am sure there are things I could have done with less code, so any feedback in that area would be sweet.

Thank you for taking time to review.

Code

Please log in to post a comment

Community feedback

No feedback yet. Be the first to give feedback on Shayne Rushton's solution.

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