Time Tracking Dashboard using REACTjs

Community feedback

• 

  Fazza Razaq Amiarso•2,340

  @fazzaamiarsoPosted over 2 years ago

  Hello Briuwu! Great work! You sure improved a lot!

  I just want to give quick tips.

  • On your logos array, you better put it above the component. Why? because the array content doesn't depend on any state or props. So, by putting the array above, it prevents React from unnecessarily re-creating the array every time the component re-render.
  • On your onClick, you can just do onClick={handleButton} as the event argument will be automatically passed to the callback function.

  I hope it helps! Cheers!

  Marked as helpful

  1

  ---

  

  P

  Briuwu•750

  @BriuwuPosted over 2 years ago

  @fazzaamiarso I see, I still get confused about how to properly managed my code so this is really helpful! Thank you~ (≧∇≦)ﾉ

  0
• 

  Kamasah-Dickson•5,590

  @Kamasah-DicksonPosted over 2 years ago

  Your solution looks great and responsive on smaller devices also.

  I don't see any problems with your buttons everything is working.

  • Also try to fix the accessibility issue.

  Besides good job there👍 Have a nice day👍💻

  1

  ---

  

  P

  Briuwu•750

  @BriuwuPosted over 2 years ago

  @Kamasah-Dickson thanks!

  1

  

  Kamasah-Dickson•5,590

  @Kamasah-DicksonPosted over 2 years ago

  @Briuwu you are always welcome

  1