Looking to hire developers?
Used grid and sass for the 1st time.
@artimys
Posted
Nice job Bazz,
I recommend adding a max-width to your main#wrapper so it doesn't expand to far on wider screens.
And add a hover and `focus state on your toggle button to help with accessibility
Keep it up 👍👍
@Basit-Dev
Posted
@artimys thanks for the feedback ! makes sense with the wrapper and toggle button.
@ApplePieGiraffe
Posted
Yeah, nice job, Bazz!
Kudos for trying out CSS grid and Sass! 👏 (I think they make CSS a lot easier and more fun. 😆)
Your light/dark themes work well and your solution responds nicely. Just follow artimys' suggestions and you should be good to go! 👍
Keep coding (and happy coding, too)! 😁
@Basit-Dev
Posted
@ApplePieGiraffe think you 🙂👍
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
