Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Social-media-dashboard-with-theme-switcher

Zajac 150

@Zajaczkowski23

Desktop design screenshot for the Social media dashboard with theme switcher coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
2junior
View challenge

Design comparison


SolutionDesign

Solution retrospective


Another cool project! :D All feedbacks welcome

Community feedback

Yemisrach 1,430

@Yemisrach15

Posted

Hey Zajac, good job! The report under your solution gives you a lot of information you can improve on. But here are a few of my suggestions,

  • If an image or icon/svg has a meaning within the context its used, it should have an alt attribute with text describing the image. So maybe you should use the inline svgs as images.
  • A lot of the text you put in a div should be in a p tag. div has no meaning/semantic and is usually used for decorating or layout purposes.
  • Overview - Today should be a heading in a h2

Other than these the solution is nice but read up on semantic HTML elements.

Marked as helpful

0

Please log in to post a comment

Log in with GitHub
Discord logo

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