Changeset 17596

Timestamp:

04/05/2011 03:58:25 AM (14 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){};'
     ) );