Skip to content
Submitted over 4 years ago

Dark Mode Nativo con CSS Grid, Flexbox y Js

José Hidalgo•105
@ImJoseHidalgo
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Social media dashboard with theme switcher
  • HTML
  • CSS
  • JS
2junior

Design comparison

SolutionDesign

Solution retrospective

Did you find any errors? Is there something you don't like? Tell me.

Community feedback

  • Hari Ram•370
    @hariramjp777Posted over 4 years ago

    Hi Brother, It's a good solution and it's responsive. I recommend the following,

    Add hover effects for cards.

    Second title should be white in dark theme and black in light theme. (Overview - Today)

    section and article must need a heading inside to pass html validation. So, I recommend using div for containers here in this challenge since it'll reduce your html issues.

    All the best, brother..

    1
    José Hidalgo•105
    @ImJoseHidalgoPosted over 4 years ago

    @hariramjp777 thank you. I will take into account everything you say 😀

    1

Please log in to post a comment

Log in with GitHub

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