Changeset 45926 for trunk/src/wp-admin/nav-menus.php

Timestamp:

09/01/2019 05:12:43 PM (5 years ago)

Author:

SergeyBiryukov

Message:

I18N: Improve translator comments.

• Add missing translator comments.
• Fix placement of some translator comments. Translator 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 translator comments.

Includes minor code layout fixes.

Polyglots, rejoice! All WordPress core files now have translator comments for all strings with placeholders!

Props jrf, subrataemfluence, GaryJ, webdados, Dency, swissspidy, alvarogois, marcomartins, mihaiiceyro, vladwtz, niq1982, flipkeijzer, michielatyoast, chandrapatel, thrijith, joshuanoyce, FesoVik, tessak22, bhaktirajdev, cleancoded, dhavalkasvala, garrett-eclipse, bibliofille, socalchristina, priyankkpatel, 5hel2l2y, adamsilverstein, JeffPaul, pierlo, SergeyBiryukov.
Fixes #44360.

File:

• trunk/src/wp-admin/nav-menus.php (modified) (4 diffs)

trunk/src/wp-admin/nav-menus.php

@@ -530 +530 @@
 
 if ( ! current_theme_supports( 'menus' ) && ! $num_locations ) {
-    $messages[] = '<div id="message" class="updated"><p>' . sprintf( __( 'Your theme does not natively support menus, but you can use them in sidebars by adding a &#8220;Navigation Menu&#8221; widget on the <a href="%s">Widgets</a> screen.' ), admin_url( 'widgets.php' ) ) . '</p></div>';
+    $messages[] = '<div id="message" class="updated"><p>' . sprintf(
+        /* translators: URL to Widgets screen */
+        __( 'Your theme does not natively support menus, but you can use them in sidebars by adding a &#8220;Navigation Menu&#8221; widget on the <a href="%s">Widgets</a> screen.' ),
+        admin_url( 'widgets.php' )
+    ) . '</p></div>';
 }
 
 if ( ! $locations_screen ) : // Main tab
-    $overview = '<p>' . __( 'This screen is used for managing your navigation menus.' ) . '</p>';
-    /* translators: 1: Widgets admin screen URL, 2 and 3: The name of the default themes */
-    $overview .= '<p>' . sprintf( __( 'Menus can be displayed in locations defined by your theme, even used in sidebars by adding a &#8220;Navigation Menu&#8221; widget on the <a href="%1$s">Widgets</a> screen. If your theme does not support the navigation menus feature (the default themes, %2$s and %3$s, do), you can learn about adding this support by following the Documentation link to the side.' ), admin_url( 'widgets.php' ), 'Twenty Seventeen', 'Twenty Nineteen' ) . '</p>';
+    $overview  = '<p>' . __( 'This screen is used for managing your navigation menus.' ) . '</p>';
+    $overview .= '<p>' . sprintf(
+        /* translators: 1: Widgets admin screen URL, 2 and 3: The names of the default themes */
+            __( 'Menus can be displayed in locations defined by your theme, even used in sidebars by adding a &#8220;Navigation Menu&#8221; widget on the <a href="%1$s">Widgets</a> screen. If your theme does not support the navigation menus feature (the default themes, %2$s and %3$s, do), you can learn about adding this support by following the Documentation link to the side.' ),
+        admin_url( 'widgets.php' ),
+        'Twenty Seventeen',
+        'Twenty Nineteen'
+    ) . '</p>';
     $overview .= '<p>' . __( 'From this screen you can:' ) . '</p>';
     $overview .= '<ul><li>' . __( 'Create, edit, and delete menus' ) . '</li>';
@@ -656 +665 @@
             echo '<p>' . __( 'Your theme supports one menu. Select which menu you would like to use.' ) . '</p>';
         } else {
-            echo '<p>' . sprintf( _n( 'Your theme supports %s menu. Select which menu appears in each location.', 'Your theme supports %s menus. Select which menu appears in each location.', $num_locations ), number_format_i18n( $num_locations ) ) . '</p>';
+            echo '<p>' . sprintf(
+                /* translators: %s: number of menus */
+                _n(
+                    'Your theme supports %s menu. Select which menu appears in each location.',
+                    'Your theme supports %s menus. Select which menu appears in each location.',
+                    $num_locations
+                ),
+                number_format_i18n( $num_locations )
+            ) . '</p>';
         }
         ?>
@@ -756 +773 @@
             <?php
             printf(
+                /* translators: %s: URL to create a new menu */
                 __( 'Edit your menu below, or <a href="%s">create a new menu</a>. Don&#8217;t forget to save your changes!' ),
                 esc_url(
@@ -817 +835 @@
                 <?php
                 printf(
+                    /* translators: %s: URL to create a new menu */
                     __( 'or <a href="%s">create a new menu</a>. Don&#8217;t forget to save your changes!' ),
                     esc_url(