Open Bug 1598541 Opened 5 years ago Updated 2 years ago

Highlight the dismissed doorhanger icon shown when a password field is modified

Categories

(Toolkit :: Password Manager, enhancement, P3)

enhancement

Tracking

()

People

(Reporter: sfoster, Unassigned)

References

Details

(Whiteboard: [passwords:capture-UI])

When a password field is edited, and the dimissed doorhanger shown, the anchor element should get a blue "attention" highlight

This is currently driven by the notifySaved argument on the promptToShowPassword / promptToChangePassword LoginManagerPrompter methods, but that argument is also used to trigger the "saved" toast when we auto-save a login for generated passwords. Teasing these functions apart probably means updating the signature to use a jsval options object so we can pass separate flags as properties for attention, notifySaved, dismissed, etc.

(In reply to Sam Foster [:sfoster] (he/him) from comment #0)

When a password field is edited, and the dimissed doorhanger shown, the anchor element should get a blue "attention" highlight

Why? Isn't that going to conflate the meaning of the blue icon? Right now blue means a login was "automatically saved" but that wouldn't be the case here.

Blocks: 1536728
No longer depends on: 1536728
Priority: -- → P3
Whiteboard: [passwords:capture-UI]
Depends on: 1619851

The goal here is to highlight the new/changed dismissed doorhanger (the key icon in the address bar.)
Using the same blue attention color we use for notifying when an login has been auto-saved might be confusing, so some other visual treatment - a small animation or different highlight color - would be good.

Severity: normal → S3
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.