Need proper plural form for password age (passwordsDetails.age)

RESOLVED WONTFIX

Status

()

RESOLVED WONTFIX
4 years ago
3 years ago

People

(Reporter: flod, Unassigned)

Tracking

(Blocks: 1 bug)

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

4 years ago
Reference changeset:
http://hg.mozilla.org/mozilla-central/rev/4f270128536a

::: mobile/android/locales/en-US/chrome/aboutPasswords.properties
@@ +1,5 @@
> +# This Source Code Form is subject to the terms of the Mozilla Public
> +# License, v. 2.0. If a copy of the MPL was not distributed with this
> +# file, You can obtain one at http://mozilla.org/MPL/2.0/.
> +
> +passwordsDetails.age=Age: %S days

This is broken even for English (1 days), we need a proper plural form here, especially for locales with complex plural forms.

Also a comment explaining what "Age" means in this context would definitely help.
(Reporter)

Updated

4 years ago
Summary: Need proper plural form for password age (passwordsDetails.age)1093201 → Need proper plural form for password age (passwordsDetails.age)
This part of the code is leftover from a prototype i built. I was using "Age" to show how long it had been since you had updated the password for this login. Old passwords might be more likely to be exposed to others, so updating a password could be seen as good practice.

That said, we may not even ship v1 with this feature.

Comment 2

4 years ago
Sorry about that, I should have caught that in a review.

That said, this is behind a Nightly build flag for now while we iterate on the UI, and this string will probably change before we ever ship this, since "Age" can also be a confusing label for en-US users.
Bug 1144413 will remove the Details page, so I'm going to WONTFIX this bug.
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.