You might consider abandoning the primary links construct and just create a custom region for your theme and put the primary links block in that region. If the "menu-block" gives you what you wan't that might work. You then disable primary and secondary links in the menu.
That being said. It sounds like your problems are that your javascript is doing a a lot of DOM walking specific stuff (otherwise why would extra divs be a problem?). Is that accurate? This is part of what JQUERY was written for. In JQUERY you can bind to on-click events using typical CSS selectors. That rocks for this kind of work (even if I don't do a ton of it).
I realize that you have some already tested code. But if it's highly specified to the organization of the document it could cause you a lot of problems in the long run, so I thought you'd want to know about JQuery.
Hope some of this helps anyway.
Dave
________________________________
From: support-bounces@drupal.org [mailto:support-bounces@drupal.org] On Behalf Of sander-martijn Sent: Thursday, August 16, 2007 5:15 PM To: support@drupal.org Subject: Re: [support] Menu question
Making some progress here. I've realized that to some extent what I want to do is override theme_menu_tree and theme_menu_item. That gives me some of the control I need BUT the default menu system still has extra junk I don't want (such as a Primary links header) AND I need two types of menu trees that display different things - one horizontal in the top (primary links) and one vertical down the left (current section and its children).
so far i think menus are a pain in the ass in drupal. a tradeoff for the power but I think the multiple meanings that the word "menu" has in drupal just adds to the confusion of it all. What I need is really not that complicated, but it's becoming incredibly complicated fast.
sander-martijn wrote:
I want to customize my menus, but since I want it to work with the html/css/javascript code I've already written and tested I want to get the system to output the menu in html as I want it. Actually what I need is quite simple. I don't really want to try to customize someone else's module and I'd rather not build one. After some digging around I figured out that I could override the theme_menu_links method in my template.php file. Now that's exactly what I need... I got really excited but I must have some things missing/misunderstood. what I have in the menu system under primary links: primary links - menu item 1 - menu 1 subitem 1 - menu 1 subitem 2 - menu item 2 - menu 2 subitem 1 etc - fairly standard What I need to output is the following: <ul id="nav1" class="nav"> <li><a href="#">menu item 1</a> <ul> <li><a href="#">menu 1 subitem 1</a></li> <li><a href="#">menu 1 subitem 2</a></li> </ul> </li> </ul> <ul id="nav2" class="nav"> <li>menu item 2 <ul> <li><a href="#">menu 2 subitem 1</a></li> </ul> </li> </ul> etc. - also pretty straight forward. SO - I put in page.tpl.php the following: <?php print theme('menu_links', $primary_links); ?> and put in template.php the following: function tpg_menu_links($links){ if (!count($links)) { return ''; } $level_tmp = explode('-', key($links)); $level = $level_tmp[0]; $output = "<ul id="nav" class="nav">\n"; foreach ($links as $index => $link) { $output .= "<li>". l($link['title'], $link['href'], $link['attributes'], $link['query'], $link['fragment']) ."</li>\n"; } $output .= '</ul>'; return $output; } ?> Which is basically a modified version of theme_menu_links in menu.inc It works as a start, but there are a couple of issues. 1. even though my class attribute in $output is hard coded, it's still being replaced by class="active" when you're on the page. Not a
disaster, i can always modify my css to do the same thing for class="active" as class="nav". 2. this is the bigger issue. It's not outputting the subitems. I'd be happy to add in the proper call in the foreach loop to either call another function that i also override or to load them directly in here if anyone can point me in the direction of what i need to call in order to load them.