Social-media-dashboard-with-theme-switcher

Community feedback

• 

  Yemisrach•1,430

  @Yemisrach15Posted about 2 years ago

  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