The megamenu is obstructing the top section
I have had to push down the following divs. A better solution should be devised.

CommentFileSizeAuthor
#14 diff.txt5.95 KBjoergent
#10 Captured002.png196.37 KBjoergent

Comments

lyd’s picture

lyd commented

Marinelli is great, however, the primary menu issue is a real problem.

I wish we could have the former dropdown menu option as well. The megamenu does not work well at all at this time.

Lioz’s picture

Lioz commented
Status: Active » Postponed (maintainer needs more info)

@joergent: i don't see any overlapping in your site. The behaviour seems correct to me.
menu items open on the right if you don't set any alt class, see http://drupal.org/node/1000286

@lyd: which primary menu issue do you have?

joergent’s picture

joergent commented
Status: Postponed (maintainer needs more info) » Active

You don't see any overlapping because I fixed it by changing the css to lower it by 40 pixels I think !

joergent’s picture

joergent commented

In the link in above response there is a reference to main-menu which does not exist. I had to change it to primary-links

diff -rbB ./template.php /home/rainbow-vikings/public_html/sites/all/themes/marinelli/template.php
104c104
< $vars['mainmenu'] = theme('mega_menu', array('menu' => menu_tree_all_data('main-menu')));
---
> $vars['mainmenu'] = theme('mega_menu', array('menu' => menu_tree_all_data('primary-links')));

In page.tpl.php I have among other things changed

<     <div id="navigation-primary" class="sitemenu">
---
>     <div id="navigation-primary" class="sitemenu" style="position:relative; top:40px;" >

and
<   <div id="pageBorder" <?php print $noborder; ?>>
---
> <div id="pageBorder" style="position:relative; top:40px;" <?php print $noborder; ?>>
Lioz’s picture

Lioz commented
Status: Active » Closed (works as designed)

@joergent you modified the standard banner layout so i don't think i can give any support here...in my online demo with the latest beta version i don't see any overlapping (tested it with major browsers/OS)

Furthermore in the latest beta version you can choose any menu as the megamenu source see this code in template.php

    $vars['mainmenu'] = theme('mega_menu', array('menu' => menu_tree_all_data(theme_get_setting('menu_element'))));
joergent’s picture

joergent commented

That is too easy. The overlapping is taking place before any modifications are made.

joergent’s picture

joergent commented

The logo image used is actually enlarged (compared to the size displayed in an image tool program) when displayed and the menu tabs are in front of it.

joergent’s picture

joergent commented
Version: 7.x-3.0-beta6 » 7.x-3.0-beta7
Status: Closed (works as designed) » Needs work
joergent’s picture

joergent commented
Status: Needs work » Active
joergent’s picture

joergent commented
StatusFileSize
new Captured002.png196.37 KB

Screendump illustrating the problem

Lioz’s picture

Lioz commented

sorry joergent but i don't see any problem looking at your screenshot.
The logo is not supposed to stay over the main menu

joergent’s picture

joergent commented

Well, then it is a change request. This looks awful. A menu should not have such a prominent display.
Also, I would request the logo not to be enlarged. It makes the display of it less crisp.

Lioz’s picture

Lioz commented
Category: bug » feature
joergent’s picture

joergent commented
StatusFileSize
new diff.txt5.95 KB

I have made the needed changes. Now optionally the banner can be placed in the upper right of the top section and the menu is pushed down, so it does not obscure the top section. The real tricky part was modifying the CSS img.slide on the fly. The sliding never really worked using another name.

I am uploading a diff with the changes, which can be used for verification and further development.