3rd level mobile menu issue

  • Vmwp

    (@vmwp)


    7 years, 3 months ago

    Hi,

    In mobile view, I’m noticing that with a three level menu, if I don’t close the 3rd level, but close the 1st level, I can no longer highlight any 1st level items that the 3rd level menu had displayed over.

    However, if I drill down to the 3rd level and close that menu, then close the 2nd menu, and then the first menu, then everything on the 1st level highlights correctly.

    The solution/workaround I have found for now was changing line 195 in the maxmegamenu.js from
    plugin.hidePanel($(this), false);
    to
    plugin.hideAllPanels();

    Edit:
    However, should only do hideAllPanels() when clicking the top level menu item. The above change also closes everything if clicking on the 3rd or 2nd level menu, which, yeah, isn’t quite right.

    Edit 2:
    changing ln 195 to:
    plugin.hideSiblingPanels($(this));

    seems like it may function properly.

    If this seems correct, just wondering if it could make sense to merge the change into maxmegamenu.js? Otherwise, I dont know if something could be added to a theme, external to the plugin, to achieve similar behavior?

    Thanks!

    • This topic was modified 7 years, 3 months ago by Vmwp.
    • This topic was modified 7 years, 3 months ago by Vmwp.
    • This topic was modified 7 years, 3 months ago by Vmwp.
Viewing 3 replies - 1 through 3 (of 3 total)
  • Plugin Author megamenu

    (@megamenu)

    7 years, 3 months ago

    Hi vmwp,

    Thanks for reporting this and for providing a detailed explanation. I think I understand what you’re saying, but so far I haven’t been able to replicate the same behaviour here.

    Please could you post a screencast which shows the problem? (Just so I know I’m looking at the same thing)

    Please can you also confirm which settings you have for the following:
    Under Appearance > Menus, the “Event” setting for the menu
    Under Mega Menu > General Settings, the “Mobile Menu Behaviour” (standard or accordion).

    Thanks,
    Tom

    Thread Starter Vmwp

    (@vmwp)

    7 years, 3 months ago

    Hi Tom,
    Thank you for the reply and checking into it. At this point I’m thinking it’s a theme issue (CSS). Sorry, I should have looked into it a bit further first. I tested the settings w/ a clean install w/ the twentysixteen theme and everything seems to function properly. I was seeing behavior where the 2nd/3rd level mobile menu was overlaying over the top level menu content, rather than pushing it below (which I saw w/ the clean install). Given the overlay behavior, I can see why closing the children menus w/ js helped the situation, but clearly something external was interfering. I appreciate your time and thanks for developing a great responsive menu plugin.

    Plugin Author megamenu

    (@megamenu)

    7 years, 3 months ago

    Hi vmwp,

    Thanks for the update, makes sense 🙂

    Glad you like the plugin.

    Regards,
    Tom

Viewing 3 replies - 1 through 3 (of 3 total)
  • The topic ‘3rd level mobile menu issue’ is closed to new replies.