1697484 - Update copy for password panels

Status: RESOLVED FIXED

(Toolkit :: Password Manager, task, P3)

Description

[Mike Conley (:mconley) (:⚙️)]

The strings for the password panel are in https://searchfox.org/mozilla-central/rev/2b99ea2e97eef00a8a1c7e24e5fe51ab5304bc42/toolkit/locales/en-US/chrome/passwordmgr/passwordmgr.properties#5-47

Do not port these strings to Fluent - PopupNotifications, which powers the password panels, is pretty tightly coupled to how .properties files work, so per flod, we're going to bump porting these to Fluent for now.

Note that the string IDs will need to be bumped. Also note that if string interpolation changes (like for saveLoginMsg and saveLoginMsgNoUser) with insertion points being removed, that you'll need to update the code that uses that string to pass the correct formatted string arguments in.

We know we want to change:

• saveLoginMsg should be changed to Save login for %S?
• saveLoginMsgNoUser should be changed to Save password for %S?
• saveLoginButtonDeny.label should be changed to Don't save - this is a casing change, so the string ID does not need to change.
• saveLoginButtonNever.label should be changed to Never save - this is a casing change, so the string ID does not need to change.
• updateLoginMsg should be changed to Update this login?
• updateLoginMsgNoUser should be changed to Update this password?
• updateLoginMsgAddUsername should be changed to Add username to saved password?
• updateLoginButtonDeny.label should be changed to Don’t update - this is a casing change, so the string ID does not need to change.
• updateLoginButtonDelete.label should be changed to Remove saved login

Comment 1

[Mike Conley (:mconley) (:⚙️)]

Attachment: Bug 1697484 - Update copy for Add/Update Password panel. r?sfoster!,flod!

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/bcdf461c94db
Update copy for Add/Update Password panel. r=sfoster,flod

Comment 3

[Sandor Molnar]

https://hg.mozilla.org/mozilla-central/rev/bcdf461c94db