Changeset 52929 for trunk/src/wp-content/themes/twentynineteen/inc/template-tags.php

Timestamp:

03/13/2022 08:01:05 PM (4 years ago)

Author:

audrasjb

Message:

i18n: Define List item separator as a WP_Locale property.

The list item separator is a locale property, and it doesn't make much sense to translate it separately in multiple projects. This changeset implements the following modifications:

• Define list item separator as a new WP_Locale property
• Add wp_get_list_item_separator() as a wrapper for WP_Locale::get_list_item_separator
• Replace $wp_locale->get_list_item_separator() calls with wp_get_list_item_separator()
• Added a compatibility layer for bundled themes

Props SergeyBiryukov, swissspidy, rsiddharth, johnbillion, audrasjb.
Fixes #39733.

File:

• trunk/src/wp-content/themes/twentynineteen/inc/template-tags.php (modified) (2 diffs)

trunk/src/wp-content/themes/twentynineteen/inc/template-tags.php

@@ -83 +83 @@
             twentynineteen_posted_on();
 
-            /* translators: Used between list items, there is a space after the comma. */
-            $categories_list = get_the_category_list( __( ', ', 'twentynineteen' ) );
+            $categories_list = get_the_category_list( wp_get_list_item_separator() );
             if ( $categories_list ) {
                 printf(
@@ -95 +94 @@
             }
 
-            /* translators: Used between list items, there is a space after the comma. */
-            $tags_list = get_the_tag_list( '', __( ', ', 'twentynineteen' ) );
+            $tags_list = get_the_tag_list( '', wp_get_list_item_separator() );
             if ( $tags_list && ! is_wp_error( $tags_list ) ) {
                 printf(