Social media dashboard with theme switcher

Community feedback

• 

  Kethmar Salumets•195

  @kethmars

  Posted about 5 years ago

  You can check how I solved it from here: https://youtu.be/zfXb3CNBHMQ

  10
• 

  

  Matt Studdert•13,611

  @mattstuddert

  Posted about 5 years ago

  Also, have you ever tried using min-width media queries instead of max-width? It's quite a common workflow with front-end developers to use them and work mobile-first. It can often lead to less CSS code and has the benefit of loading in fewer styles for mobile users, which can be a nice performance gain.

  01
• 

  

  Matt Studdert•13,611

  @mattstuddert

  Posted about 5 years ago

  Awesome work on this challenge, Kethmar! 🙌

  Great to see you posting the video of you working through it as well. At the moment, the toggle isn't working. In the JS you've got const toggleButton = document.getElementById('toggle'); but there doesn't seem to be an element with an ID of "toggle" in your HTML.

  Your layout looks really good though and attention to detail with the design comparison is great. One small addition could be changing up the responsiveness slightly so that it goes from a single column on mobile to a 2-by-2 grid on tablet and then rows of 4. As opposed to going from single all the way to a row of 4 in one jump.

  Keep up the great work! I'm loving the videos and they'll help a lot of people I'm sure! 🙂

  01