Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
First off, this shaping out to be a great module.
I'm having a few compatibility issues with the administration menu module (http://drupal.org/project/admin_menu). The admin menu drop downs appear and function correctly but it's impossible to select menu items when the two menu's (admin and mega) collide. I guess it's probably a CSS issue but I've not had time to look into this further.
Thanks.
Comment | File | Size | Author |
---|---|---|---|
#28 | megamenu.admin-conflict.png | 68.27 KB | Grayside |
#15 | megamenu-scnshot-menuissue-grimf.jpg | 128.76 KB | grimfandango |
#11 | features-mega-menu.jpg | 159.99 KB | grimfandango |
#10 | admin-mega-menu.png | 24.03 KB | klonos |
#6 | megamenu_output.jpg | 32.08 KB | Anonymous (not verified) |
Comments
Comment #1
Anonymous (not verified) CreditAttribution: Anonymous commentedI'll have to look into that.... I suspect it might be a z-indexing thing.
Comment #2
sittard CreditAttribution: sittard commentedAdding this CSS to my custom-skin.css fixed the issue for me:
I had to apply the z-index at the block level (.block-megamenu) as I'd given this a fixed height and background color to improve the appearance of my menu.
Any news on the next dev release?
Thanks.
Comment #3
Aren Cambre CreditAttribution: Aren Cambre commentedComment #4
klonosI have enabled megamenu for the admin menu, but I see no effect whatsoever (I flashed caches and all). What might I be doing wrong?
Comment #5
klonos...I am guessing that there should be 2nd level menus as 'slots' with 3rd level menus placed under them. Right?
Could anyone that had success with it provide a screenshot of what I should see please?
Comment #6
Anonymous (not verified) CreditAttribution: Anonymous commentedThe output looks like this.
Comment #7
klonosThanks for taking the time to post the screenie Andrew, but is this the default install of Admin menu? I am asking because I was expecting to see some of the 1st, 2nd and 3rd levels of that module that I am used to. Or perhaps you've added some custom items ('lvl1', 'lvl2' and the 't3 - x' children) for testing purposes.
Also, is this a customized css alteration of the default Admin menu theme I see there?
Comment #8
grimfandango CreditAttribution: grimfandango commentedThis screen shot shows the Primary Links menu Interface with the levels and the resulting menu
http://drupal.org/files/issues/grimfan-megamenu-scnshot.jpg
Comment #9
grimfandango CreditAttribution: grimfandango commentedBTW, I don't think that this module is meant to replace the Admin Menu with a Mega Menu.
It is primarily meant to replace the Primary and Secondary Links Menus.
Comment #10
klonosThank you John for taking the time to provide a second screenshot. Things are clear to me now. I realize that this shows a custom menu rendered as a Mega menu (I had already thought so).
My confusion here has to do with:
a) the mention of Admin menu in #0.
b) the issue's title, since I misinterpreted 'Compatibility' (was spelled wrong btw) with 'Support'.
c) the mention of an 'Administration menu' in the module's settings page (please see attached screenshot)
Should have noticed that this is a 'bug report'. Here's a 'feature request' then: #869030: Support for the admin_menu module - a.k.a. convert admin_menu to a mega menu ;)
Comment #11
grimfandango CreditAttribution: grimfandango commentedI was going to install the admin menu to my demo install and then look at mega menu. Interestly, I don't get the Administration Menu Item. I get a Features item.
Comment #12
klonosInteresting indeed!
Do you see this in d6.17? Are you using latest devs of modules? What menus do you see in your menus list (.../admin/build/menu)?
Comment #13
grimfandango CreditAttribution: grimfandango commentedDrupal core 6.17
Administration Menu 6.x-1.5 (I am not using the latest alpha). I installed 6.x-3.0-alpha4 and so now I also get Administration Menu listed in my Menu List and in the Mega Menu interface.
Menu List
Administration menu
Features (Menu items for any enabled features)
Navigation
Primary links
Seconadary links
Comment #14
Anonymous (not verified) CreditAttribution: Anonymous commented@grimfandango, what's the issue exactly?
Comment #15
grimfandango CreditAttribution: grimfandango commentedSome modules with "menu" support like, Administration Menu (some versions) and Features appear in the megamenu configuration menu, see screenshot.
These menus however are not supported by Mega Menu (to my knowledge) therefore are confusing to users. The only options that should be presented are supported menus (currently Primary Links and Secondary Links).
Comment #16
Anonymous (not verified) CreditAttribution: Anonymous commentedThe problem with limiting it to primary and secondary links, is that this module is supposed to megatize any user created menu as well. Maybe we can exclude administrative menus.
Comment #17
klonosI agree with you Andrew on excluding admin menus (or any menu known to have issues), but only once we have a 'stable' release.
I think for dev versions this exclusion needs to be omitted. The community needs to be able to test with ease and report back any regressions. Perhaps a 2.x dev branch could hold the extra soon-to-be-supported menus, but it is silly to talk of that since we didn't yet reach 'stable'-ish stage in 1.x (not even a 'unsupported' 1.x alpha). So, I guess a 'Only for testing - not supported yet!' help text under each menu entry in the list of menus would suffice for now.
Comment #18
Anonymous (not verified) CreditAttribution: Anonymous commentedDuplicate of #869030: Support for the admin_menu module - a.k.a. convert admin_menu to a mega menu
Comment #19
klonos@FilmKnurd: Andrew, please take a look again at the description of that issue. It is a feature request, suggesting that this module should convert Admin menu to a Mega menu, whereas this one is a bug report dealing with any issues that might come up when both modules are installed.
Anyways, if any of the two should be marked as duplicate it should be the other one for two reasons:
- this one has a patch available (...well kinda patch, at post #2) that deals with the issue being reported
- the other is newer (this one was started first)
Comment #20
Anonymous (not verified) CreditAttribution: Anonymous commentedThat makes sense. This thread became confusing because it morphed into a feature request :-) What should we do about the original issue? I suppose we could add a z-index rule to the default css.
Comment #21
klonosI agree with you. Perhaps some of the posts here actually belong to the other issue and that does confuse things.
Once this is fixed (by adding the css rule), then I guess there will be no issue and all admin_menu feature-request-related posts will go to the other one. Well... not until any new bug-related issue might come up and this is re-opened ;)
Comment #22
klonos...missed an 'i' there ;)
Comment #23
Aren Cambre CreditAttribution: Aren Cambre commentedCorrected spelling.
Comment #24
klonosyea! I put it in the wrong place ;) thanx
Comment #25
Anonymous (not verified) CreditAttribution: Anonymous commentedAdded z-index of 10 to megamenu in #410510
Comment #26
Grayside CreditAttribution: Grayside commentedThe version of Megamenu I checked out the other day (I couldn't get at anything recent without pulling Head from CVS*) does not have this Fix in it. Marking as Needs Work.
* P.S., Please clean up CVS or create a stable release ;)
Comment #27
Anonymous (not verified) CreditAttribution: Anonymous commentedI just want to clarify that it is not the goal of this module to "support" other menu modules. This module is for themers that need a particular markup for a three tiered menu tree and some basic JS to achieve the mega menu behavior. It is the responsibility of the themer to structure and style their mega menu.
These issues are a useful place to identify CSS conflicts, but that isn't a guarantee that every CSS tweak will end up in the default example styles.
@grayside
Yeah, much clean up is in order. We have a couple of patches waiting to be integrated and tested, but we are getting closer to a stable release. Stay tuned :-)
Comment #28
Grayside CreditAttribution: Grayside commented@FilmKnurd: The zany Feature Request that takes up most of this issue is actually a hijacking. The problem here is a bug report. If you reread the original post it describes the problem I am seeing.
Attached is a screenshot of the out-of-the-box behavior you get when using MegaMenu and Admin Menu. This is what happens as you mouse down along the admin menu. A split second after the cursor floats above the megamenu, the admin menu closes and you are simply on the megamenu.
#2 was addressing this problem.
You say you committed a Fix, but I could not find the fix in CVS. Therefore I have to assume a problem ;)
Unless you can point me to the right tag?
Comment #29
Anonymous (not verified) CreditAttribution: Anonymous commented@grayside You're right. I'm changing the name of this issue so it doesn't confuse me :-)
Comment #30
willvincent CreditAttribution: willvincent commentedThis is definitely a z-index issue with the included skins. It's also present in the 7.x-1.x version as well. I presume, as it's css related that by using a custom skin rather than one of the included ones it will resolve itself, but it seems to me the example skins that come with the module should be updated to fix this issue so that people creating new skins won't run into this problem and then be confused what is causing it.
Comment #31
Anonymous (not verified) CreditAttribution: Anonymous commentedComment #32
anacronaut CreditAttribution: anacronaut commentedthe megamenu.css sets the z-index of .megamenu-menu .megamenu-parent to 5000 and the z-index of .megamenu-menu .megamenu-bin to 10000. The admin menu has a default z-index of 999. You can force the admin menu to appear over your mega menu by overriding it in your theme's css:
Comment #33
sittard CreditAttribution: sittard commentedSimplest fix is to change the z-index value of .megamenu-menu .megamenu-parent to 999. That worked for me.