Closed Bug 1177272 Opened 9 years ago Closed 8 years ago

Uplift "SHOW" placeholder text to Aurora for the remember password doorhanger

Categories

(Toolkit :: Password Manager, defect)

defect
Not set
normal

Tracking

()

RESOLVED WONTFIX
Tracking Status
firefox39 --- wontfix
firefox40 --- affected
firefox41 --- unaffected

People

(Reporter: MattN, Unassigned)

References

Details

(Keywords: late-l10n)

In order to implement bug 1153217, the text "SHOW" was needed to indicate that the password will be shown so the change is less surprising. This string already exists from accounts.firefox.com (though not uppercase) so could be re-used from there. It's only one string so hopefully it isn't a large issue to uplift.
Flags: needinfo?(l10n)
(In reply to Matthew N. [:MattN] from comment #0)
> This string already exists from accounts.firefox.com (though not uppercase) so
> could be re-used from there.

If I understand it correctly, this doesn't make any sense, we're talking about in-product strings (Mercurial) and a website, nothing reusable there. 

Also, we're less than a week away for the end of the cycle and merge day, and that's very prominent UI, not some hidden preference.
What flod said.
Flags: needinfo?(l10n)
(In reply to Francesco Lodolo [:flod] from comment #1)
> (In reply to Matthew N. [:MattN] from comment #0)
> > This string already exists from accounts.firefox.com (though not uppercase) so
> > could be re-used from there.
> 
> If I understand it correctly, this doesn't make any sense, we're talking
> about in-product strings (Mercurial) and a website, nothing reusable there. 

I was mentioning it since localizers could use the same string by referring to the other repo.

> Also, we're less than a week away for the end of the cycle and merge day,
> and that's very prominent UI, not some hidden preference.

The fact that it's prominent is why we want to uplift the string. Without it it can be confusing.
(In reply to Matthew N. [:MattN] from comment #3)
> The fact that it's prominent is why we want to uplift the string. Without it
> it can be confusing.

Firefox 40 is in Beta now. Large part of our localizations won't even be able to localize it at this point, since their localization tool is bound to only one branch (Aurora). 

Personally I consider this a WONTFIX, it's way too late to introduce new strings in the main UI (it was for Aurora as well). Having said that, decision is up to release-drivers.
The UI has changed so this string is no longer needed.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.