Show the dismissed doorhanger upon filling a generated password
Categories
(Toolkit :: Password Manager, enhancement, P2)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox70 | --- | verified |
People
(Reporter: MattN, Assigned: sfoster)
References
Details
(Whiteboard: [passwords:capture-UI] [passwords:generation] [skyline])
Attachments
(4 files)
Actual result
To:
- ensure that the user can always add their username after saving a generated password
- ensure that the user can manually save the generated password in cases where we don't auto-save it
We need to show the capture doorhanger (including a potential username) to allow the user to save/update the username and/or password.
|
Assignee | |
Updated•5 years ago
|
|
Reporter | |
Comment 1•5 years ago
|
||
Implementation order we discussed:
Part 1: Show doorhanger without username
Part 2: Include the username from content
|
|
Assignee | |
Comment 2•5 years ago
|
||
|
|
Assignee | |
Comment 3•5 years ago
|
||
Depends on D36859
|
|
Assignee | |
Comment 4•5 years ago
|
||
Depends on D36860
|
|
Assignee | |
Comment 5•5 years ago
|
||
|
|
Reporter | |
Comment 6•5 years ago
|
||
(In reply to Sam Foster [:sfoster] (he/him) from comment #5)
Try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=8b8204f7edb39352b6c155de227a811b47b74b39
I was going to land this for you but it looks like there are test failures.
|
|
Assignee | |
Comment 7•5 years ago
|
||
Pushed by sfoster@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/ab17d02d2093 Part 1: Share the doFillGeneratedPasswordContextMenuItem test helper. r=MattN https://hg.mozilla.org/integration/autoland/rev/540908f19ed1 Part 2: Create dismissed prompt when filling with a generated password login. r=MattN https://hg.mozilla.org/integration/autoland/rev/49509781bc6b Part 3: Include username in save-password prompt when filling with generated password. r=MattN
|
||
Comment 9•5 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/ab17d02d2093
https://hg.mozilla.org/mozilla-central/rev/540908f19ed1
https://hg.mozilla.org/mozilla-central/rev/49509781bc6b
|
|
Reporter | |
Updated•5 years ago
|
|
||
Comment 10•5 years ago
|
||
Hey Sam,
After filling in the password field with the generator and without any username the password will be auto-saved.
Tapping the blue key icon will display the doorhanger with pre-filled password and with a placeholder for the user name (as expected).
However, the doorhanger string will be: "Would you like to update this password?"
Is this correct? Shouldn't it be to update the login info for the given site?
Screenshot attached for reference.
|
|
||
Comment 11•5 years ago
|
||
|
|
Assignee | |
Comment 12•5 years ago
|
||
(In reply to Timea Babos from comment #10)
After filling in the password field with the generator and without any username the password will be auto-saved.
Tapping the blue key icon will display the doorhanger with pre-filled password and with a placeholder for the user name (as expected).However, the doorhanger string will be: "Would you like to update this password?"
Is this correct? Shouldn't it be to update the login info for the given site?
It is using the correct string, but I agree that in this context we are really offering the opportunity where arguably "Would you like to update this login" might make more sense than "Would you like to update this password?" That would need to be filed as a separate bug though.
|
|
||
Comment 13•5 years ago
|
||
Thank you Sam!
Verified - Fixed on latest Nightly 70.0a1 (2019-07-29) (64-bit) on Windows 10, MacOS 10.14 and Ubuntu 18.04.
The dismissed doorhanger can be accessed via the key icon after filling in a generated password:
- "No username" placeholder can be seen for the empty username field
- Filling the username on the form and submitting will trigger once more the doorhanger to update the login information
- The username and password can be edited in both the above-mentioned doorhangers
Submitted Bug 1569917 for the string discussed in Comment 12.
Description
•