Closed Bug 1612258 Opened 1 year ago Closed 8 months ago

Show/adjust the dismissed login capture doorhanger in response to input events in the password & username field

Categories

(Toolkit :: Password Manager, enhancement, P2)

enhancement

Tracking

()

VERIFIED FIXED
mozilla80
Tracking Status
firefox80 --- verified

People

(Reporter: sfoster, Assigned: bdanforth)

References

(Blocks 1 open bug, Regressed 2 open bugs)

Details

(Keywords: parity-chrome, Whiteboard: [passwords:capture-UI] )

Attachments

(1 file)

Bug 1536728 uses changes events to show a dismissed login capture doorhanger when the user edits a password field. If we use input events the doorhanger can be shown and updated in real-time to user input.

Blocks: 1536728
No longer depends on: 1536728
Priority: -- → P2
Blocks: 1629226
Assignee: nobody → bdanforth
Status: NEW → ASSIGNED
Attachment #9159769 - Attachment description: Bug 1612258 - Show/adjust the dismissed login capture doorhanger in response to input events in the password & username field → Bug 1612258 - Show/adjust the dismissed login capture doorhanger in response to input events in the password & username field. r=MattN
Pushed by malexandru@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/2e37d9851441
Show/adjust the dismissed login capture doorhanger in response to input events in the password & username field. r=MattN,severin
Status: ASSIGNED → RESOLVED
Closed: 8 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla80

QA verification notes

Previous behavior
The dismissed doorhanger would not show up when editing a password input field until the field was blurred; i.e. it only showed up on "change"1 events.

New behavior
The dismissed doorhanger shows up when the field is edited, without requiring the field to be blurred; i.e. it shows on "input"2 events.

Unchanged behavior
As before, we expect the values of the username and password field (when the password field is non-empty) in the update/save doorhanger to match the values in the form.

Autosave of the initial fill and subsequent edits of generated passwords should also work as before; i.e. editing a generated password field will get autosaved on "change" events.

Known bugs that are out of scope

  • Bug 1612257 (i.e. the behavior of the doorhanger when an edit clears the password field after a dismissed doorhanger was previously shown).
    • This should just behave the same way it did before this patch.
Flags: qe-verify+

Verified-fixed on the latest Firefox Nightly 80.0a1 (2020-07-14) (64-bit) on Windows 10, MacOS 10.13 and Ubuntu 16.04.

  • the grey key icon is displayed right after starting to write in the password field and the values are captured and reflected in the dismissed doorhanger based on user input. There is no need to focus out of the field anymore as the updates are auto-made and correctly captured/displayed in the doorhanger
  • the same behavior can be observed for editing filled in fields, dismissed doorhanger appears when the username or password is edited
  • there are no changes or problems observed for password generation's save on the fly aspect
  • Bug 1612257 is still reproducible and acknowledged as known issue
Status: RESOLVED → VERIFIED
Flags: qe-verify+
Regressions: 1653945
Regressions: 1653971
Regressions: 1654466
Blocks: 1638587
Regressions: 1659655
No longer regressions: 1659655
Regressions: 1690996
You need to log in before you can comment on or make changes to this bug.