Time Tracking Dashboard using plain JavaScript

Community feedback

• 

  Mirna Andrišić•350

  @mandrisicPosted 24 days ago

  hello, I just threw a quick glance at your code and I have some things you could improve:

  • you didn't use the font that was used in the design preview
  • smallest mobile version 320px doesn't look good, it's not centered (probably because the container is too big)
  • tablet version would look better if it also used the same design as the mobile version (everything in one column)

  just practice and you'll se these things more clearly.

  Marked as helpful

  1

  ---

  

  P

  kephalosk•260

  @kephaloskPosted 24 days ago

  @mandrisic hi, thanks for your review :)

  • font: there was a typo. I tried to relate "rubik" instead of "Rubik". Now It works.
  • width: like a neverending story. In my last Project we agreed on 350px as min-width. So I set the min-width of the cards to 300px (+48px padding = 348px). Anyway, I applied your suggestion and set min-width of cards to 270px. Now it looks good til 318px device-width.
  • tablet-version: I personally think, that 1 column looks even for the highest mobile-width (my breakpoint is 600px) a bit off. My Idea to improve it would be to fix the profile card to the left side of the screen for tablet. So you can scroll and still see the profile info and the Switch-Labels Daily/Weekly/Monthly if you scroll down to far (instead of an empty left column).

  Have a great day :)

  0

  

  Mirna Andrišić•350

  @mandrisicPosted 24 days ago

  @kephalosk feels good to be of help :)

  0