Closed Bug 362000 Opened 13 years ago Closed 13 years ago
AUS feature in Preferences window needs to make &brand
Short Name; localizable
Bug 341971 landed the AUS feature in Sunbird, but unlike Firefox or Thunderbird Preferences > Advanced > Update Tab, &brandShortName; is not localizable (hardcoded in xul file). A quick search in lxr showed that at least for the moment, cs, fi, lt and pl locales need that : http://lxr.mozilla.org/l10n-mozilla1.8/search?string=enableAppUpdate.label
According to Bug 341971 Comment #16 this was done on purpose. Entity brandShortName is already defined in /calendar/locales/en-US/chrome/branding/brand.dtd /other-licenses/branding/sunbird/locales/en-US/brand.dtd so why is this not localizable?
(In reply to comment #1) > According to Bug 341971 Comment #16 this was done on purpose. Entity > brandShortName is already defined in > /calendar/locales/en-US/chrome/branding/brand.dtd > /other-licenses/branding/sunbird/locales/en-US/brand.dtd > so why is this not localizable? > I wasn't clear : entity brandShortName in pl locale for browser for instance : http://lxr.mozilla.org/l10n-mozilla1.8/source/pl/browser/chrome/branding/brand.dtd http://lxr.mozilla.org/l10n-mozilla1.8/source/pl/other-licenses/branding/firefox/brand.dtd is translated differently in http://lxr.mozilla.org/l10n-mozilla1.8/source/pl/browser/chrome/browser/preferences/advanced.dtd#55 So I guess it will be the same for Calendar. That's why this line in advanced.xul : http://lxr.mozilla.org/mozilla1.8/source/calendar/base/content/preferences/advanced.xul#168 should be replaced by label="&pref.calendar.advanced.update.enableAppUpdate.label;" and added in chrome/calendar/preferences/advanced.dtd I hope this was clearer
To make it more clear, you can't just do sth like this: &checkForUpdatesFor; * &brandShortName; as the brandShortName may not be all you would need in some languages, i.e. it may need to be declensed, like in Polish: Sprawdz dostepnosc aktualizacji dla: * programu Sunbird This is not a real declension of the Sunbird name, as we are using a helper word here, since you can't really automate the declension of the entities. So, we really need to add the word "programu" before the brand short name. This applies to many more languages.
Readds the entity.
Assignee: cedric.corazza → lilmatt
Status: NEW → ASSIGNED
Attachment #246743 - Flags: first-review?(jminta)
Comment on attachment 246743 [details] [diff] [review] rev0 - Adds entity for appname r=jminta
Attachment #246743 - Flags: first-review?(jminta) → first-review+
Patch checked in on MOZILLA_1_8_BRANCH and trunk. -> FIXED
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Component: Sunbird Only → Preferences
QA Contact: sunbird → preferences
You need to log in before you can comment on or make changes to this bug.