Changeset 26071

Timestamp:

11/09/2013 11:27:28 PM (11 years ago)

Author:

lancewillett

Message:

Twenty Fourteen: allow slider "Previous" and "Next" labels to be translated, props michahwave. Fixes #25897.

Location:

trunk/src/wp-content/themes/twentyfourteen

Files:

• functions.php (modified) (1 diff)
• js/slider.js (modified) (1 diff)

trunk/src/wp-content/themes/twentyfourteen/functions.php

@@ -236 +236 @@
         wp_enqueue_script( 'jquery-masonry' );
 
-    if ( is_front_page() && 'slider' == get_theme_mod( 'featured_content_layout' ) )
-        wp_enqueue_script( 'twentyfourteen-slider', get_template_directory_uri() . '/js/slider.js', array( 'jquery' ), '20131028', true );
+    if ( is_front_page() && 'slider' == get_theme_mod( 'featured_content_layout' ) ) {
+        wp_enqueue_script( 'twentyfourteen-slider', get_template_directory_uri() . '/js/slider.js', array( 'jquery' ), '20131109', true );
+        wp_localize_script( 'twentyfourteen-slider', 'featuredSliderDefaults', array(
+            'prevText' => __( 'Previous', 'twentyfourteen' ),
+            'nextText' => __( 'Next', 'twentyfourteen' )
+        ) );
+    }
 
     wp_enqueue_script( 'twentyfourteen-script', get_template_directory_uri() . '/js/functions.js', array( 'jquery' ), '20131102', true );

trunk/src/wp-content/themes/twentyfourteen/js/slider.js

@@ -547 +547 @@
         controlsContainer: '',    // jQuery Object/Selector: container navigation to append elements.
 
-        // Text labels: @todo allow translation
-        prevText: 'Previous',     // String: Set the text for the "previous" directionNav item.
-        nextText: 'Next'          // String: Set the text for the "next" directionNav item.
+        // Text labels.
+        prevText: featuredSliderDefaults.prevText, // String: Set the text for the "previous" directionNav item.
+        nextText: featuredSliderDefaults.nextText  // String: Set the text for the "next" directionNav item.
     };