Last Comment Bug 774234 - [bedrock] need to be able to define per locale l10n blocks
: [bedrock] need to be able to define per locale l10n blocks
u=localizer c=l10n p=0
Classification: Other
Component: Bedrock (show other bugs)
: unspecified
: All All
P1 normal
: Future
Assigned To: Paul [:pmac] McLanahan
Depends on:
Blocks: bedrock-l10n
  Show dependency treegraph
Reported: 2012-07-16 05:18 PDT by Pascal Chevrel:pascalc
Modified: 2013-02-14 12:06 PST (History)
6 users (show)
See Also:
QA Whiteboard:
Iteration: ---
Points: ---


Description User image Pascal Chevrel:pascalc 2012-07-16 05:18:10 PDT
You can define an l10n block for a template that allow customization of a portion of the page, here is an example in the Firefox 14 whatsnew page where we added a specific paragraph to the Russian page to announce a partnership change affecting users of this locale:

Extracting this template just for Russian is easy: l10_check ru

But the next time I run an l10n_check for all locales, that l10n block is going to be extracted in the locale folder for all of our locales.

We need a way to define in the l10n block that this only affect one locale or a set of locales, something like:

{% l10n whatsnew_feature ; locales=ru,uk,es-ES ; 20120712 %}
Comment 1 User image Ben (:bensternthal) 2012-09-18 13:51:48 PDT
Triaging, these needs analysis by bedrock devs.
Comment 2 User image Kushal Das 2012-11-07 03:07:33 PST
Pull request created in
Comment 3 User image Anthony Ricaud (:rik) 2012-11-13 08:34:54 PST
I think a better syntax for this would be:
{% l10n whatsnew_feature locales=ru,uk,es-ES 20120712 %}

Separator is a space. This will require changing the existing usage of this tag but it's a quick thing.
Comment 4 User image Jennifer Bertsch [:jbertsch] 2012-12-06 16:27:35 PST
We'll most likely need this functionality as part of the MWC wave.
Comment 5 User image Paul [:pmac] McLanahan 2013-01-28 13:46:25 PST
Taking this and finishing up the work started by Kushal. Thanks again for the help Kushal!
Comment 6 User image [github robot] 2013-02-04 19:12:52 PST
Commits pushed to master at
Bug 774234: we can now define per locale l10n blocks

The blocks can be any of these 3 style:

{% l10n whatsnew_feature locales=ru,bn-IN 20120712 %}
{% l10n whatsnew_feature 20120712 %}
{% l10n whatsnew_feature, 20120712 %}

For the First example we will have templates in the given two
2nd and 3rd example, we will have the templates for all locales.
Bug 774234: Add tests for l10n blocks and langs support.

Refactor template handling and parsing.
Merge pull request #614 from pmclanahan/bug-774234-l10n-block-locales-support

Bug 774234: Add langs definition to l10n blocks.
Comment 7 User image Paul [:pmac] McLanahan 2013-02-14 12:06:08 PST
This is in production.

Note You need to log in before you can comment on or make changes to this bug.