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.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Anonymous’s picture

I'll have to look into that.... I suspect it might be a z-indexing thing.

sittard’s picture

Adding this CSS to my custom-skin.css fixed the issue for me:

.block-megamenu {
  z-index:10;
}

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.

Aren Cambre’s picture

Status: Active » Needs review
klonos’s picture

I have enabled megamenu for the admin menu, but I see no effect whatsoever (I flashed caches and all). What might I be doing wrong?

klonos’s picture

...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?

Anonymous’s picture

FileSize
32.08 KB

The output looks like this.

klonos’s picture

Thanks 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?

grimfandango’s picture

This screen shot shows the Primary Links menu Interface with the levels and the resulting menu

http://drupal.org/files/issues/grimfan-megamenu-scnshot.jpg

grimfandango’s picture

BTW, 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.

klonos’s picture

Title: Compatability with Adminstration Menu » Compatibility with Adminstration Menu
FileSize
24.03 KB

Thank 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 ;)

grimfandango’s picture

FileSize
159.99 KB

I 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.

klonos’s picture

Interesting 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)?

grimfandango’s picture

Drupal 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

Anonymous’s picture

@grimfandango, what's the issue exactly?

grimfandango’s picture

Some 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).

Anonymous’s picture

The 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.

klonos’s picture

I 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.

Anonymous’s picture

Status: Needs review » Closed (duplicate)
klonos’s picture

Status: Closed (duplicate) » Needs review

@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)

Anonymous’s picture

That 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.

klonos’s picture

I 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 ;)

klonos’s picture

Title: Compatibility with Adminstration Menu » Compatibility with Adminsitration Menu

...missed an 'i' there ;)

Aren Cambre’s picture

Title: Compatibility with Adminsitration Menu » Compatibility with Administration Menu

Corrected spelling.

klonos’s picture

yea! I put it in the wrong place ;) thanx

Anonymous’s picture

Added z-index of 10 to megamenu in #410510

Grayside’s picture

Status: Needs review » Needs work

The 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 ;)

Anonymous’s picture

Status: Needs work » Closed (won't fix)

I 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 :-)

Grayside’s picture

Status: Closed (won't fix) » Needs work
FileSize
68.27 KB

@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?

Anonymous’s picture

Title: Compatibility with Administration Menu » Interferers with Administration Menu--cannot click on admin links on top of a mega menu

@grayside You're right. I'm changing the name of this issue so it doesn't confuse me :-)

willvincent’s picture

This 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.

Anonymous’s picture

Assigned: Unassigned » silverfly
anacronaut’s picture

the 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:

#admin-menu, #admin-menu ul {
    z-index: 10000 !important;
}
sittard’s picture

Simplest fix is to change the z-index value of .megamenu-menu .megamenu-parent to 999. That worked for me.

.megamenu-menu .megamenu-parent {
	display: inline;
	position: relative;
	z-index: 999;
}