Needs work
Project:
Responsive Menus
Version:
7.x-1.5
Component:
MeanMenu Style
Priority:
Critical
Category:
Support request
Assigned:
Reporter:
Created:
8 Jan 2015 at 18:32 UTC
Updated:
12 Jul 2016 at 15:41 UTC
Jump to comment: Most recent
Comments
Comment #1
daflava1987 commentedInstead 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.
Comment #2
Ritch commentedHi,
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.
Comment #3
slipit commentedI 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.
Comment #4
lamp5Hi,
Download the latest version and copy to responsive_menus/styles/meanMenu
https://github.com/meanthemes/meanMenu
I hope this helps.
Comment #5
earthangelconsulting commentedre #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!
Comment #6
yosia_ken commented#4 work for me, thanks @lamp5!
Comment #7
lamp5Comment #8
jwjoshuawalker commentedThe 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.