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