Primary Links

Yup. Same issue observed.

Oops. Looks like it's coming from a padding property in system-menus.css. My quick testing leads me to believe that adding the following code to style.css will resolve this issue:

#primary li.leaf {
  padding: 0;
}

Any chance one of you can confirm this? I'll update the repository as soon as I get some type of confirmation from someone seeing the problem.

Confirmed. Adding that code to the end of the style.css file corrected the error for me in firefox, chrome and IE.

Thanks for the feedback.

Not sure what to tell you, hopfrog... this css code fixes things for me on all browsers, and it seems to have fixed things for other users. Are you sure you've cleared your browser cache after changing the CSS code? Is it possible to post a link to your site?

Hey, you switched themes on me before I could take a look! :)

It seems that because I am an anonymous user, I only see the Home link in the primary links menu. With only one menu item, I cannot see the problem.

Looks like one more CSS definition is in order:

#primary li.collapsed {
  padding:0;
}

I had the exact issue and adding both definitions resolved the issue.

I can confirm this fix as well. Thanks!

I just ran into this problem, even after adding

#primary li.leaf, #primary li.collapsed { padding: 0 }

to styles.css.

After comparing 6.x-1.7 LiteJazz to Drupal 6.8's Garland theme, I changed

    <?php if ($primary_links): ?>
    <div id="primary" class="clear-block"> <?php print theme('menu_tree', menu_tree('primary-links')); ?> </div>
    <?php endif; ?>
    <?php if ($secondary_links): ?>
    <div id="secondary" class="clear-block"> <?php print theme('menu_links', $secondary_links); ?> </div>
    <?php endif; ?>

to

  <?php if (isset($primary_links)) : ?>
  <div id="primary" class="clear-block"> <?php print theme('links', $primary_links, array('class' => 'primary-links')) ?></div>
  <?php endif; ?>
  <?php if (isset($secondary_links)) : ?>
  <div id="secondary" class="clear-block"><?php print theme('links', $secondary_links, array('class' => 'secondary-links')) ?></div>
  <?php endif; ?>

in page.tpl.php.

I had the primary links assigned as the source for both the primary and secondary navigation.

Without this change,

• The primary links were still including the sub-menus
• The secondary links were empty

After the change, these two problems were fixed. I can't guarantee that this change won't break some other option—I'm not sure that the HTML generated by the new code is exactly the same as the HTML generated by the old code.

I applied the fixes mentioned in #2 and #11 and it seems to work great now in FF and IE 6 and 7. I am not using suckerfish menus and have no secondary links.

Thanks!

Hmmm, stranegly enough, this doesn't work for me - added both css snippets, uploaded, refreshed; nothing. Does not work in IE 8 and FF 3.5, eiter.

Menu system *would* work if I was using primary links within suckerfish area; however, I do NOT want to use suckerfish menus for themeing reasons and rather keep the "regular" primary links.

Any idea? I attached a screenshot (above red line is unexpanded, below when expanded- it breaks after "Schritt-für-Schritt Anleitungen").

Thanks!
R.