Menu options very sensitive avoid scroll this menu

[xzegga]

I have problem in mobile device when I try to scroll this menu if ammount of menu options not fit on all height of the screen, Apears that the menu is very sensitive, when I try to scroll the options are activated and avoid to scroll the menu to view all options bellow.

Best regads

[FizzyGiant]

I've been having the same issue, I did you manage to find a fix?

[xzegga]

Yes, I resolve this issue, I don't remember at this time how to resolve but I will check my old projects to get solution for you.

[FizzyGiant]

I ended up getting around the issue by using Hammer.js and rewriting the javascript logic code to use functions provided by hammer.js instead.

I'd still be interested to see how you fixed it without using an external lib though, but don't worry if you can't find it :)

[TonyShark19]

ev.preventDefault(); is preventing scrolling in the mlpushmenu.js

working on a fix for this....

[TonyShark19]

change line 136 to always be 'click'
el.querySelector( 'a' ).addEventListener('click', function( ev ) {

this was hanging up the menu on new page load so i added this to line 174
//initialize menu closed
self._resetMenu();

[jtphelan]

Changing line 136 worked for me, but adding self._resetMenu(); to line 174 gave me a "not a function error" in the console. Seems to work without it though. Not sure if the line numbers are off now, some context would help to know if I placed it in the right place.

[TonyShark19]

Glad the first part worked for you! I'm new to JS, so that's a win that it worked for both of us.
Here is the resetMenu() command in context. I added this because sometimes on page load the menu would load already opened.

[jtphelan]

That works, thanks. I had it a couple lines below under _openMenu : function( subLevel ) which did not work.

[metew]

My solution/workaround.

.min file
{L="touchend";g="touchmove"} -> {L="click";g="touchmove"}

or if you want to read the full non min version; this is the offending line for scrolled content.
Replace the 'touchend' with 'click'
.js file (starts line 1109 for me)
if( mobileCheck() ) {
clickEventType = 'touchend';
dragEventType = 'touchmove';
}