Changeset 43596

Timestamp:

08/30/2018 12:35:08 PM (6 years ago)

Author:

jrf

Message:

I18n: Improve translators comments [2].

• Add missing translators comments.
• Fix placement of some translators comments. Translators comments should be on the line directly above the line containing the translation function call for optimal compatibility with various .pot file generation tools. The CS auto-fixing, which changed some inconsistent function calls to multi-line function calls, is part of the reason why this was no longer the case for a select group of translators comments.

Patch 44360.wp-includes_customize.3.patch of the series.

Props marcomartins, mihaiiceyro, michielatyoast
See #44360

Location:

trunk/src/wp-includes/customize

Files:

• class-wp-customize-header-image-control.php (modified) (1 diff)
• class-wp-customize-nav-menu-locations-control.php (modified) (1 diff)
• class-wp-customize-themes-section.php (modified) (1 diff)

trunk/src/wp-includes/customize/class-wp-customize-header-image-control.php

@@ -183 +183 @@
                     _e( 'Click “Add new image” to upload an image file from your computer. Your theme works best with an image that matches the size of your video — you’ll be able to crop your image once you upload it for a perfect fit.' );
                 } elseif ( $width && $height ) {
-                    /* translators: %s: header size in pixels */
                     printf(
+                        /* translators: %s: header size in pixels */
                         __( 'Click “Add new image” to upload an image file from your computer. Your theme works best with an image with a header size of %s pixels — you’ll be able to crop your image once you upload it for a perfect fit.' ),
                         sprintf( '<strong>%s &times; %s</strong>', $width, $height )
                     );
                 } elseif ( $width ) {
-                    /* translators: %s: header width in pixels */
                     printf(
+                        /* translators: %s: header width in pixels */
                         __( 'Click &#8220;Add new image&#8221; to upload an image file from your computer. Your theme works best with an image with a header width of %s pixels &#8212; you&#8217;ll be able to crop your image once you upload it for a perfect fit.' ),
                         sprintf( '<strong>%s</strong>', $width )
                     );
                 } else {
-                    /* translators: %s: header height in pixels */
                     printf(
+                        /* translators: %s: header height in pixels */
                         __( 'Click &#8220;Add new image&#8221; to upload an image file from your computer. Your theme works best with an image with a header height of %s pixels &#8212; you&#8217;ll be able to crop your image once you upload it for a perfect fit.' ),
                         sprintf( '<strong>%s</strong>', $height )

trunk/src/wp-includes/customize/class-wp-customize-nav-menu-locations-control.php

@@ -75 +75 @@
                                 <span class="theme-location-set">
                                     <?php
-                                    /* translators: %s: menu name */
                                     printf(
+                                        /* translators: %s: menu name */
                                         _x( '(Current: %s)', 'menu location' ),
                                         '<span class="current-menu-location-name-' . esc_attr( $location ) . '"></span>'

trunk/src/wp-includes/customize/class-wp-customize-themes-section.php

@@ -88 +88 @@
                     <p class="no-themes-local">
                         <?php
-                        /* translators: %s: "Search WordPress.org themes" button */
                         printf(
+                            /* translators: %s: "Search WordPress.org themes" button text */
                             __( 'No themes found. Try a different search, or %s.' ),
                             sprintf( '<button type="button" class="button-link search-dotorg-themes">%s</button>', __( 'Search WordPress.org themes' ) )