Allow all Colorbox plugin settings to be controlled via url queries [#1161342]

Hi,
I'm the maintainer of Field Slideshow, and one of the options is to open images in Colorbox.
Everything is working fine, but I'm trying to add an option to start the slideshow in Colorbox when an image is opened in Colorbox.
For now, links look like this:
<a href="/sites/default/files/japart.png" class="colorbox" rel="field-slideshow[2]">
so perhaps it should be somethink like:
<a href="/sites/default/files/japart.png?slideshow=true" class="colorbox" rel="field-slideshow[2]">

Tell me.

Comment	File	Size	Author
#9	colorbox_load_bug_fix_ 1161342_9.patch	1.4 KB	frjo
#9
#7	colorbox_load_bug_fix.patch	801 bytes	frjo
#7
#1	colorbox_load_parameters.patch	4.23 KB	frjo
#1

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

frjo CreditAttribution: frjo commented 19 May 2011 at 12:23

File	Size
colorbox_load_parameters.patch	4.23 KB

There is settings for it on Configuration -> Colorbox. See under Styles and options -> Custom -> Slideshow settings. The settings makes this however a global option, not what you are looking for perhaps.

The attached patch make the colorbox-load feature accepts any Colorbox settings as url queries. You can see all the options at http://colorpowered.com/colorbox/.

Could you test the patch and see how it works for you? It's against the latest 7-dev code?

A link could look like this:

<a class="colorbox-load" rel="something" href="some/path?slideshow=true&slideshowAuto=true&slideshowSpeed=2500">Something</a>

The class would need to change from "colorbox" to "colorbox-load". You would also need to enable Colorbox load in the Colorbox configurations.

My tips is that you run "variable_set('colorbox_load', 1)" in your module when users select to use the Colorbox function.

Comment #2

frjo CreditAttribution: frjo commented 20 May 2011 at 12:29

Field Slideshow is a quite neat module! Believe I will use it on one of my current projects, thanks for developing it!

Comment #3

jdanthinne CreditAttribution: jdanthinne commented 26 May 2011 at 14:08

Thanks, and thanks again for the patch, I'll have a try this week-end.

Comment #4

frjo CreditAttribution: frjo commented 1 July 2011 at 07:16

Status:

Active

» Fixed

Committed patch #1 to 7-dev.

Comment #5

jdanthinne CreditAttribution: jdanthinne commented 14 July 2011 at 12:48

Many Thanks.
Updated my module to use that, and it's working fine!

Comment #6

jdanthinne CreditAttribution: jdanthinne commented 14 July 2011 at 13:22

Status:

Fixed

» Active

Mmm.
Just added an option in my module to choose between auto or manual slideshow, but it's not working.
If I choose Auto (and then adds &slideshowAuto=true), it's ok, but Manual (&slideshowAuto=false) goes automatic as well…
What's wrong?

Comment #7

frjo CreditAttribution: frjo commented 14 July 2011 at 23:52

Status:

Active

» Needs review

File	Size
colorbox_load_bug_fix.patch	801 bytes

Do you happen to have more than one colorbox-load link on the page? I have found a bug in my #1 patch. The bug make all settings the same as for the last link on the page. Sloppy testing by me.

Here are a patch that I believe will fix this bug.

Would be glad if you could test it.

Comment #8

jdanthinne CreditAttribution: jdanthinne commented 15 July 2011 at 07:05

I've tested that with multiple links on the page, and it seems ok.
But #6 bug is still there… but I don't know if it's your module that's faulty or Colorbox itself…