Closed
Bug 1630282
Opened 5 years ago
Closed 5 years ago
Not all locales have the same number of map service prefs
Categories
(MailNews Core :: Localization, task)
MailNews Core
Localization
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 77.0
People
(Reporter: iannbugzilla, Assigned: iannbugzilla)
References
Details
Attachments
(1 file, 1 obsolete file)
3.19 KB,
patch
|
rjl
:
review+
Pike
:
feedback+
|
Details | Diff | Splinter Review |
At the moment l10n.toml ignores parts of region.properties for when a locale has a different number of prefs to en-US, e.g. for search prefs, but there isn't anything to cover mapit_url
This seems to be the right syntax and seems to work but asking Pike for feedback to confirm
Attachment #9140781 -
Flags: review?(rob)
Attachment #9140781 -
Flags: feedback?(l10n)
Comment 2•5 years ago
|
||
Comment on attachment 9140781 [details] [diff] [review]
Add ignore for mapit
Review of attachment 9140781 [details] [diff] [review]:
-----------------------------------------------------------------
This also needs a change to the respective `filter.py` files, but as far as the toml files are concerned, this is OK.
Now, conceptually, it'd be good to get rid of these entries altogether. It's really not something that localizers can properly handle, and for the m-c products, we moved most of that stuff into the core code.
Attachment #9140781 -
Flags: feedback?(l10n) → feedback-
Update patch from feedback and created some bugs to cover part of the work for the centralisation of these settings.
Attachment #9140781 -
Attachment is obsolete: true
Attachment #9140781 -
Flags: review?(rob)
Attachment #9140856 -
Flags: review?(rob)
Attachment #9140856 -
Flags: feedback?(l10n)
Comment 4•5 years ago
|
||
Comment on attachment 9140856 [details] [diff] [review]
Add ignore for mapit v1.1
Review of attachment 9140856 [details] [diff] [review]:
-----------------------------------------------------------------
lgtm, thanks.
Attachment #9140856 -
Flags: feedback?(l10n) → feedback+
Updated•5 years ago
|
Attachment #9140856 -
Flags: review?(rob) → review+
Keywords: checkin-needed-tb
Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/0665e9d6b9ec
Not all locales have the same number of map service prefs. r=rjl
Updated•5 years ago
|
Target Milestone: --- → Thunderbird 77.0
You need to log in
before you can comment on or make changes to this bug.
Description
•