codrops / MultiLevelPushMenu

An experimental push menu with multi-level functionality that allows endless nesting of navigation elements.
743 stars 201 forks source link

page scrolling isn't working with this menu? #17

Open NJWebDesign opened 9 years ago

NJWebDesign commented 9 years ago

I've been experimenting with this menu and it looks really good, and is a nice way to offer navigation on a site, however something I've done, I assume, has stopped page scrolling working: http://www.cairnsmorecottages.co.uk/

Is this perhaps a bug/issue or maybe someone can spot the obvious mistake.

Appreciate it. Thank you. NJ

botelho commented 9 years ago

You have .scroller { overflow-y: none; } when it should be .scroller { overflow-y: scroll; } in component.css line 13. I think it might be that.

BTW, maybe you've realized that already but if not, the staggering on scroll is because of the box shadow of #topbackground (styles.css line 51).

Cheers.

NJWebDesign commented 9 years ago

Thank you botelho. That resolves it on most devices but it still won't scroll in landscape view on a Windows Phone. Appreciate the help and advice. Thank you. NJ

NJWebDesign commented 9 years ago

Why doesn't the scroll bar work in this scenario? If I roll my mouse wheel, or scroll (swish) my finger, the page scrolls but if I try to manually click the scroll bar nothing happens?

botelho commented 9 years ago

I can't really help you with this, I didn't have the chance to test this on a Windows Phone. I'll leave this issue open in case somebody did and has a fix for it.