Mobile-first Time Tracker Challenge using NextJS

Community feedback

• 

  Mtalafa•350

  @MtalafaPosted about 1 year ago

  Hi, well done on completing this challenge.

  I`m not very good yet with JavaScript but I did have a quick look at your CSS.

  You can add this: transition: all 0.3s; to everything that changes on hovering, like for example the .lastweek

  This will make the transition from the original color to the hover color more smooth.

  Marked as helpful

  0

  ---

  

  Frida 🌼•90

  @FridaWaldtPosted about 1 year ago

  @Mtalafa Thank you so much for reviewing my CSS Mtalafa! I tend to forget to add transitions, thank you for pointing this out, it's great help. 😊

  0
• 

  Kenny Ng•400

  @kennylun123Posted about 1 year ago

  Hi Frida,

  I hope you're doing well! I wanted to take a moment to congratulate you on completing the coding challenge. Great job! 😊

  I also wanted to provide some feedback on your code:

  Imagine that each of your category-card is a reusable component. Inside of it you just need one div for category-card-stats. And you could set background image to category-card (parent container). You could either set the background using in-line style or assign an id to each component and use CSS selector as you did. After that you can finish the category-card-stats.

  Keep up the great work! I'm looking forward to seeing what you'll create next.

  Best regards,

  Kenny

  Marked as helpful

  0

  ---

  

  Frida 🌼•90

  @FridaWaldtPosted about 1 year ago

  Thank you so much @kennylun123 ! Just the feedback I needed, I'll try this solution, I truly appreciate you taking the time to look at my code. :)

  0