Interactive chart using React

Community feedback

• 

  Ibrahim San•230

  @ibra-sanPosted about 2 years ago

  Hey! Congrates on completing the projects. Superb job!

  Haha I need to take some few points from you myself. If would allow me to be a bit nit picky, then this would be my only suggestion.

  1. The bar should be green color depending on the day of the week, not on the cursor click. For instance if today is Monday then once the page is loaded the bar representing today should be colored green indicating the expenditure of today. This could be achieved by getting which day of the week is it using the date function in JS. See if todays day matches the names of the day in your HTML. If it does then change the background to green, else leave it as normal.

  This suggestion is after serious nit picking. Congrates man on a superb job and have a nice day.

  Marked as helpful

  1
• 

  Elaine•11,400

  @elaineleungPosted about 2 years ago

  Hi Joel, I think this was really well done, and good job building it with React. Just to follow what Ibramhim mentioned, right now you got the Monday bar colored and also with the tool tip amount showing. The tool tip tag is meant to be shown when hovered over, just like how the bar color change when hovered over; in other words, when you hover over the bar, two things happen: the color changes to a lighter one, and the tool tip appears. Other than this, I think everything looks good; the font size in desktop view just needs to be bigger and some more padding can be added, and that would look closer to the original. Great work on the whole! 😊

  Marked as helpful

  0