Changeset 55655 for trunk/src/wp-content/themes/twentynineteen/comments.php

Timestamp:

04/18/2023 08:22:08 AM (18 months ago)

Author:

audrasjb

Message:

Twenty Nineteen: Fix a translation issue in Comments navigation text.

This changeset combines text strings for "Previous Comments" and "Next Comments" links to ensure translators can control word order in each string. The span tags are included in the translation to give more control over what is hidden on small screens.

Props sabernhardt.
Fixes #58149.

File:

• trunk/src/wp-content/themes/twentynineteen/comments.php (modified) (1 diff)

trunk/src/wp-content/themes/twentynineteen/comments.php

@@ -88 +88 @@
         // Show comment navigation.
         if ( have_comments() ) :
-            $prev_icon     = twentynineteen_get_icon_svg( 'chevron_left', 22 );
-            $next_icon     = twentynineteen_get_icon_svg( 'chevron_right', 22 );
-            $comments_text = __( 'Comments', 'twentynineteen' );
+            $prev_icon = twentynineteen_get_icon_svg( 'chevron_left', 22 );
+            $next_icon = twentynineteen_get_icon_svg( 'chevron_right', 22 );
             the_comments_navigation(
                 array(
-                    'prev_text' => sprintf( '%s <span class="nav-prev-text"><span class="primary-text">%s</span> <span class="secondary-text">%s</span></span>', $prev_icon, __( 'Previous', 'twentynineteen' ), __( 'Comments', 'twentynineteen' ) ),
-                    'next_text' => sprintf( '<span class="nav-next-text"><span class="primary-text">%s</span> <span class="secondary-text">%s</span></span> %s', __( 'Next', 'twentynineteen' ), __( 'Comments', 'twentynineteen' ), $next_icon ),
+                    'prev_text' => sprintf(
+                        '%1$s <span class="nav-prev-text">%2$s</span>',
+                        $prev_icon,
+                        /* translators: Comments navigation link text. The secondary-text element is hidden on small screens. */
+                        __( '<span class="primary-text">Previous</span> <span class="secondary-text">Comments</span>', 'twentynineteen' )
+                    ),
+                    'next_text' => sprintf(
+                        '<span class="nav-next-text">%1$s</span> %2$s',
+                        /* translators: Comments navigation link text. The secondary-text element is hidden on small screens. */
+                        __( '<span class="primary-text">Next</span> <span class="secondary-text">Comments</span>', 'twentynineteen' ),
+                        $next_icon
+                    ),
                 )
             );