My attempt to chart making

Community feedback

• 

  WandoCode•840

  @WandolePosted over 2 years ago

  Hey, that's a pretty honest work that you've done! It's very not far from being very good!

  The advices I could give you:

  • try to stick more to the given design (for example: add a sparator between the chart and the .content_summary, add a border radius for the graph bars, improve the font-size on some elements, etc.)
  • In the JS, you coded well the hover effect; but it would be better (and easier) to use .bar:hover pseudo-class in your CSS to style the component when the mouse is hovering on it!
  • From a general point of view, you should remove every styling from your JS (beside height that is usefull in the challenge) and put it in you CSS (the background-color for example). In your JS, just add or remove class from HTML elements.

  I hope it will help you!

  Marked as helpful

  0