Responsive News Homepage
Solution retrospective
It was nice to do DOM manipulation using JS. This challenge tested my planning of the layout. I tried to abide by the DRY principle while writing the CSS for this page.
Any feedback would be welcome!
Please log in to post a comment
Log in with GitHubCommunity feedback
- @Karrar-Hussein
Beautiful job, one thing to take notice of is that instead of changing the body element background color when opening the hamburger menu you could get a better shade by creating an overlay "empty div" with position fixed and size it so that it covers the whole screen and give it a black background color with an alpha value "smth like 0.5 👌" and set its display property to "none", then you can attach an event listener to toggle the overlay display property whenever you click the nav button.
also, don't forget to give the sidebar a z-index higher than the overlay 😊
Marked as helpful - @Katleho-codes
This is very good! Keep it up!
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