999163 - Clean up bootstrap mozL10n API use in FTU

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::First Time Experience, defect)

Description

[Zibi Braniecki [:zbraniecki][:gandalf]]

FTU currently initializes two codepaths using mozL10n.ready and manually set document's lang/dir.

It should use mozL10n.once and should not set document's lang/dir manually (l10n.js does that).

Comment 1

[Zibi Braniecki [:zbraniecki][:gandalf]]

Attachment: pull request

Comment 2

[Zibi Braniecki [:zbraniecki][:gandalf]]

Actually, I left single mozL10n.ready and a flag since it wasn't cleaner to separate it into mozL10n.ready and mozL10n.once code.

I just renamed the flag to be more self-explanatory and removed manual setting of document's lang/dir.

Comment 3

[Fernando Campo (:fcampo)]

Comment on attachment 8409852 [details] [review]
pull request

Right, lang and dir are set on translation. Has been any change about this or have we been using this wrong from the beginning?

I'm a little lost with l10n.js after the refactor, but does not function once() wait till isReady flag has been set and then trigger the callback?
Doesn't that make .once() and .ready() do the same?

Comment 4

[Zibi Braniecki [:zbraniecki][:gandalf]]

(In reply to Fernando Campo (:fcampo) from comment #3)
> Comment on attachment 8409852 [details] [review]
> pull request
> 
> Right, lang and dir are set on translation. Has been any change about this
> or have we been using this wrong from the beginning?

Refactor cleaned this.

> I'm a little lost with l10n.js after the refactor, but does not function
> once() wait till isReady flag has been set and then trigger the callback?
> Doesn't that make .once() and .ready() do the same?

.once will fire once (yay!) and deregister.
.ready will fire once and then on every language change when new language resources are ready.

Use .once for all basic code initialization patterns. Use .ready for code that sets UI using language.

Comment 5

[Ryan VanderMeulen [:RyanVM]]

Master: https://github.com/mozilla-b2g/gaia/commit/d8904c5af6152f5d647a93a0c31227171ddecd87