Solution with React, Context Api and Grid

Community feedback

• 

  Raymart Pamplona•16,100

  @pikapikamartPosted about 3 years ago

  Hey, great work on this one. The layout is really good. This challenge is one of my favorite here in fem :>. The mobile layout is good as well and the responsiveness is great as well.

  Some suggestions would be:

  • On this one, I prefer adding the "Social Media Dashboard" heading tag in the header element since it is up on top along with the total followers and color toggler.
  • Your theme switcher lacks a visual indicator on its focus-visible selector. Adding an outline or box-shadow would be a great addition on the focus-visible state so that users would know where they are when navigating in this. The theme toggler as well lacks screen reader information, use an aria-label attribute like aria-label="dark mode" or more descriptive one. Personally now, when I am implementing a colormode toggler, I would use the fieldset attribute with the legend. Contained on those are the radio buttons that will chosen upon on the theme color modes. You can see what I mean on this challenge that I tackled on the rest api

  Just those two above. This is really great. Good work.

  Marked as helpful

  0