Submitted over 3 years ago

mobile first / CSS grid + flexbox /SCSS social media dashboard

Salah Eddine• 160

@dacitto

Preview site View code

This is a solution for...

Social media dashboard with theme switcher

HTML
CSS
JS

2junior

View challenge

Design comparison

SolutionDesign

Solution retrospective

Hey There 👋, hope you take a look 👀 and give me your helpful feedback🖊️ 💖Thank you💖

Community feedback

ApplePieGiraffe• 30,565

@ApplePieGiraffe

Posted over 3 years ago

Hey there, Salah Eddine! 👋

Great work on this challenge! 🙌 Like GerbenDol mentioned, you've done a nice job—everything looks good and responds well! 👍

In addition to clearing up your solution report, I only suggest adding a focused state (such as by adding an outline) to the toggle-switch so that keyboard users can easily tell when it is focused. 😉

Keep coding (and happy coding, too)! 😁

Salah Eddine• 160

@dacitto

Posted about 3 years ago

Thank you a lot @ApplePieGiraffe , I will do what you mentioned, hope you see my other challenges I appreciate it 🙏.

Gerben Dol• 3,115

@GerbenDol

Posted over 3 years ago

Hi Salah Eddine,

First thing I noticed: your solution is looking really nice! Loving the spacing, typography and overall feel. Well done! 👌

A thing to consider:

Check out your accessibility and HTML issue rapports

Honestly, going through your code I think you did really well and I don't have a lot of feedback. Keep up the great work! 😁

Salah Eddine• 160

@dacitto

Posted over 3 years ago

@GerbenDol Thanks a lot Gerben for your feedback I appreciate it 🙏

Please log in to post a comment

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord