flexbox,css-grid,sass,css-variables, theme switch with css only.

Community feedback

• 

  Ayush Gandhi•220

  @ayush577Posted over 4 years ago

  I believe you did amazing work, but there are a few things. I think you have to consider

  1. Theme switcher toggle should have a gradient background all the time
  2. Overview cards should have more height and equal padding from all the sides

  0

  ---

  

  Gerben Dol•3,115

  @GerbenDolPosted over 4 years ago

  @ayush577 I think the toggle is fine right now, at least according to the design. In the light version of the design the toggle has the grey background. Only on hover and on the dark version does it have the gradient background.

  Great feedback on the padding! I think having that evened out would really tighten up the cards look.

  1

  

  Ayush Gandhi•220

  @ayush577Posted over 4 years ago

  @GerbenDol I think you right 😅 about toggle. My bad😑

  1