Time-tracking-dashboard

Community feedback

• 

  Adriano•42,780

  @AdrianoEscarabotePosted 4 months ago

  Hi Cesare, how are you doing? I really loved the outcome of your project, but I have a few suggestions that I think might be helpful:

  Using Flexbox or Grid on the body to center elements ensures a more responsive and adaptive layout, fitting different screen sizes seamlessly. It avoids manual calculations and constant adjustments needed with margin, padding, or absolute positioning. These techniques provide more consistent alignment and simplify the code.

  flexbox:

  body {
      display: flex;
      justify-content: center;
      align-items:  center;
      min-height: 100vh;
  }
  

  grid:

  body {
      display: grid;
      place-content: center;
      min-height: 100vh;
  }
  

  The rest is excellent.

  I hope you find it useful. 👍

  1
• 

  Dylan Heslop•2,380

  @dylan-dot-cPosted 4 months ago

  Hey bro this is a good solution but there are some places to improve.

  The website is not responsive as I'm seeing desktop grid on my phone

  You should also make sure the active timeframe is highlighted in white.

  There is a much better way than having HTML code in your Js, using templates

  I suggest that you have a look at my code as I think it's a good solution and I addressed all these issues and even added some URL state management, so please have a look and tell me what you think

  0
• 

  Salim•320

  @limsaelPosted 4 months ago

  I like that you used react for this project

  0