Most recent version of Colorbox stores jquery.colorbox-min.js in a different place. Here's a simple patch that fixes the path.

I'm wondering if we should even add colorbox library this way, could we just call drupal_add_library('media_browser_plus', 'colorbox'); somewhere?

CommentFileSizeAuthor
#1 media_browser_plus_fix_colorbox_path-1896574-1.patch555 bytespaulihuhtiniemi

Comments

paulihuhtiniemi’s picture

paulihuhtiniemi commented
StatusFileSize
new media_browser_plus_fix_colorbox_path-1896574-1.patch555 bytes

Here's the actual patch.

frjo’s picture

frjo commented

Since you already have an dependency for colorbox why not use its library integration?

dependencies[] = libraries (2.x)
dependencies[] = colorbox (2.x)

and then you can load the Colorbox plugin where ever you need with a simple

$variant = variable_get('colorbox_compression_type', 'minified');
libraries_load('colorbox', $variant);
frjo’s picture

frjo commented
Issue summary: View changes

updated js file name.

kruser’s picture

kruser commented

I think this need fixed in 7.x-3.x-dev too because I'm getting errors of it looking for /sites/all/libraries/colorbox/colorbox/jquery.colorbox-min.js?v=1.3.9 - but it should be /sites/all/libraries/colorbox/jquery.colorbox-min.js?v=1.3.9

salvis’s picture

salvis
he/his
commented
Issue summary: View changes
Status: Needs review » Closed (outdated)