Skip to content
Submitted about 2 years ago

News Homepage HTML/CSS/JS

Jordan KRARIA•270
@JOJOK63
A solution to the News homepage challenge
Preview (opens in new tab)Code (opens in new tab)

Solution retrospective

Hi all, give me some advice about my code, to step up please. Thank

Code

Please log in to post a comment

Log in

Community feedback

  • Ivan Mirković•220
    @imd94

    Hi Jordan,

    Well done on completing the challenge.

    One small improvement, though, that you could make is to set overflow-y: hidden on the body element when toggling the mobile nav to prevent vertical scrolling while the mobile nav is open. Right now, when you toggle the mobile nav menu, you can scroll down, and your overlay doesn't cover the entire page.

    Hope this suggestion helps.

    Marked as helpful

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