The simple menu disappears when the Zen theme is activated.

Steps to reproduce:
1) install simplemenu module and configure to run on Garland Theme
2) install Zen theme and activate the theme

Expected behavior:
Simplemenu will appear in the Zen theme

Actual Behavior:
Simplemenu does not appear

CommentFileSizeAuthor
#3 fix-firefox-with-zen-theme.patch313 bytesjsaints

Comments

jsaints’s picture

jsaints commented

Update...

I notice that the menu show is if you use the "original" theme for simplemenu. It disappears with the "black and blue" theme.

Even though the original theme shows, the menu does not display correctly in the Zen theme. It does not clear the header. I am using Firefox 2

jsaints’s picture

jsaints commented

I have confirmed that this is only a problem in Firefox. If I add the .clear-block class to the ul#simplemenu item, then things are fixed across multiple browsers.

I am not sure this is a good fix because it relies on default.css which may or may not be present in contrib themes.

jsaints’s picture

jsaints commented
StatusFileSize
new fix-firefox-with-zen-theme.patch313 bytes

here is a patch for the fix

monjohn’s picture

monjohn commented

I am not getting the menu to show up in Firefox at all. It doesn't matter the themes I use and I have inserted the line from the patch into the .js file and I still see nothing. Since the menu is not printing, I can't use firebug to figure out what is going wrong.

chrisschaub’s picture

chrisschaub commented
Status: Active » Needs review

This patch in #3 is now working for me as well. I think most themes are using default.css now? Please commit since this bug will break the menu for many.

zroger’s picture

zroger commented

Committed, slightly modified. Added the 'clear-block' class on line 11, but same result.

@jsaints: I think it is fine to use the clear-block class, since a lot of core modules use it as well. And if you know enough to exclude defaults.css from your theme, then you should understand that there are certain things that you will lose.

As part of this commit, I reverted a change made in #245656. float:left was added to the simplemenu element to make sure the <ul> background clears the <li>s. Using clear-block fixes both of these problems, but float:left doesn't work with Zen because of its use over overflow-y: hidden.

zroger’s picture

zroger commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.

madhudvs’s picture

madhudvs commented

can someone port the patch for the 7.x-1.x-dev version ?
the beviour of the problwm is same ....
am using the Zen theme and when i choose "blue and black " for menu ... it disappers and is wont disapper for "orginal " style !
please help me out ...
thanks

AlexisWilke’s picture

AlexisWilke commented

This was added the 6.x in 2008. The 7.x version was created around 2010. So if you have a similar problem, that's new/different, I think.

Thank you
Alexis Wilke