Hover effect over chart done with CSS and Javascript.

Community feedback

• Account deleted

  Hello Austin, congrats completing the component. Some points regarding the responsiveness.

  • When applying width or height to elements is a best practise set min or max, that will free the element from static size. The body element has a heigh:100vh, try a min-height instead. Also a fixed width is unnecessary.
  • It seems there is extra padding and margins on the overall component. Could be a good idea to decreasing units for both properties in several elements.
  • There are elements sized with px and others with rem or em , try be consistent across your CSS, that will allow you to control the overall sizes and spacing easier.
  • Some font-size are disproportionate. For example the <h1> nested in div.numb-month-total has a font-size of 60px.
  • Check always the automatic report generated by the plattform. In this case it seems there is duplicationid.

  I think that those points could really increase responsiveness in your project. Other than that I do really like how you styled the bar chart and the JS logic seems pretty well organized. Have a gooda weekend and see you around!

  0