Don't close the login autocomplete popup when the search string becomes empty
Categories
(Toolkit :: Password Manager, enhancement, P2)
Tracking
()
People
(Reporter: MattN, Assigned: MattN)
References
(Blocks 1 open bug)
Details
(Keywords: parity-chrome, parity-safari, Whiteboard: [passwords:fill-ui])
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
Since the initial landing of satchel/autocomplete, when the field text becomes empty, the popup closes, even if there are relevant results to show. This may make sense for the awesomebar(?) but it doesn't make sense as much sense for logins fields where the initial value may have been populated from the site and the user could greatly benefit from login suggestions immediately after clearing the field.
The two main cases when showing the popup upon clearing the field value:
- Show the full list of logins available to fill (or the View Saved Logins footer)
- Showing the insecure login field warning
It seems like Chrome and Safari only have this behaviour for login fields, not <input list="…">
or for address autofill so we should probably do the same. We should also leave the address bar behaviour alone so we should probably add an attribute on nsIAutoCompleteInput
to control this. nsFormFillController can then return a boolean depending on whether the focused field is marked as a login field: mPwmgrInputs.Get(…)
.
Assignee | ||
Updated•6 years ago
|
Assignee | ||
Comment 1•6 years ago
|
||
Assignee | ||
Updated•6 years ago
|
Assignee | ||
Updated•6 years ago
|
Comment 3•6 years ago
|
||
bugherder |
Comment 4•6 years ago
|
||
Matt, could you confirm that the change here to verify is that the saved logins pop-up remains open after the text clear of the field? Does this apply to insecure scenario, as well?
Assignee | ||
Comment 5•6 years ago
|
||
(In reply to Adrian Florinescu [:adrian_sv] from comment #4)
Matt, could you confirm that the change here to verify is that the saved logins pop-up remains open after the text clear of the field?
Yes
Does this apply to insecure scenario, as well?
Yes, I believe it should.
Assignee | ||
Updated•6 years ago
|
Comment 6•6 years ago
|
||
Verified as fixed on Nightly 68.0a1 / 2019-03-28 Mac 10.13, Windows 10, Ubuntu 16.04.
Also checked on 67.0b5 2019-03-25 Win 10/Ubuntu 16.04 turning "signon.showAutoCompleteFooter" pref. on, the behavior is as expected.
The above applies for both the secure and insecure scenarios.
Description
•