WordPress.org

Make WordPress Core

Ticket #24633: 24633.13.patch

File 24633.13.patch, 8.4 KB (added by michalzuber, 4 years ago)

Cursor pointer for icon, separate table row for repeat password, password show/hide icon visible always to toggle password visibility

  • src/wp-admin/admin-ajax.php

     
    6060        'wp-remove-post-lock', 'dismiss-wp-pointer', 'upload-attachment', 'get-attachment', 
    6161        'query-attachments', 'save-attachment', 'save-attachment-compat', 'send-link-to-editor', 
    6262        'send-attachment-to-editor', 'save-attachment-order', 'heartbeat', 'get-revision-diffs', 
    63         'save-user-color-scheme', 'update-widget', 'query-themes', 'parse-embed', 'set-attachment-thumbnail' 
     63        'save-user-color-scheme', 'generate_password', 'update-widget', 'query-themes', 'parse-embed', 'set-attachment-thumbnail' 
    6464); 
    6565 
    6666// Register core Ajax calls. 
  • src/wp-admin/css/forms.css

     
    451451        border-color: #8dff1c; 
    452452} 
    453453 
    454 .indicator-hint { 
    455         padding-top: 8px; 
     454#password2 .indicator-hint { 
     455        padding: 4px 0 0 220px; 
    456456} 
    457457 
    458458p.search-box { 
     
    651651        border: none; 
    652652} 
    653653 
    654 .color-option { 
     654.color-option, 
     655.pass1-password-visibility { 
    655656        cursor: pointer; 
    656657} 
    657658 
     659.pass1-password-visibility { 
     660        line-height: 25px; 
     661        margin-right: 10px; 
     662} 
     663 
     664.pass1-password-visibility.show { 
     665        color: #666; 
     666} 
     667 
     668.pass1-password-visibility.hide { 
     669        color: #000; 
     670} 
     671 
    658672/*------------------------------------------------------------------------------ 
    659673  19.0 - Tools 
    660674------------------------------------------------------------------------------*/ 
     
    948962                padding: 0; 
    949963                line-height: 2; 
    950964        } 
     965 
     966        #password2 .indicator-hint { 
     967                padding-left: 0; 
     968        } 
    951969} 
    952970 
    953971@media only screen and (max-width: 768px) { 
  • src/wp-admin/includes/ajax-actions.php

     
    26662666 
    26672667        wp_send_json_success( $parsed ); 
    26682668} 
     2669 
     2670/** 
     2671 * Generates a password via ajax 
     2672 * 
     2673 * @since 4.0.0 
     2674 */ 
     2675function wp_ajax_generate_password() { 
     2676        die( wp_generate_password() ); 
     2677} 
     2678 No newline at end of file 
  • src/wp-admin/includes/user.php

     
    174174 
    175175        if ( $update ) { 
    176176                $user_id = wp_update_user( $user ); 
     177 
     178                // Encourage the user to reset their password 
     179                if( ! empty( $_POST['reset_password'] ) ) { 
     180                        update_user_option( $user_id, 'default_password_nag', true, true ); 
     181                } 
     182 
     183                // Send the new, plaintext password to the user 
     184                if( ! empty( $_POST['send_password'] ) ) { 
     185                        $blogname = wp_specialchars_decode( get_option( 'blogname' ), ENT_QUOTES ); 
     186                        $message  = sprintf( __( 'Username: %s' ), $user->user_login ) . "\r\n"; 
     187                        $message .= sprintf( __( 'Password: %s' ), $pass1 ) . "\r\n"; 
     188                        $message .= wp_login_url() . "\r\n"; 
     189 
     190                        wp_mail( $user->user_email, sprintf( __( '[%s] Your username and password' ), $blogname ), $message ); 
     191                } 
     192 
    177193        } else { 
    178194                $user_id = wp_insert_user( $user ); 
    179195                wp_new_user_notification( $user_id, isset( $_POST['send_password'] ) ? wp_unslash( $pass1 ) : '' ); 
  • src/wp-admin/js/user-profile.js

     
    3434                var $colorpicker, $stylesheet, user_id, current_user_id, 
    3535                        select = $( '#display_name' ); 
    3636 
    37                 $('#pass1').val('').keyup( check_pass_strength ); 
    38                 $('#pass2').val('').keyup( check_pass_strength ); 
     37                /* Passwords */ 
     38 
     39                var visibilityIconShow = $( '.pass1-password-visibility.show' ), 
     40                        visibilityIconHide = $( '.pass1-password-visibility.hide' ), 
     41                        pass1 = $( '#pass1' ), 
     42                        pass2 = $( '#pass2' ), 
     43                        password_repeat = $( '.password-repeat' ); 
     44 
     45                pass1.val('').keyup(function(e) { 
     46                        check_pass_strength(); 
     47                        changed = true; 
     48 
     49                        if ( 'text' == pass1.attr('type') ) { 
     50                                pass2.attr('type', 'text'); 
     51                        } 
     52                        password_repeat.show(); 
     53                }); 
     54                pass2.val('').keyup( check_pass_strength ); 
     55                visibilityIconShow.show(); 
     56 
     57                $( '#generate-password' ).on('click', function() { 
     58                        if ( 'profile' == pagenow ) { 
     59                                // User's own password 
     60                                pass1.attr('type', 'text'); 
     61                                pass2.attr('type', 'text'); 
     62                        } else { 
     63                                pass1.attr('type', 'password'); 
     64                                pass2.attr('type', 'password'); 
     65                        } 
     66                        $.post( ajaxurl, { action: 'generate_password' }, function( response ) { 
     67                                pass1.val( response ).trigger('keyup'); 
     68                                pass2.val( response ).trigger('keyup'); 
     69                                if( 'profile' == pagenow ) { 
     70                                        // User's own password 
     71                                        visibilityIconHide.show(); 
     72                                } else { 
     73                                        visibilityIconHide.hide(); 
     74                                } 
     75                                pass1.focus(); 
     76                                password_repeat.hide(); 
     77                        }); 
     78                }); 
     79                visibilityIconShow.on('click', function(e) { 
     80                        visibilityIconShow.hide(); 
     81                        visibilityIconHide.show(); 
     82                        e.preventDefault(); 
     83                        pass1.attr('type', 'text'); 
     84                        pass2.attr('type', 'text'); 
     85                }); 
     86                visibilityIconHide.on('click', function(e) { 
     87                        visibilityIconHide.hide(); 
     88                        visibilityIconShow.show(); 
     89                        e.preventDefault(); 
     90                        pass1.attr('type', 'password'); 
     91                        pass2.attr('type', 'password'); 
     92                }); 
     93 
    3994                $('#pass-strength-result').show(); 
     95 
     96                /* End Passwords */ 
     97 
    4098                $('.color-palette').click( function() { 
    4199                        $(this).siblings('input[name="admin_color"]').prop('checked', true); 
    42100                }); 
  • src/wp-admin/user-edit.php

     
    459459        <th><label for="pass1"><?php _e( 'New Password' ); ?></label></th> 
    460460        <td> 
    461461                <input class="hidden" value=" " /><!-- #24364 workaround --> 
    462                 <input type="password" name="pass1" id="pass1" class="regular-text" size="16" value="" autocomplete="off" /><br /> 
    463                 <span class="description"><?php _e( 'If you would like to change the password type a new one. Otherwise leave this blank.' ); ?></span> 
    464         </td> 
     462                <input type="password" name="pass1" id="pass1" class="regular-text" value="" autocomplete="off" /> 
     463                <span class="dashicons dashicons-visibility pass1-password-visibility hidden show" title="<?php _e( 'Show Password' ); ?>"></span> 
     464                <span class="dashicons dashicons-visibility pass1-password-visibility hidden hide" title="<?php _e( 'Hide Password' ); ?>"></span> 
     465                <input type="button" name="generate-password" id="generate-password" value="<?php _e( 'Generate Password' ); ?>" class="button hide-if-no-js" /> 
     466        </td> 
    465467</tr> 
    466 <tr> 
    467         <th scope="row"><label for="pass2"><?php _e( 'Repeat New Password' ); ?></label></th> 
     468<tr id="password2"> 
     469        <th><label class="password-repeat hidden" for="pass2"><?php _e( 'Repeat New Password' ); ?></label></th> 
    468470        <td> 
    469         <input name="pass2" type="password" id="pass2" class="regular-text" size="16" value="" autocomplete="off" /><br /> 
    470         <span class="description" for="pass2"><?php _e( 'Type your new password again.' ); ?></span> 
    471         <br /> 
    472         <div id="pass-strength-result"><?php _e( 'Strength indicator' ); ?></div> 
    473         <p class="description indicator-hint"><?php _e( 'Hint: The password should be at least seven characters long. To make it stronger, use upper and lower case letters, numbers, and symbols like ! " ? $ % ^ &amp; ).' ); ?></p> 
     471                <div class="password-repeat hidden"> 
     472                        <input name="pass2" type="password" id="pass2" class="regular-text" value="" autocomplete="off" /> 
     473                        <br /> 
     474                        <div id="pass-strength-result"><?php _e( 'Strength indicator' ); ?></div> 
     475                        <p class="description indicator-hint"><?php _e( 'Hint: The password should be at least seven characters long. To make it stronger, use upper and lower case letters, numbers and symbols like ! " ? $ % ^ &amp; ).' ); ?></p> 
     476                </div> 
     477                <p class="description"><?php _e( 'If you would like to change the password type or generate a new one. Otherwise leave this blank.' ); ?></p> 
     478                <?php if( ! IS_PROFILE_PAGE ) { ?> 
     479                <p class="hide-if-no-js"> 
     480                        <label for="send_password"> 
     481                                <input type="checkbox" name="send_password" id="send_password" value="1" /> <?php _e( 'Send this password to the user by email.' ); ?> 
     482                        </label> 
     483                        <br /> 
     484                        <label for="reset_password"> 
     485                                <input type="checkbox" name="reset_password" id="reset_password" value="1" /> <?php _e( 'Encourage the user to change their password, once logged in.' ); ?> 
     486                        </label> 
     487                </p> 
     488                <?php } ?> 
    474489        </td> 
    475490</tr> 
    476491<?php endif; ?>