Closed Bug 1212436 Opened 9 years ago Closed 8 years ago

Don't show password automatically when field gets focus in 'Remember this password' doorhanger

Categories

(Firefox :: General, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: jdiggs, Unassigned)

Details

Steps to reproduce:
1. Fill out a web form with a password input
2. Press submit button
3. Tab amongst the controls in the resulting doorhanger

Expected results: The password text, which was obscured by bullet characters, would not be displayed as plain text upon focus.

Actual results: The password text becomes displayed as plain text automatically upon focus.

Impact: If a keyboard user (happens to be blind and/or cannot use a mouse) Tabs to explore the contents of that doorhanger and/or is trying to move focus to the Remember push button, that user's password is displayed to whoever is walking by -- and spoken allowed in the case of a screen reader user who is not using headphones.

This is probably unexpected by, and undesirable to, a sufficient number of users that I feel considering an alternative behavior is warranted.
Brian, I noticed you got looped in with the control center stuff. Can you redirect this to someone who can do the requisite UI/UX design so we stop inadvertently showing (vision-impaired) keyboard users' passwords?
Flags: needinfo?(bgrinstead)
(In reply to :Gijs Kruitbosch from comment #1)
> Brian, I noticed you got looped in with the control center stuff. Can you
> redirect this to someone who can do the requisite UI/UX design so we stop
> inadvertently showing (vision-impaired) keyboard users' passwords?

Sure, forwarding the request to Aislinn
Flags: needinfo?(bgrinstead) → needinfo?(agrigas)
(In reply to Brian Grinstead [:bgrins] from comment #2)
> (In reply to :Gijs Kruitbosch from comment #1)
> > Brian, I noticed you got looped in with the control center stuff. Can you
> > redirect this to someone who can do the requisite UI/UX design so we stop
> > inadvertently showing (vision-impaired) keyboard users' passwords?
> 
> Sure, forwarding the request to Aislinn

I think this is Ryan Feeley's area. NIing him.
Flags: needinfo?(agrigas) → needinfo?(rfeeley)
The team agreed to move to a more conventional show password checkbox which is reflected in this UX spec:

https://www.lucidchart.com/documents/view/87ab1cc8-e708-49d3-8b91-6e2e6da346fb/1

Is the bug for the work to do this missing?
Flags: needinfo?(rfeeley) → needinfo?(MattN+bmo)
This appears to be fixed in Nightly at least.
Status: NEW → RESOLVED
Closed: 8 years ago
Flags: needinfo?(MattN+bmo)
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.