[wp-trac] [WordPress Trac] #42888: Add a "Show" button next to password fields on mobile

WordPress Trac noreply at wordpress.org
Sat Jun 9 08:45:22 UTC 2018


#42888: Add a "Show" button next to password fields on mobile
-------------------------------------+-------------------------------------
 Reporter:  johnbillion              |       Owner:  Iceable
     Type:  enhancement              |      Status:  assigned
 Priority:  normal                   |   Milestone:  5.0
Component:  Login and Registration   |     Version:
 Severity:  normal                   |  Resolution:
 Keywords:  good-first-bug has-      |     Focuses:  accessibility,
  patch dev-feedback has-            |  javascript
  screenshots has-ux-feedback 2nd-   |
  opinion ux-feedback                |
-------------------------------------+-------------------------------------
Changes (by Iceable):

 * keywords:  good-first-bug has-patch dev-feedback has-screenshots has-ux-
     feedback =>
     good-first-bug has-patch dev-feedback has-screenshots has-ux-feedback
     2nd-opinion ux-feedback


Comment:

 Thank you very much @boemedia for taking the time to try it!

 I thought this patch would need a refresh as it is already 4 months old,
 though it still seems to apply and work correctly on my end. I just re-
 tested it on a Debian machine and on a MacOS one, both with a brand new
 VVV install.

 Steps taken:
 - Checked-out the latest commit from git://develop.git.wordpress.org/ on
 master (d3014a at the time of writing)
 - `$ grunt patch:42888`, then chose 42888-4.diff
 - `$ grunt build`

 Here is a screencap of wp-login.php in Chromium 66.0.3359.117 (Developer
 Build) on Debian 9.4.

 [[Image(https://i.imgur.com/NLLAzeq.gif)]]

 It also looks and works the same for me in Chrome 66.0.3359.181 (Official
 build) on MacOS.

 Not sure whether there is something wrong with this patch that doesn't
 show on my end, or if something went wrong during your test.

 Additional tests and/or 2nd opinion are most welcome.

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/42888#comment:19>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list