"Show passwords" button in preferences should be renamed

RESOLVED FIXED in Firefox 3 beta5

Status

()

RESOLVED FIXED
12 years ago
11 years ago

People

(Reporter: wgianopoulos, Assigned: Dolske)

Tracking

({late-l10n})

Trunk
Firefox 3 beta5
late-l10n
Points:
---
Bug Flags:
blocking-firefox3 -
wanted-firefox3 +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

12 years ago
Currently, to show the saved passwords you go to Tools -> Options -> Security then click a show passwords button which brings up a page that does not show passwords but has another button labeled Show passwords .  This is a bit klunky and confusing to the user.

I suggest changing the button label on the Tools -> Options -> Security page from "Show Passwords..." to "Saved logins..." or something similar.
Duplicate of this bug: 383628
OS: Windows XP → All
Hardware: PC → All
Summary: Show password option page should be renamed → "Show passwords" button in preferences should be renamed (Show Logins?)
beltzner: "Show Logins..." or "Saved Logins..." or something else?
Flags: blocking-firefox3?
Keywords: uiwanted
"Saved Passwords..." for the button.
Then we'll also need to change the string at the top of the password manager to say "Password Manager has saved the passwords for the following sites:"

Not a blocker, but I'll approve the patch.
Flags: blocking-firefox3? → blocking-firefox3-
Whiteboard: [wanted-firefox3]
(Assignee)

Comment 4

11 years ago
Created attachment 285645 [details] [diff] [review]
Patch for review, v.1

This changes the button, and I reworded the resulting window's title a spiel a bit to better match Cookies and similar dialogs. I thought about using "logins" instead of "passwords", which is technically more complete/correct, but the task emphasis here is really on "passwords", and that's more likely to catch one's eye.

I do note that we currently have "Show Cookies..." and "Show Passwords..." buttons. Should I change the cookies button to "Saved Cookies..." for consistency? [Phil Hartman's 'Anal Retentive UI Designer' character is very concerned about this problem.]
Assignee: nobody → dolske
Status: NEW → ASSIGNED
Attachment #285645 - Flags: ui-review?(beltzner)
Flags: wanted-firefox3+
Whiteboard: [wanted-firefox3]
Comment on attachment 285645 [details] [diff] [review]
Patch for review, v.1

No need to update the cookie button, since in that case we *are* showing the cookies; in this case we're just showing the list of saved passwords
Attachment #285645 - Flags: ui-review?(beltzner)
Attachment #285645 - Flags: ui-review+
Attachment #285645 - Flags: approval1.9+
Keywords: uiwanted → checkin-needed
Dolske can land his own patches.
Keywords: checkin-needed
(Assignee)

Comment 7

11 years ago
Entity names need revved, too, and I'd not sure is wallet/SM are using these strings or not.
(Assignee)

Comment 8

11 years ago
Whoops, this got dropped on the floor. Checked with beltzner, and we're still good to take this for B5. Updated patch coming up...
Keywords: late-l10n
Target Milestone: --- → Firefox 3 beta5
(Assignee)

Comment 9

11 years ago
Created attachment 308064 [details] [diff] [review]
Patch v.2

This changed the entity names, removes an unused string, and syncs back to trunk.
Attachment #285645 - Attachment is obsolete: true
Comment on attachment 308064 [details] [diff] [review]
Patch v.2

a=beltzner
Attachment #308064 - Flags: approval1.9+
(Assignee)

Comment 11

11 years ago
Checking in browser/components/preferences/security.xul;
  new revision: 1.23; previous revision: 1.22
Checking in browser/locales/en-US/chrome/browser/preferences/security.dtd;
  new revision: 1.9; previous revision: 1.8
Checking in browser/locales/en-US/chrome/help/prefs.xhtml;
  new revision: 1.60; previous revision: 1.59
Checking in toolkit/components/passwordmgr/content/passwordManager.js;
  new revision: 1.25; previous revision: 1.24
Checking in toolkit/components/passwordmgr/content/passwordManager.xul;
  new revision: 1.17; previous revision: 1.16
Checking in toolkit/components/passwordmgr/content/passwordManagerExceptions.xul;
  new revision: 1.2; previous revision: 1.1
Checking in toolkit/locales/en-US/chrome/passwordmgr/passwordManager.dtd;
  new revision: 1.7; previous revision: 1.6
Checking in toolkit/locales/en-US/chrome/passwordmgr/passwordmgr.properties;
  new revision: 1.18; previous revision: 1.17

Status: ASSIGNED → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → FIXED
Summary: "Show passwords" button in preferences should be renamed (Show Logins?) → "Show passwords" button in preferences should be renamed
You need to log in before you can comment on or make changes to this bug.