social media dashboard using javascript | HTML | CSS

Community feedback

• 

  Ovidiu-Antonio•3,125

  @ovidiuantonioPosted over 4 years ago

  Hello,

  You did a very good job on this. The site is responsive and it looks very good. But I found some things that you can improve to make the solution even better:

  • just when the page loads, the hover color on the cards is kinda too bright, but it changes when I toggle the themes
  • replace the cursor: grab; with cursor: pointer; since the switcher is not draggable
  • you can align the arrow and percentages with flexbox. Flexbox is a very good way of aligning elements and I use this pretty much everywhere
  • instead of toggling classes you can use css variables in your css. Variables can be modified in js. That's an easier way to create a theme switcher

  Happy coding! Keep going!

  1