Need better accesskey for show password context menu.
Categories
(Core :: Layout: Form Controls, defect)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox98 | --- | fixed |
People
(Reporter: emilio, Assigned: emilio)
References
(Blocks 1 open bug)
Details
Attachments
(2 files)
All letters are taken:
- S for "Suggest Strong Password"
- h for "This frame"
- o for "use saved password / use saved login"
- w for "switch text direction"
- P for "Paste"
- a for "Select All"
- r for "Redo"
- d for "Delete"
Any good ideas? Should we change the context menu entry to use maybe "Reveal password" or something? Maybe "v" or "e" aren't used.
|
||
Comment 1•2 years ago
|
||
(In reply to Emilio Cobos Álvarez (:emilio) from comment #0)
Maybe "v" or "e" aren't used.
They're not in any items that show up when using the context menu on a password field, so I think this would be OK...
We could also do a "change the access keys" dance with the password specific items... "g" seems unused and we could use that for "suggest strong password", and then use "S" for "Show Password". Or talk to #content on slack for alternative naming / access key suggestions.
|
||
Comment 2•2 years ago
|
||
The severity field is not set for this bug.
:dholbert, could you have a look please?
For more information, please visit auto_nag documentation.
|
||
Updated•2 years ago
|
|
Assignee | |
Comment 3•2 years ago
|
||
The pseudo-class and nsContextMenu context attribute were using reveal,
the pseudo-element and webidl attribute were using "show".
Use reveal consistently and update the accesskey so that there aren't
conflicts with existing commands. Also enable the feature in
browser_contextmenu_input.js so that this change is tested.
|
||
Updated•2 years ago
|
|
|
Assignee | |
Comment 4•2 years ago
|
||
The reveal password button might or might not be web-compatible, but it
might be worth having a separate pref for the context-menu entry.
Depends on D136086
Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/fab0ca85e81c Use "reveal" rather than show terminology consistently for the reveal password button. r=Gijs,fluent-reviewers https://hg.mozilla.org/integration/autoland/rev/51400fba807a Split reveal-password and button and context-menu prefs. r=Gijs
|
||
Comment 6•2 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/fab0ca85e81c
https://hg.mozilla.org/mozilla-central/rev/51400fba807a
Description
•