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

WordPress Trac noreply at wordpress.org
Wed Jun 20 08:08:55 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 2nd-opinion ux-        |
  feedback                           |
-------------------------------------+-------------------------------------
Changes (by boemedia):

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


Comment:

 Me & @Iceable briefly sat together during WCEU contributor day. We checked
 this on both mine (MAMP) and his (VVV) local environment, but there was a
 difference in display. We couldn't quite replicate what caused the display
 being wrong on my install (we did use grunt build) because of time and
 wifi issues, I have to see if I can get VVV installed on my computer and
 test again. Maybe someone else can test on MAMP and see if they can
 replicate my issue?

 Replying to [comment:20 adamsilverstein]:
 > @boemedia can you please give this another test following the steps
 @Iceable outlined? notably, use grunt patch and grunt build to make sure
 the patch applies to the correct code and is built into the build
 directory.. Also, since https://core.trac.wordpress.org/changeset/43309
 you'll need to be testing by running WordPress from the build directory as
 well.
 >
 > Thanks!

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


More information about the wp-trac mailing list