Submitted about 3 years ago
Mobile First Social media dashboard with theme switcher challenge hub
@Jung2313
Design comparison
SolutionDesign
Solution retrospective
any feedback is welcome
Community feedback
- @ahmedAgawadPosted about 3 years ago
Great work ,
but i think you should use a localStorage in javascript so the theme that the user enable it will be saved for the next time
0 - @Dharmik48Posted about 3 years ago
Hey👋,
Great job with the solution! Looks really nice! Just a few issues:
- Use semantic html tags like
main
,header
,footer
,section
, etc. instead of just div. It is a very good practice and also good for SEO. - And a couple of minor issues: The
background
doesn't match the design and the white space between the cards is a bit less than design, but these two are not so important.
Keep Developing👍
0 - Use semantic html tags like
Please log in to post a comment
Log in with GitHubJoin 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