News Homepage HTML/CSS/JS

Community feedback

• 

  Ivan Mirković•180

  @imd94Posted over 1 year ago

  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

  ---

  

  Jordan KRARIA•270

  @JOJOK63Posted over 1 year ago

  @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