I don't know if this is feasible, but could we have support for lightbox2 in addition to thickbox, also, is thickbox even supported for 5.x?

Comments

a_c_m’s picture

Version: 5.x-1.x-dev » 6.x-2.x-dev

+ 1 to this but for D6.

seanr’s picture

Any particular reason to use Lightbox2 instead of thickbox? We're using thickbox on a D5 site with no problems.

jwilde’s picture

Hi,

Seanr ,thanks for picking up the support and maintenance of splash.

Anyway, I'm not able to get the thickbox to work in 6.x.

Kind regards,

Jim

j0rd’s picture

Lightbox2 is better than thickbox in my opinion. I switched off thickbox a while ago and It's much faster, better looking and a more elegant solution in general. I think people are moving off thickbox and into lightbox2. Take ubercart for example.

I'd like this to be supported as well. I'd also like to remove thickbox from inside this module and instead require thickbox / lightbox module dependencies instead. Selectable "popup" support would be the best.

I was able to get Thickbox working in 6.x, but I did quite a bit custom programming and fixes in my version, so I'm not sure if I fixed this or if it just worked.

This module does need a little bit of clean up work to get it to where it's a viable option for casual users. Thanks for taking over support and I appreciate any future work.

seanr’s picture

jwilde, can you try it with one of the core themes? It works fine with garland for me, so it may be a theme related issue.

jwilde’s picture

Ok. I tried the thickbox option with garland and nothing happens. I also tried zen and got the same results.

I'll look in to the module more when I get some time. Thanks for your speedy reply.

Kind regards,

Jim

seanr’s picture

jwilde, can you list your exact settings for all the tabs so I can try to reproduce the problem? Also, do you have any other modules enabled that do anything JavaScript or jQuery related? Also, make sure you've got the latest versions of everything.

seanr’s picture

Status: Active » Postponed (maintainer needs more info)

I discovered there was a conflict between this module and the thickbox module because this one packaged it's own (and thus dusplicate) copy of the thickbox javascript code. I'm looking into integrating with Lightbox as well, but having a very difficult time due to the way Lightbox2 was designed. I have an issue in their queue seeking assistance: #565554: Invoke Lightbox2 on page load?.

dirksonii’s picture

Looking at the lightbox thread, your original issue appears mostly solved. Any chance we could get an ETA on this? I'd loooove to stop using thickbox, seems silly to use it on the front page, then use lightbox everywhere else on the site.

nicktech’s picture

Subscribing

lpalgarvio’s picture

also Colorbox and Shadowbox.

Colorbox is the upgrade path for Thickbox.

jmseigneur’s picture

+1 for the integration of Colorbox as it is the Thickbox Drupal projet official upgrade. Thanks to let us know if Colorbox is supported as well or not.

g.k’s picture

+1 for colorbox

mogop’s picture

+1 for lightbox2
[it's most used and it's loaded with ubercart by default]

bt82’s picture

+1 for colorbox

lpalgarvio’s picture

Title: support for lightbox » support for lightbox, colorbox and other box modules

changed title

bryancasler’s picture

subscribe, +1 for colorbox

moejarv’s picture

StatusFileSize
new821 bytes
new1.37 KB

Here is a patch for colorbox against the dev version at 2011-Feb-25.

This is the very first time I make patch... Please let me know if there is problem!

This should add the colorbox option in the admin section, and load the colorbox js on document.ready

bryancasler’s picture

I would love to test this, but I've moved on to D7. If and when that becomes available I will happily give it a try.

bserem’s picture

I'm giving a try to these patches, I got this upon patching:

# patch < splash.module-288043-18.patch 
patching file splash.module
Hunk #1 succeeded at 197 (offset 1 line).

is this normal?

bserem’s picture

Status: Postponed (maintainer needs more info) » Reviewed & tested by the community

#18 works with colorbox in Opera, Chrome and Firefox.

It required the size to be set in settings area in order for it to work correctly under Firefox and Opera, but this is no problem at all!

madmanmax’s picture

Has anyone managed to get it working with lightbox2? I see that it checks if the module is installed and adds it as an option but there's nothing in the code to suggest it actually works.

I'm thinking of adding a patch but I'm not sure what's the best approach here. As far as I managed to read about lightbox2 you need some javascript to simulate a click on a link or add the content inside of a div. I think the link approach might be the easiest solution: use some javascript to add a link to the /splash page and programmatically click it.

Any ideas/suggestions?

chrisroane’s picture

As an FYI: I created a drupal 7 module called splashify that uses colorbox by default. Here is the sandbox project: http://drupal.org/node/1425392

I am willing to support other lightbox solutions with this module, but I need help in getting this module approved to become a full drupal project. Here is the issue that covers the project: http://drupal.org/node/1425392

mail@victorquinn.com’s picture

Status: Reviewed & tested by the community » Fixed

moejarv's patch applied (and enhanced a bit) in the latest git commit.

No clue why the Lightbox option is in the admin settings as it does nothing. Maybe it can get implemented at some point if there is enough desired from the community, but at this point with both Thickbox and Colorbox, do we need yet another *box? I suspect that's unnecessary.

Status: Fixed » Closed (fixed)

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