[wp-trac] [WordPress Trac] #42852: Add new user: new password not generated when reopening Show Password

WordPress Trac noreply at wordpress.org
Sat Feb 15 23:44:37 UTC 2020

#42852: Add new user: new password not generated when reopening Show Password
 Reporter:  afercia                              |       Owner:  (none)
     Type:  defect (bug)                         |      Status:  new
 Priority:  normal                               |   Milestone:  5.4
Component:  Users                                |     Version:  4.7
 Severity:  normal                               |  Resolution:
 Keywords:  has-patch has-screenshots needs-     |     Focuses:  javascript
  design                                         |

Comment (by bookdude13):

 My last refresh was more focused on code porting than functionality...so
 here's another attempt :)
 - This should match the four bullets mentioned by @afercia
 - For creating new users, changed it so the fields are never hidden. This
 also allows for a value to always be set and visible for the password when
 creating users. Since it's always visible I have the text set to "Generate
 New Password"
 - It still needs to be tested that passwords can't be set even if they
 were never visible, e.g. an edit profile/user page updating the password
 by accident when something else updates
 - I have not tested on other browsers or mobile, just Chrome.

 This still needs design review for the question of how to handle the
 Cancel button and where to put the Set New Password button (above or

 Replying to [comment:13 bookdude13]:
 > Patch refreshed. A few suggestions to improve (needs design review):
 > - When editing a user (or your own profile) the "Set New Password"
 button text could change to "Generate New Password" once the field
 > - When editing a user (or your own profile) the "Cancel" button could
 hide all of the fields, making the "Set New Password" button act as a
 toggle for their appearance
 > - There is not a way to generate new strong passwords during
 > - It should be checked that #42853 didn't get reverted in this. The code
 that was changed from there were removed from the original patch here.
 > - Related: #42766

Ticket URL: <https://core.trac.wordpress.org/ticket/42852#comment:14>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform

More information about the wp-trac mailing list