Social Media Dashboard with Theme Switcher

Community feedback

• 

  

  Grace•32,130

  @grace-snow

  Posted over 3 years ago

  I'm afraid the content on this is completely inaccessible at the moment. Try reading it with no images or styling(or try with a screenreader) and see how it becomes incomprehensible. That's what search engines and screenreader or other assistive tech users would get.

  It's actually the hardest part of this challenge, getting the semantics right. It definitely needs hidden headings and aria-labelledby and -describedby attributes

  And header__subtitle should n I t be a h2. That's data, numbers that would change so would never be a heading.

  I'd also use aria labeled by on the theme toggle - you have dark theme written right there in the html, let that be the label of the toggle

  Good luck

  01

  Marked as helpful
• 

  

  Grace•32,130

  @grace-snow

  Posted over 3 years ago

  Content is spilling off screen for me. Remember to use max width not width on elements to allow them to grow up to a point and shrink as needed

  I'll pop some Screenshots in slack for you

  00

  Marked as helpful
• 

  Anosha Ahmed•9,300

  @anoshaahmed

  Posted over 3 years ago

  To get rid of the accessibility/HTML issues shown in your Report, <section> and <article> usually need a heading; so if you don't need a heading in it, use some other element such as <div>

  Great job! :)

  01

  Marked as helpful