WordPress.org

Make WordPress Core

Ticket #33778: 33778.3.patch

File 33778.3.patch, 3.4 KB (added by umesh.nevase, 6 years ago)

same issue also exists on New user and Edit profile pages. I've added id and script that handled the issue for all ( Install.php, user-new.php, profile.php ) password label

  • js/user-profile.js

     
    5757                        } );
    5858
    5959                $pass1.after( $pass1Text );
    60 
     60               
    6161                if ( 1 === parseInt( $pass1.data( 'reveal' ), 10 ) ) {
    6262                        generatePassword();
    6363                }
     
    104104                                        .find( '.dashicons' )
    105105                                                .removeClass('dashicons-visibility')
    106106                                                .addClass('dashicons-hidden');
    107 
     107                                       
    108108                                $pass1Text.focus();
     109                               
     110                                $pass1Row.find('th').children('label').attr({'for':'pass1-text'});
    109111
    110112                                if ( ! _.isUndefined( $pass1Text[0].setSelectionRange ) ) {
    111113                                        $pass1Text[0].setSelectionRange( 0, 100 );
     
    125127                                                .addClass('dashicons-visibility');
    126128
    127129                                $pass1.focus();
     130                               
     131                                $pass1Row.find('th').children('label').attr({'for':'pass1'});
    128132
    129133                                if ( ! _.isUndefined( $pass1[0].setSelectionRange ) ) {
    130134                                        $pass1[0].setSelectionRange( 0, 100 );
     
    177181                $passwordWrapper = $pass1Row.find('.wp-pwd').hide();
    178182
    179183                bindToggleButton();
     184               
     185                if( $pass1Row.find( 'button.wp-generate-pw' ).length )
     186                        $pass1Row.find('th').children('label').attr({'for':'wp-generate-pw'});
     187                else
     188                        $pass1Row.find('th').children('label').attr({'for':'pass1-text'});
    180189
    181190                $generateButton = $pass1Row.find( 'button.wp-generate-pw' ).show();
    182191                $generateButton.on( 'click', function () {
  • user-new.php

     
    404404if ( apply_filters( 'show_password_fields', true ) ) : ?>
    405405        <tr class="form-field form-required user-pass1-wrap">
    406406                <th scope="row">
    407                         <label for="pass1">
     407                        <label for="wp-generate-pw">
    408408                                <?php _e( 'Password' ); ?>
    409409                                <span class="description hide-if-js"><?php _e( '(required)' ); ?></span>
    410410                        </label>
     
    411411                </th>
    412412                <td>
    413413                        <input class="hidden" value=" " /><!-- #24364 workaround -->
    414                         <button type="button" class="button button-secondary wp-generate-pw hide-if-no-js"><?php _e( 'Show password' ); ?></button>
     414                        <button type="button" id="wp-generate-pw" class="button button-secondary wp-generate-pw hide-if-no-js"><?php _e( 'Show password' ); ?></button>
    415415                        <div class="wp-pwd hide-if-js">
    416416                                <?php $initial_password = wp_generate_password( 24 ); ?>
    417417                                <span class="password-input-wrapper">
  • user-edit.php

     
    463463<h3><?php _e('Account Management'); ?></h3>
    464464<table class="form-table">
    465465<tr id="password" class="user-pass1-wrap">
    466         <th><label for="pass1"><?php _e( 'New Password' ); ?></label></th>
     466        <th><label for="wp-generate-pw"><?php _e( 'New Password' ); ?></label></th>
    467467        <td>
    468468                <input class="hidden" value=" " /><!-- #24364 workaround -->
    469                 <button type="button" class="button button-secondary wp-generate-pw hide-if-no-js"><?php _e( 'Generate Password' ); ?></button>
     469                <button type="button" id="wp-generate-pw" class="button button-secondary wp-generate-pw hide-if-no-js"><?php _e( 'Generate Password' ); ?></button>
    470470                <div class="wp-pwd hide-if-js">
    471471                        <span class="password-input-wrapper">
    472472                                <input type="password" name="pass1" id="pass1" class="regular-text" value="" autocomplete="off" data-pw="<?php echo esc_attr( wp_generate_password( 24 ) ); ?>" aria-describedby="pass-strength-result" />