Closed Bug 362000 Opened 13 years ago Closed 13 years ago

AUS feature in Preferences window needs to make &brandShortName; localizable

Categories

(Calendar :: Preferences, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED
Sunbird 0.5

People

(Reporter: cedric.corazza, Assigned: mattwillis)

Details

(Keywords: l12y)

Attachments

(1 file)

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.