User login

SooperThemes Blog

Where we talk about Drupal, front-end development, open source software and other cool topics

You are here

Popular Themes

SooperFish 0.2: extra classes for theming

Author:JurriaanRoelofs

Datum:15-6-'10

Tags: 

I just released SooperFish Multi-Column dropdown menus version 0.2. Changelog:

SooperFish 0.2, 15-06-2010
----------------------

- Added classnames to indicate when menus are multicolumn
- Checked compatibility with jQuery 1.4, worked fine for me

Nothing special. I got a complaint about the menu not loading with jQuery 1.4.2. I tried it and it worked fine with 1.4.2, and also with 1.3.x. If anyone still has problems with this, let me know, and if possible also check for any errors in the firebug console!

Comments

There is a bug in Sooperfish both v0.1 & v0.2. This bug occurs when auto arrows are enabled. The way that it adds an extra every time the menu goes one level deeper. So for the first level you have one , for the next sub level you get , and for the next sub level you get 3 times the and so on. The affected code is at line 58 in both versions and it looks like this:

if (o.autoArrows) {
$('>a',this).append('');
}

The fixed code that works correctly looks like this:

if ((o.autoArrows) && ($('>a .'+sf.c.arrowClass,this).length == 0)) {
$('>a',this).append('');
}

If anyone has run into this before, I hope that this helps. I've tested the fix and it works.

In regards to situations where Sooperfish doesn't work with jQuery 1.3.x and 1.4.2: this has happened to me as well, and this occurs usually when someone loads the Sooperfish script and the Sooperfish Easing scripts after the Cufon script in Drupal using the drupal_add_js function.

One more thing: it would be great if actual support for the hoverClass would be added - sfHover. This is helpful when someone wants to style the first level of an active menu.

Thank you for Sooperfish, it's working great otherwise.

Add new comment