Thickbox.js doesn't involve any animation any more (developers decision) and to add animation one needs to alter the thickbox.js file. To make future upgrades easier I've patched the module to check the current theme folder for the existence of two files, thickbox.js and thickbox.css. If either exist they are used instead of the module files. This allows the module to remain untouched while providing the possibility of a customised theme based solution.

I've attached the patch to enable this feature. The code is adapted from something similar in the jquery lightbox module.

Let me know if you think it makes sense to include this or whether it should be the task of the template.php to override the files.

CommentFileSizeAuthor
thickbox-theme.patch1.36 KBtanc

Comments

frjo’s picture

frjo commented
Assigned: Unassigned » frjo

Neat functionality! I will commit it after I have had a chance to test it out.

frjo’s picture

frjo commented
Version: 5.x-1.2 » 6.x-1.x-dev

I need to revisit this.

frjo’s picture

frjo commented
Status: Needs review » Closed (won't fix)

Please test the new Colorbox module and see it that works better.

http://drupal.org/project/colorbox

I have ported all Thickbox features to Colorbox so it should work as a drop in replacement.

When Colorbox has a stable release I will start actively recommend people to switch/upgrade from Thickbox to Colorbox. I'm the maintainer of Thickbox and a co-maintainer of Colorbox.

tanc’s picture

tanc
he/him
Primary language English
Location Italy
commented

Hi frjo, I independently came across colorbox while creating a Drupal 7 site and really like its functionality so far. I haven't tried to theme it at all so I can't report on its flexibility, but from what I can see it looks like there are theme overrides for the css. Thanks for porting to colorbox, a worthy replacement.