Time Tracking Dashboard using ReactJS, and CSS Grid

Community feedback

• 

  Johandry López•220

  @DarKaRorPosted almost 3 years ago

  Hey, that's pretty cool! I like your initiative, I decided to do the same as well now that I got to Junior

  Maybe you just didn't try, but it'd have been nice to have a max-width in the cards instead of width, along with a width of 100% so it would resize when needed, and also maybe a transition between both of the viewports, but if you only wanted to make the designs given I totally get that!

  The only thing that I belive I can suggest is to check your handleChildClick function since it seems to have a lot of repeated code. Lines 20 to 25 can be put into a function, if you want even more, you could give the function two parameters for the set functions (data.[type] and "day,week,month"). I also used and arrow function that calls the handleClick passing the type so that could also save you having to write each type by its own.

  I really liked your active states and transitions! Keep it up

  Marked as helpful

  1