Time Tracking Dashboard

Community feedback

• 

  T.Praveen•870

  @gwtpraveenPosted over 2 years ago

  good job completing the challenge!

  to fix the issue,

      .card .data {
          position: relative;
          z-index: 4;
      }
  

  You do this because you set the z-index: 3 of the class icon and must set position relative because the z-index has no effect on the element if it is not positioned.

  i think this will help!

  Marked as helpful

  0
• 

  Abd•110

  @abd-2023Posted over 2 years ago

  Hey @Andes I have some pointers for you :

  1. I think you are overusing flex unncessarily try to reduce that like in main and .character-card there was no need
  2. I would suggest use background color on on body so that the whole web page has color and not avoid part which looks white and very odd from the rest of the page.
  3. Remember try to use flex on body and center things(vertically and horizontally) on the page whenever there's enough space remaning.
  4. Try to increase font size for text in footer and left sidebar.
  5. I think there's a lot of space b/w number and 'hrs' text and it's not need IMO and please focus on spacing between other elements too as specified by the design.
  6. there's no need to put a in p tag you can just you a independently.
  7. if you want to put 'icon' elements behing 'data' you can increase z-index of 'data' elements which you have done but haven't used position on it which is why it didn't work and also there's no need for div here img will work fine on its own. Hope you find these comments useful.

  0