Hey Ted -

Thanks for the new version of SimpleMenu :)
I noticed a little bug that I wanted to let you know about. I had the following, seemingly normal line in my CSS, however it caused a relatively large vertical gap to appear between the second and third levels of the menu (e.g. between Content Management -> Categories...):

p, ul, ol, dl {
	font-size: 1.1em;
	line-height: 1.5em;
	padding: 0;
}

I added the following code to my CSS and it fixed the problem. You might want to consider adding something along these lines to "reset" any potential user/theme CSS and ensure that SimpleMenu always displays correctly. Maybe some additional CSS resetting wouldn't hurt as well, just in case.

#simplemenu ul {
	font-size: 1em;
	line-height: 1em;
}

The issue occurred with both the original SimpleMenu theme and the new blackblue theme. I had also previously tried out the modified SimpleMenu 3.x theme that blackblue comes from (I love it by the way - thanks for including it, and thank you Jeremey/Nerdliness for making it!) and this behavior did not occur with the original version of blackblue. I'm attaching a small screenshot to illustrate the bug. Mine is a custom theme, though it's a simple enough bit of CSS that it could occur in many others.

Anyhow, hope this helps :) Keep up the great work on SimpleMenu and all the other great stuff you do for Drupal :D

CommentFileSizeAuthor
simplemenu-bug.jpg8.03 KBdnewkerk

Comments

m3avrck’s picture

Status: Active » Fixed

thanks, patch has been comitted!

Anonymous’s picture

Status: Fixed » Closed (fixed)

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