Social media dashboard & theme switcher SCSS, Flexbox, Grid, CSS Vars

Community feedback

• 

  Alex Kim•1,325

  @alex-kim-dev

  Posted about 5 years ago

  Wow. That's a great pixel perfect solution, Simon! And there's a lot of clever sass techniques. I will definitely look more closer into your code to learn something new.

  21
• 

  

  Matt Studdert•13,611

  @mattstuddert

  Posted about 5 years ago

  Awesome work on this challenge, Simon, and congrats on submitting your first solution. I hope you enjoyed the challenge. Your project looks amazing and scales down really well to mobile.

  The one thing I'd recommend improving would be to remove the display: none; declaration from the theme toggle checkbox input. Doing this makes that element inaccessible to anyone who's not using a mouse/trackpad to navigate the page. You've got an sr-only class in your code, so I'd recommend using that to visually hide the input but keep it there for assistive technology users.

  Keep up the great work! 👍

  11