Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

News Homepage HTML/CSS/JS

@JOJOK63

Desktop design screenshot for the News homepage coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
2junior
View challenge

Design comparison


SolutionDesign

Solution retrospective


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

Community feedback

@imd94

Posted

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

0

@JOJOK63

Posted

@imd94

Hi Ivan,

Thank you for taking time to answer to me.

I try what you say. And it's ok i don't have the scroll bar on the right side.It's clearly better.

thankfull

0

Please log in to post a comment

Log in with GitHub
Discord logo

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