Changeset 43598 for trunk/src/wp-admin/includes/class-theme-installer-skin.php

Timestamp:

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

Author:

jrf

Message:

I18n: Improve translators comments [3].

• 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.

Includes minor code layout fixes.

Patch 44360-wp-admin-includes-dir.patch of the series.

Props flipkeijzer, alvarogois, michielatyoast
See #44360

File:

• trunk/src/wp-admin/includes/class-theme-installer-skin.php (modified) (2 diffs)

trunk/src/wp-admin/includes/class-theme-installer-skin.php

@@ -76 +76 @@
 
         if ( current_user_can( 'edit_theme_options' ) && current_user_can( 'customize' ) ) {
-            $customize_url              = add_query_arg(
+            $customize_url               = add_query_arg(
                 array(
                     'theme'  => urlencode( $stylesheet ),
@@ -83 +83 @@
                 admin_url( 'customize.php' )
             );
-            $install_actions['preview'] = '<a href="' . esc_url( $customize_url ) . '" class="hide-if-no-customize load-customize"><span aria-hidden="true">' . __( 'Live Preview' ) . '</span><span class="screen-reader-text">' . sprintf( __( 'Live Preview &#8220;%s&#8221;' ), $name ) . '</span></a>';
+            $install_actions['preview']  = '<a href="' . esc_url( $customize_url ) . '" class="hide-if-no-customize load-customize">';
+            $install_actions['preview'] .= '<span aria-hidden="true">' . __( 'Live Preview' ) . '</span>';
+            /* translators: %s: theme name */
+            $install_actions['preview'] .= '<span class="screen-reader-text">' . sprintf( __( 'Live Preview &#8220;%s&#8221;' ), $name ) . '</span></a>';
         }
-        $install_actions['activate'] = '<a href="' . esc_url( $activate_link ) . '" class="activatelink"><span aria-hidden="true">' . __( 'Activate' ) . '</span><span class="screen-reader-text">' . sprintf( __( 'Activate &#8220;%s&#8221;' ), $name ) . '</span></a>';
+        $install_actions['activate']  = '<a href="' . esc_url( $activate_link ) . '" class="activatelink">';
+        $install_actions['activate'] .= '<span aria-hidden="true">' . __( 'Activate' ) . '</span>';
+        /* translators: %s: theme name */
+        $install_actions['activate'] .= '<span class="screen-reader-text">' . sprintf( __( 'Activate &#8220;%s&#8221;' ), $name ) . '</span></a>';
 
         if ( is_network_admin() && current_user_can( 'manage_network_themes' ) ) {