The views 2.9 or 2.10 update broke the inline javascript that conditionally hides the options form for each contrib slideshow plugin. Each plugin's option fieldset is always visible regardless of which plugin is selected in the "mode" dropdown. Patch to follow ..

Comments

fearlsgroove’s picture

Status: Active » Needs review
StatusFileSize
new3.26 KB

This patch uses views dependent.js to show and hide the appropriate options forms when the mode is changed. There's a bit of trickery involved since fieldsets can't be used directly with dependent.js, as they have no unique ID and do not get the "#process" property applied.

redndahead’s picture

Thanks didn't know there was a dependent.js to use. I'm currently in the process of rewriting views_slideshow for 3.x and have removed the fieldsets for various other reasons. So hopefully the hack won't be needed then.

I'm assuming you get a js error and that's why it's not working. Can you tell me what that error is? Mine works fine.

fearlsgroove’s picture

I don't get an error actually, it just doesn't execute the script (at least on Firefox 3.6) ... my guess is that views changed the way it displays the returned results such that it doesn't process inline javascript anymore. You're testing 2.x-dev and views 2.10?

redndahead’s picture

Yep...I'm using FF 3.6 also. It would seem impossible to not let it use my js as it's just printed, but maybe since I'm not using a document ready there is a race condition. Anyway using dependent.js seems the way to go. It'll take me a bit to test your code, but I look forward to it.

redndahead’s picture

Status: Needs review » Fixed
StatusFileSize
new6.1 KB

Ok code tested and the patch attached will be committed. It adds a change to singleframe to use dependent.js for the pager options. It's a nice reduction in code I like it.

Status: Fixed » Closed (fixed)

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