Closed Bug 1816988 Opened 1 year ago Closed 1 year ago

Consider enabling layout.forms.reveal-password-context-menu.enabled

Categories

(Firefox :: Menus, enhancement, P1)

enhancement

Tracking

()

VERIFIED FIXED
112 Branch
Tracking Status
relnote-firefox --- 112+
firefox112 --- verified

People

(Reporter: emilio, Assigned: emilio)

References

Details

Attachments

(2 files)

No description provided.

It seems there are no UX concerns or what not about this.

Pushed by ealvarez@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/c3539271beba
Enable reveal password context-menu. r=sgalich
Status: NEW → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 112 Branch

I think we could have a release note about it as this is exposing a new useful feature in the UI, emilio would you have a suggested wording? Thanks

Type: defect → enhancement
Flags: needinfo?(emilio)

Something like this?

Release Note Request (optional, but appreciated)
[Why is this notable]: Nice user-exposed feature.
[Affects Firefox for Android]: No
[Suggested wording]: Right-clicking on password fields now shows an option to reveal the password.

relnote-firefox: --- → ?
Flags: needinfo?(emilio)

Looks good to me thanks! Note added to our nightly release notes.

Severity: -- → S3
Priority: -- → P1
Flags: qe-verify+
Attached image password_contextr.gif

Verified that layout.forms.reveal-password-context-menu.enabled pref is set to true in Firefox 112.0 on Windows 10x64, macOS 11, and Ubuntu 20.04. Also, the Reveal Password context menu option is displayed inside the password field for saved and not saved passwords while they are masked and works accordingly when used by showing/masking passwords.

However, I noticed that are some websites that already have this function and it seems that they don't communicate with each other. For example, when using the context menu option to unmask the password the webpage's unmask password does not change and the button does not function anymore. Is this expected or should we file an issue for this? Thank you!

Flags: needinfo?(emilio)

That is expected

Flags: needinfo?(emilio)

(In reply to Emilio Cobos Álvarez (:emilio) from comment #8)

That is expected

Thank you!

Status: RESOLVED → VERIFIED
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: