[wp-trac] [WordPress Trac] #3534: Hide password in setup-config.php

WordPress Trac noreply at wordpress.org
Tue Jan 3 20:34:00 UTC 2023


#3534: Hide password in setup-config.php
-------------------------------------------------+-------------------------
 Reporter:  xmarcos                              |       Owner:  rob1n
     Type:  enhancement                          |      Status:  reopened
 Priority:  low                                  |   Milestone:  6.2
Component:  Upgrade/Install                      |     Version:  2.1
 Severity:  normal                               |  Resolution:
 Keywords:  has-patch needs-testing has-         |     Focuses:
  testing-info has-screenshots                   |  accessibility
-------------------------------------------------+-------------------------
Changes (by ironprogrammer):

 * keywords:  has-patch needs-testing has-testing-info => has-patch needs-
     testing has-testing-info has-screenshots


Comment:

 Thanks @bgoewert! I think this is almost there.

 == Test Report
 Patch tested: https://github.com/WordPress/wordpress-
 develop/pull/3725.diff (applied to `trunk`, testing based on instructions
 from comment:40)

 === Environment
 - Hardware: MacBook Pro Apple M1 Pro
 - OS: macOS 12.6.1
 - Browser: Safari 16.1, Google Chrome 108.0.5359.124, Mozilla Firefox
 107.0.1
 - Server: nginx/1.23.2
 - PHP: 7.4.33
 - WordPress: 6.2-alpha-54642-src
 - Password Manager: 1Password 7 v7.9.8

 === Actual Results
 - ✅ Password field shows "password" placeholder, defaults to `input
 type="password"`, and "open eye" icon is displayed.
 - ✅ "Eye" icon is not covered by 1Password 7 suggestion toggle icon in
 either the hidden or visible state.
 - ✅ Toggle functions as expected, hiding or showing the password.
 - ⚠️ In Chrome and Firefox on macOS, the toggle button's `:focus` outline
 is activated when clicked, which is visually different from Safari. ''See
 Figure 4.''
 - ⚠️ In RTL languages, the "eye" toggle shifts to the left of the field,
 separate from the 1Password toggle (compare this to RTL results in
 https://core.trac.wordpress.org/ticket/9883#comment:44 where the toggles
 remain grouped). ''See Figure 5.''

 === Supplemental Artifacts

 [[Image(https://cldup.com/ki9Fy_Dpfg.png, 80%)]]
 ''Figure 1: Default Password field state.''

 [[Image(https://cldup.com/1_eLv8G6lh.png, 80%)]]
 ''Figure 2: Field focused with password text entered.''

 [[Image(https://cldup.com/j-Iv1XKM40.png, 80%)]]
 ''Figure 3: Field focused with password visibility toggled on (Safari).''

 [[Image(https://cldup.com/qclOblawBW.png, 30%)]]
 ''Figure 4: Toggle's `:focus` is activated when clicked (macOS Chrome and
 Firefox).''

 [[Image(https://cldup.com/g-rnFLDQU7.png, 80%)]]
 ''Figure 5: RTL toggle placement separated from password app icon.''

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


More information about the wp-trac mailing list