#10717 closed enhancement (fixed)
js: make it unnecessary to call babel.format explicitly
Reported by: | Christian Boos | Owned by: | Christian Boos |
---|---|---|---|
Priority: | normal | Milestone: | 1.0 |
Component: | i18n | Version: | 0.13dev |
Severity: | normal | Keywords: | babel javascript |
Cc: | Branch: | ||
Release Notes: |
(JavaScript API improvement) |
||
API Changes: |
babel.js: it's no longer necessary to call |
||
Internal Changes: |
Description (last modified by )
On the topic of babel.format
, any reason why it couldn't be integrated in gettext
?
For example, writing:
title: _("Select ticket %(id)s for modification", {id: tId}),
instead of the cumbersome:
title: babel.format(_("Select ticket %(id)s for modification"), {id: tId}),
The change is fully backward compatible.
Now that Jun started to "customize" our babel.js in r11047… Ok, at some point someone should backport this to Babel:source:trunk/contrib/babel.js ;-)
Related branches
Attachments (0)
Change History (4)
comment:1 by , 12 years ago
Description: | modified (diff) |
---|
comment:2 by , 12 years ago
comment:3 by , 12 years ago
API Changes: | modified (diff) |
---|---|
Resolution: | → fixed |
Status: | new → closed |
Thanks for the feedback! I integrated your fix and rebased the result, plus additional whitespace fixes in repos:cboos.git:ticket10717/babeljs-format-in-gettext.2.
Then committed this as r11068.
comment:4 by , 12 years ago
Release Notes: | modified (diff) |
---|
Nice idea! I just tested and it works fine.
However,
ngettext()
doesn't format when the string ofsingular
is not in the catalog, 7055b55/jomae.git.