Open Bug 1407596 Opened 7 years ago Updated 2 years ago

customizeMode.autoHideDownloadsButton.label string landed for bug 1397447 has misleading label

Categories

(Firefox :: Downloads Panel, enhancement, P5)

enhancement

Tracking

()

Tracking Status
firefox57 --- wontfix

People

(Reporter: stef, Unassigned)

References

Details

https://hg.mozilla.org/mozilla-central/diff/b9e11056a978/browser/locales/en-US/chrome/browser/browser.dtd Some localizations are sensitive to what is being translated and using such misleading identifiers (it is checkbox not button label) results in bad l10n. Please change id to sth like customizeMode.autoHideDownloadsButtonCheckbox.label
(In reply to Stefan Plewako [:stef] from comment #0) > https://hg.mozilla.org/mozilla-central/diff/b9e11056a978/browser/locales/en- > US/chrome/browser/browser.dtd > > Some localizations are sensitive to what is being translated and using such > misleading identifiers (it is checkbox not button label) results in bad l10n. It's the "downloadsbutton" that is being autohidden, so I think the label makes sense... I mean, I'll take a patch, but clearly it's too late now. Maybe a localization comment would be better?
(In reply to :Gijs (no reviews, PTO end of today - Nov) from comment #1) > (In reply to Stefan Plewako [:stef] from comment #0) > > https://hg.mozilla.org/mozilla-central/diff/b9e11056a978/browser/locales/en- > > US/chrome/browser/browser.dtd > > > > Some localizations are sensitive to what is being translated and using such > > misleading identifiers (it is checkbox not button label) results in bad l10n. > > It's the "downloadsbutton" that is being autohidden, so I think the label > makes sense... I mean customizeMode.autoHideDownloadsButton.label is not "Auto hide downloads" button label, it is a checkbox label. In Polish (and some other locales) we translate differently whenever text is title, button, option, list, tooltip or content. > I mean, I'll take a patch, but clearly it's too late now. Maybe a > localization comment would be better? I doubt that l10n note would be sufficient but that is really not my call. I could make a patch when if it will be clear what change we want to see.
Priority: -- → P5
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.