Changeset 7190 for trunk/src/templates/default/bbpress/form-user-passwords.php

Timestamp:

02/18/2021 05:35:00 AM (5 years ago)

Author:

johnjamesjacoby

Message:

Users: Improve UX of form-user-passwords.php template part.

This change makes sure that the "Generate Password" UI is hidden by default if the browser has JavaScript enabled, and that the traditional UI is visible if JavaScript is not enabled.

It also includes the following related improvements:

• Introduce bbp_doing_script_debug() and bbp_doing_autosave() helpers, and use them where needed
• Introduce bbp_asset_version() and bbp_get_asset_version() to bust asset cache when debugging
• Update forum-user-passwords.php template part to swap out body class and add missing element ID
• Update password form styling in bbpress.css default theme compat CSS

In trunk for 2.7.0. Fixes #3421.

File:

• trunk/src/templates/default/bbpress/form-user-passwords.php (modified) (2 diffs)

trunk/src/templates/default/bbpress/form-user-passwords.php

@@ -13 +13 @@
 // Filters the display of the password fields
 if ( apply_filters( 'show_password_fields', true, bbpress()->displayed_user ) ) : ?>
+
+<script type="text/javascript">
+    document.body.className = document.body.className.replace( 'no-js', 'js' );
+</script>
 
 <div id="password" class="user-pass1-wrap">
@@ -45 +49 @@
 <div class="pw-weak">
     <label for="pw_weak"><?php esc_html_e( 'Confirm', 'bbpress' ); ?></label>
-    <input type="checkbox" name="pw_weak" class="pw-checkbox checkbox" />
+    <input type="checkbox" name="pw_weak" id="pw_weak" class="pw-checkbox checkbox" />
     <p class="description indicator-hint"><?php echo wp_get_password_hint(); ?></p>
 </div>