Closed Bug 1739822 Opened 4 years ago Closed 3 years ago

Please migrate the "Primary Password" prompt to Proton / in-content modal

Categories

(Toolkit :: Password Manager, enhancement, P3)

Firefox 96
enhancement

Tracking

()

RESOLVED DUPLICATE of bug 1714215
Tracking Status
firefox96 --- affected

People

(Reporter: ronjouch, Unassigned)

References

(Blocks 1 open bug)

Details

Attachments

(2 files)

Proton brought one change I like particularly: in-content modals (e.g. screenshot/A, Close window and quit )

I like them because they guarantee blocking visibly, comes with a helpful greying of the content, and cannot be moved away accidentally.

Contrarily, I'm frequently annoyed by the Password Required - Please enter your Primary Password. - [Cancel] [Sign In] modal, which wasn't migrated to a Proton / in-content modal and remains a "traditional" window modal (see screenshot/B). I find it annoying because:

  1. Under some circumstances unknown to me, it is sometimes seen as a separate window by my desktop (GNOME Shell), but sometimes not. So, it's not always Alt-Tabbable to.
  2. Some window management things I seem to do will sometimes cause the modal to be backgrounded (and un-findable, visually or in alt-tab or in GNOME's "Activities" menu), leaving me with a Firefox window that appears functional, but is actually unresponsive to all input, requiring me to kill it :-/
  3. And of course, from a UX consistency standpoint, it's weird to have other modals migrated but not this one.

Are there any reasons to omit this particular modal from being migrated to in-content/Proton? Can it be migrated?

Note: please re-triage to the right Firefox Component, I added this bug to the closest Component I could find, but it's probably not great.

Attached image website-login

a password promt in this design already exists. but only for website-requested logins.
i would appreciate having this. much better than having a seperate pop-up-window

(In reply to emsy from comment #1)

a password promt in this design already exists. but only for website-requested logins.
i would appreciate having this. much better than having a seperate pop-up-window

Thanks, emsy. Indeed, I'd love to see this design used for the Primary Password prompt.

And by the way, another annoyance of the current traditional prompt, to add to the list I wrote above:

  • The Primary Password prompt pops up "asynchronously", typically easily a few seconds after Firefox launching, and steals focus. So, suppose I launch Firefox, then alt-tab to another window to type something, e.g. a chat window where I start typing to a friend. Then, when the Primary Password prompt shows up, it'll steal focus and I'll be typing inside the password box, undesirably stealing input I wanted to type to my friend in the chat program, not as a password. Super annoying.

Thanks for opening this enhancement, I checked the design on win10 and MacOS 10.15 with latest nightly 96.0a1 by going to about:logins after enabled primary password,

i will set it as new for the devs to take a look at it.

Status: UNCONFIRMED → NEW
Ever confirmed: true

Bug reporter here. A quick note that this became less of an issue since a few days ago (circa Nightly 99.0a1 or 100.0a1 ~ early March 2022), when the traditional Please enter your Primary Password started to pop immediately on Firefox launch, mitigating a lot of the annoyances I listed above.

As such, leaving open but feel free to close as WONTFIX. Thanks for Firefox.

This came up in Desktop Integrations testing and doesn't appear to have been triaged; can we get it back on the radar with an appropriate severity, etc? Thanks!

Component: Toolbars and Customization → Untriaged
Component: Untriaged → Password Manager
Product: Firefox → Toolkit

Thanks for moving it to the Password Manager component, we are considering it!

Flags: needinfo?(sgalich)
Flags: needinfo?(sgalich)
Severity: -- → S3
Priority: -- → P3
Status: NEW → RESOLVED
Closed: 3 years ago
Duplicate of bug: 1714215
Resolution: --- → DUPLICATE
No longer blocks: masterpassword
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: