Responsive page with navigation toggle made with CSS Grid and JS
Design comparison
Solution retrospective
The navigation menu toggle was fun to make. Using the aria-expanded attribute I was able to work in a ternary expression and use classList.toggle()
. My grid layouts are still a little wonky but I am getting more comfortable and faster at implementing CSS Grid.
My overlay that shadows out the body content when the expanded navigation menu is open only covers the header section. I wondered if there is a way to freeze the rest of the page from scrolling when the menu is showing.
Let me know what you think of my solution. All and any feedback is welcome!
Community feedback
- @IrbainePosted over 1 year ago
Great job Amy, i like your solution and everything seems in place and the code is very clean and readable. Congratulations ππ
0
Please log in to post a comment
Log in with GitHubJoin 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