Closed Bug 1279586 Opened 4 years ago Closed 4 years ago
password manager notification access key
58 bytes, text/x-review-board-request
User Agent: Mozilla/5.0 (Android 6.0.1; Mobile; rv:50.0) Gecko/50.0 Firefox/50.0 Build ID: 20160608030219 Steps to reproduce: log in to something notification shows up about remembering password press alt+r Actual results: focuses drop down arrow next to "Remember" button Expected results: activates "Remember" button
This will likely be fixed by the redesign of the remember password prompt. Marking for fxprivacy so we can decide what to do with this bug.
Priority: -- → P3
Whiteboard: [passwords:capture-UI] → [passwords:capture-UI] [fxprivacy][triage]
Whiteboard: [passwords:capture-UI] [fxprivacy][triage] → [passwords:capture-UI] [fxprivacy]
Unfortunately this isn't yet fixed due to an access key conflict that went unnoticed. Patch incoming.
Assignee: nobody → past
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Comment on attachment 8813600 [details] Fix the access key conflict in the password manager prompt (bug 1279586). https://reviewboard.mozilla.org/r/95038/#review95246 Looks good and works for me, thanks!
Attachment #8813600 - Flags: review?(jhofmann) → review+
Pushed by email@example.com: https://hg.mozilla.org/integration/autoland/rev/966bed7e5627 Fix the access key conflict in the password manager prompt . r=johannh
For future reference, you don't need to change the string ID if you're fixing a conflict in English. Each locales has its own accesskeys, and should already have them sorted out. On the other hand, it would be great to use consistent naming to help tools. Bad: togglePasswordLabel=Show password togglePasswordAccessKey2=h Good: togglePassword.label=Show password togglePassword.accessKey=h
I have reproduced this bug with Nightly 50.0a1 (2016-06-10) on Windows 8 , 64 Bit! This bug's fix is verified with latest Beta! Build ID 20170320143328 User Agent Mozilla/5.0 (Windows NT 6.2; Win64; x64; rv:53.0) Gecko/20100101 Firefox/53.0 [bugday-20170322]
You need to log in before you can comment on or make changes to this bug.