After I have updated the module to the latest dev version of Feb 8, 2013, my FlexSlider and Galleria on the same page were broken.

If I remove the Superfish menu, both of them were backed to normal.

In Firebug I caught the below error:

TypeError: list.sfsmallscreen is not a function

list.sfsmallscreen(options.plugins.smallscreen);

http://my-url/sites/all/modules/superfish/superfish.js -- Line 31

To fix this, I have to delete the below few lines in the module's superfish.js:

            if (options.plugins.smallscreen || false) {
              list.sfsmallscreen(options.plugins.smallscreen);
            }

I am not sure about the function of the above removed lines, so I am not sure what side effect that my action will bring.

Comments

aglobalwander_’s picture

I have the same issue with a flex slider gallery. I haven't tried the suggestion above yet but am curious as to what the fix will be.

aglobalwander_’s picture

I should have read the instructions more clearly. After updating the libraries superfish folder and clicking save configuration on the superfish config page everything works as expected. Thanks for a great module.

deanflory’s picture

Yeah, it breaks all fieldset toggle-downs too. I'll try what #2 recommended and read the readme.

deanflory’s picture

I updated the superfish library.
https://github.com/mehrpadin/Superfish-for-Drupal/zipball/master

I edited and clicked "Save Configuration" on the Superfish Module admin page.
/admin/config/user-interface/superfish

I edited and clicked "Save block" on each Superfish menu block.
/admin/structure/block/manage/superfish/1/configure
/admin/structure/block/manage/superfish/2/configure
/admin/structure/block/manage/superfish/3/configure
/admin/structure/block/manage/superfish/4/configure

Flushed all caches.

The superfish update is still causing fieldset un-collapsing (toggle-downs) to not work properly, so there is definitely an error somewhere. At least after 3 days I've found the problem and can just back-track to the old module version. Moving on...

Reverting back to the previous version indeed allows the toggle-downs to again function properly.

retiredpro’s picture

i had the same error on drupal 6 when i updated to the latest dev version and library of superfish. it looks like the superfish drupal configuration page is missing a filepath for sfsmallscreen.js

so i added the following line to my superfish library paths and it fixed the issue for me.
sites/all/libraries/superfish/sfsmallscreen.js

dddave’s picture

Status: Active » Fixed

Had the same problems but ultimately it was my fault for not uploading the right version of the library. Unless somebody can show that this breaks something on a correct install I set this to fixed.

deanflory’s picture

Adding the library path on the Superfish settings page fixed the issue:

sites/all/libraries/superfish/sfsmallscreen.js

metakel’s picture

Yes, #7 fixes the problem.

However, it seems that the file downloaded from the link to the Superfish-for-Drupal on the project page does not contain the sfsmallscreen.js. We better go to the Github project page and download the zip file there directly.

deanflory’s picture

Whoa, good catch metakel.

grrr

dddave’s picture

Re #8,9: This is actually described on the modules page under "latest development version".

deanflory’s picture

Yes that the link to sfsmallscreen.js is on the modules page, but the fact that that sfsmallscreen.js file itself isn't in the library that is to be downloaded isn't noted.

mehrpadin’s picture

Hey everybody,

Ok, I've edited it, should be more clear now, please take a look.

:)

sportel’s picture

I'm not sure what to put on the Superfish settings page. Do I add just the library path for Superfish (sites/all/libraries/superfish), or just the library path for the sfsmallscreen.js file (sites/all/libraries/superfish/sfsmallscreen.js), or the comlete paths to all the files in the library?

Thanks,

Mike.

deanflory’s picture

sportel, here's what I have for settings:

Path to Superfish library

sites/all/libraries/superfish/jquery.hoverIntent.minified.js
sites/all/libraries/superfish/jquery.bgiframe.min.js
sites/all/libraries/superfish/superfish.js
sites/all/libraries/superfish/supersubs.js
sites/all/libraries/superfish/supposition.js
sites/all/libraries/superfish/sftouchscreen.js
sites/all/libraries/superfish/sfsmallscreen.js

sportel’s picture

Thanks, I installed and activated the superfish module before I copied the superfish library to it's folder. The module doesn't pick up the paths by itself. It works fine now.

Mike.

Status: Fixed » Closed (fixed)

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

jackhutton’s picture

dunno if i can reopen this.. but, I'm having this issue with a flexslider sharing the landing page with superfish menus

The flexslider does not render when I enable the superfish menu as the main menu.

I've updated the libraries .. reviewed inclusion of the noted files and path configurations in #15 ..

I do see a single error using firebug on line 39 of supersubs.js

TypeError: fontsize.attr(...).appendTo(...)[0] is undefined
[Break On This Error]

.appendTo($$)[0].clientWidth; //clientWidth is faster than width()

I see posts mentioning the z-index..
but not having success getting the superfish menu to render with the flexslider

any thoughts?
thank you