Changeset 17596

Timestamp:

04/05/2011 03:58:25 AM (15 years ago)

Author:

dd32

Message:

Pass image URL's to thickbox via L10n in script-loader.php. Allows thickbox to be used on any page in the install without defining the image URL variables. Fixes #17050

Location:

trunk/wp-includes

Files:

• js/thickbox/thickbox.js (modified) (1 diff)
• script-loader.php (modified) (2 diffs)

trunk/wp-includes/js/thickbox/thickbox.js

@@ -7 +7 @@
 
 if ( typeof tb_pathToImage != 'string' ) {
-        var tb_pathToImage = "../wp-includes/js/thickbox/loadingAnimation.gif";
+        var tb_pathToImage = thickboxL10n.loadingAnimation;
 }
 if ( typeof tb_closeImage != 'string' ) {
-        var tb_closeImage = "../wp-includes/js/thickbox/tb-close.png";
+        var tb_closeImage = thickboxL10n.closeImage;
 }
 

trunk/wp-includes/script-loader.php

@@ -185 +185 @@
         $scripts->add_data( 'jquery-table-hotkeys', 'group', 1 );
 
-        $scripts->add( 'thickbox', "/wp-includes/js/thickbox/thickbox.js", array('jquery'), '3.1-20100407');
+        $scripts->add( 'thickbox', "/wp-includes/js/thickbox/thickbox.js", array('jquery'), '3.1-20110405');
         $scripts->add_data( 'thickbox', 'group', 1 );
         $scripts->localize( 'thickbox', 'thickboxL10n', array(
@@ -194 +194 @@
                         'close' => __('Close'),
                         'noiframes' => __('This feature requires inline frames. You have iframes disabled or your browser does not support them.'),
+                        'loadingAnimation' => includes_url('js/thickbox/loadingAnimation.gif'),
+                        'closeImage' => includes_url('js/thickbox/tb-close.png'),
                         'l10n_print_after' => 'try{convertEntities(thickboxL10n);}catch(e){};'
         ) );