[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