Changeset 14457

Timestamp:

05/28/2025 05:37:40 AM (9 months ago)

Author:

dd32

Message:

WP I18N Teams: Update the team page styling with the newer block buttons.

Props dd32, dhrumilk, mikinc860, rollybueno.
Fixes #7916, #7986.
Closes ​https://github.com/WordPress/wordpress.org/pull/490.

Location:

sites/trunk/wordpress.org/public_html/wp-content/plugins/wp-i18n-teams

Files:

• css/i18n-teams.css (modified) (1 diff)
• inc/locales.php (modified) (1 diff)
• views/locale-details.php (modified) (1 diff)

sites/trunk/wordpress.org/public_html/wp-content/plugins/wp-i18n-teams/css/i18n-teams.css

@@ -104 +104 @@
     box-shadow: 0 1px 1px rgba(0, 0, 0, 0.05) inset;
 }
-#locale-header ul,
-#main ul#postlist .postcontent #locale-header ul {
-    overflow: auto;
-    margin-left: 0 !important; /* !important is necessary here to override P2's use of !important in .postcontent ul */
-}
+
 #locale-header ul li {
     list-style-type: none;
 }
-#locale-header ul#locale-details {
-    margin-bottom: 0 !important; /* !important is necessary here to override P2's use of !important in .postcontent ul */
-}
-#locale-download {
-    margin-bottom: 0 !important; /* !important is necessary here to override P2's use of !important in .postcontent ul */
-}
-#locale-header ul li.download-button,
-#main ul#postlist .postcontent #locale-header ul li.download-button {
-    float: left;
-    height: unset;
-    margin: 8px 12px 0 0;   /* todo put list spacing on the button itself, so that the ul takes up no space when there are no li's inside it */
-    padding: 6px 12px;
-    text-align: center;
-    white-space: unset;
-}
-#locale-header ul li.download-button a {
-    font-size: 14px;
-    font-size: 1.4rem;
-    line-height: 1.5;
-    color: white;
-    text-decoration: none;
-}
-@media only screen and (max-width: 600px) {
-    #locale-header ul li.download-button,
-    #main ul#postlist .postcontent #locale-header ul li.download-button {
-        width: 100%;
-    }
-}
-
-#locale-details,
-#main ul#locale-details {
+
+#locale-details {
     font-size: 1.2em;
-}
-
-.validators,
-#main ul#postlist .postcontent ul.validators {
+    padding-left: 0;
+}
+
+.validators {
     overflow: auto;
-    margin: 0 !important; /* !important is necessary here to override P2's use of !important in .postcontent ul */
     padding: 0;
 }

sites/trunk/wordpress.org/public_html/wp-content/plugins/wp-i18n-teams/inc/locales.php

@@ -26 +26 @@
 function enqueue_assets(): void {
     if ( is_singular() && false !== strpos( get_post()->post_content, '[wp-locales' ) ) {
-        wp_enqueue_style( 'wp-i18n-teams', plugins_url( 'css/i18n-teams.css', PLUGIN_FILE ), [], 13 );
-        wp_enqueue_script( 'wp-i18n-teams', plugins_url( 'js/i18n-teams.js', PLUGIN_FILE ), [ 'jquery', 'o2-app' ], 5 );
+        wp_enqueue_style( 'wp-i18n-teams', plugins_url( 'css/i18n-teams.css', PLUGIN_FILE ), [], filemtime( dirname( __DIR__ ) . '/css/i18n-teams.css' ) );
+        wp_enqueue_script( 'wp-i18n-teams', plugins_url( 'js/i18n-teams.js', PLUGIN_FILE ), [ 'jquery', 'o2-app' ], filemtime( dirname( __DIR__ ) . '/js/i18n-teams.js' ) );
     }
 }

sites/trunk/wordpress.org/public_html/wp-content/plugins/wp-i18n-teams/views/locale-details.php

@@ -45 +45 @@
 
     <?php if ( $locale_data['localized_core_url'] ) : ?>
-        <ul id="locale-download">
-            <li class="button download-button">
-                <a href="<?php echo esc_url( $locale_data['localized_core_url'] ); ?>" role="button">
-                    <?php printf( __( 'Download WordPress in %s', 'wporg' ), $locale->english_name ); ?>
+        <div id="locale-download" class="wp-block-buttons is-layout-flex wp-block-buttons-is-layout-flex">
+            <div class="wp-block-button">
+                <a class="wp-block-button__link wp-element-button" href="<?php echo esc_url( $locale_data['localized_core_url'] ); ?>">
+                    <?php
+                    // translators: %s is the english variant of the locale name.
+                    printf( __( 'Download WordPress in %s', 'wporg' ), $locale->english_name );
+                    ?>
                 </a>
-            </li>
-
+            </div>
             <?php if ( $locale_data['language_pack_url'] ) : ?>
-                <li class="button download-button">
-                    <a href="<?php echo esc_url( $locale_data['language_pack_url'] ); ?>" role="button">
+                <div class="wp-block-button is-style-outline">
+                    <a class="wp-block-button__link wp-element-button" href="<?php echo esc_url( $locale_data['language_pack_url'] ); ?>" role="button">
                         <?php
-                        // translators: %s is the latest version
+                        // translators: %s is the latest version.
                         printf( __( 'Download language pack (%s)', 'wporg' ), $locale_data['language_pack_version'] );
                         ?>
                     </a>
-                </li>
+                </div>
             <?php endif; ?>
-        </ul>
+        </div>
     <?php endif;  ?>
 </div>