Resposive Frontend Mentor | Time tracking dashboard with flex and js.

Community feedback

• 

  Muhamad Rukhul Kirom•380

  @rukhulkirom

  Posted 3 months ago

  Hello there👋. Good job on completing the challenge!

  I have some suggestions about your code that might interest you.

  1. Currently, the <li> element is used to select a timeframe. However, <li> is better suited for static lists, not interactive ones. Replace it with <button> to improve accessibility and UX.

  2. In the profile section (.report_for), the vertical padding is quite large. Use display: flex; align-items: center; to be more flexible.

  I hope you find it useful! Above all, the solution you submitted is great!

  Happy Coding!

  11