I got a weird issue ..

When I open the Website on a tablet in portrait mode - i see the meanmenu. If i switch from portrait to landscape, also meanmenu (sometimes i wonder why becaue of width, sometimes its ok). So when I switch back (portrait->landscape->portrait) the meanmenu is gone - like for real gone, its even gone in the html code...

very strange - been looking and searching for solutions for a long time, nothing helped me .. or I am just really stupid and wasted hours for nothing

hope you can help me

Comments

daflava1987’s picture

Instead of starting a new support request. I'll elaborate more on this issue. Using a tablet on portrait view(on page load) Mean Menu works just as designed. However, when switching to landscape view and clicking the menu toggle no navigation links are shown.

However, thats not the important issue here. This is: when reverting back to portrait view from landscape, mean menu is not applied. I see my menus hidden in the background of other elements and divs, but even my main navigation classes are ignored, most likely due to mean menu still being active but not rendering.

This is a bug. I thought perhaps it was due to a css or loading error but the same issue can be documented on your example pages as seen here. http://drastikbydesign.com/demo/responsive-menus-demo-1-mean-menu

From what I can see, this issue is only related to tablets or devices with larger screens. Most devices with smaller viewports don't seem to be affected.

Ritch’s picture

Hi,
I had the same problem.
So I put setTimout 500 in window.onorientationchange = function () in jquery.meanmenu.min.js. It's work for me.
I hope this helps.

slipit’s picture

I have the same problem. When Mean Menus style is used and viewed in a desktop size page the menu disappears. If the window is sized down to mobile size the toggle menu is intact. This is with the Zen theme and a menu setup in Admin/Structures/Menus.

When the page is reverted back to the desktop size it appears. If the page is immediately refreshed after coming from the mobile size it disappears.

lamp5’s picture

Hi,
Download the latest version and copy to responsive_menus/styles/meanMenu

https://github.com/meanthemes/meanMenu

I hope this helps.

earthangelconsulting’s picture

re #4 ... thanks @lamp5, that worked! much appreciate that.

attention maintainer: i noticed that the MeanMenu source has been slightly modified for a Drupal version? but i am not sure exactly why/when the mods are necessary, because as at first glance it seemed to work without them? i couldn't figure out how to download MeanMenu 2.0.6 so that i could run a diff against the "drupal-modded" 2.0.6 version that's in the latest dev release of responsive_menus, otherwise it would have been easy to see exactly what the mods were.

as i it was, i had to go by the comments that were in your modded version, and i THINK i have successfully moved them into the 2.0.8 version, and everything seems to be is working... but it would be great if we could get some clarity here on exactly what the mods are and why they are necessary (so if necessary we can make the same fix ourselves, whenever a new version of MeanMenu comes out).

overall though: this module ROCKS and has saved my butt!

yosia_ken’s picture

#4 work for me, thanks @lamp5!

lamp5’s picture

Status: Active » Needs review
jwjoshuawalker’s picture

Status: Needs review » Needs work

The modifications to MeanMenu library were to bypass core's Contextual Links menu.
It can interfere with MeanMenu if the CL module is active, and you have the permission to see Contextual Links.

It could be solved another way, in the "glue" js file added by RM for MeanMenu integration.
We'll have to do something a bit more invasive, but this will allow us to use the MeanMenu library directly from source.
A few options (legend: Contextual Links = CL):
- Remove the CL menu added to the real menu being targeted. (remove DOM element completely)
- Nest the CL menu under another parent element.
- Convert (and CL menu children) it to a different element type (div, dd/tt, etc), since MeanMenu library looks for <ul> specifically.