Changeset 50264

Timestamp:

02/09/2021 12:32:54 PM (4 years ago)

Author:

SergeyBiryukov

Message:

Docs: Update some recently added inline comments per the documentation standards.

Follow-up to [50256], [50258], [50259].

See #51800.

Location:

trunk/src

Files:

• js/media/views/modal.js (modified) (1 diff)
• wp-admin/includes/media.php (modified) (1 diff)
• wp-includes/script-loader.php (modified) (1 diff)

trunk/src/js/media/views/modal.js

@@ -133 +133 @@
         }
 
-        // Pause current audio/video even after closing the modal
+        // Pause current audio/video even after closing the modal.
         $( '.mejs-pause button' ).click();
 

trunk/src/wp-admin/includes/media.php

@@ -431 +431 @@
  * @since 5.3.0 The `$post_id` parameter was made optional.
  *
- * @param string[]  $file_array Array that represents a `$_FILES` upload array.
- * @param int       $post_id    Optional. The post ID the media is associated with.
- * @param string    $desc       Optional. Description of the side-loaded file. Default null.
- * @param array     $post_data  Optional. Post data to override. Default empty array.
+ * @param string[] $file_array Array that represents a `$_FILES` upload array.
+ * @param int      $post_id    Optional. The post ID the media is associated with.
+ * @param string   $desc       Optional. Description of the side-loaded file. Default null.
+ * @param array    $post_data  Optional. Post data to override. Default empty array.
  * @return int|WP_Error The ID of the attachment or a WP_Error on failure.
  */

trunk/src/wp-includes/script-loader.php

@@ -269 +269 @@
         }
 
-        // Manually set the text direction localization after wp-i18n is
-        // printed. This ensures that wp.i18n.isRTL() returns true in RTL
-        // languages. We cannot use $scripts->set_translations( 'wp-i18n' ) to
-        // do this because WordPress prints a script's translations *before*
-        // printing the script, which means, in the case of wp-i18n, that
-        // wp.i18n.setLocaleData() is called before wp.i18n is defined.
+        /*
+         * Manually set the text direction localization after wp-i18n is printed.
+         * This ensures that wp.i18n.isRTL() returns true in RTL languages.
+         * We cannot use $scripts->set_translations( 'wp-i18n' ) to do this
+         * because WordPress prints a script's translations *before* the script,
+         * which means, in the case of wp-i18n, that wp.i18n.setLocaleData()
+         * is called before wp.i18n is defined.
+         */
         if ( 'wp-i18n' === $handle ) {
             $ltr    = _x( 'ltr', 'text direction', 'default' );