1090944 - Clean up conference group header and text code to use latest l10n API

Status: RESOLVED WONTFIX

(Firefox OS Graveyard :: Gaia::Dialer, defect)

Description

[Germán Toro del Valle (:gtorodelvalle)]

We should be using the L10n Web API (https://developer.mozilla.org/en-US/docs/Web/API/L10n_API) exposed by the navigator.mozL10n object as well as the new way to define the localisation metadata in the HTML files. We are currently partially doing it. For example, we are not currently using the once() or ready() functions to detect when the L10n context is ready to be used.

Apart from this:
  1. We should also avoid using the LazyL10n library.
  2. We shouldn't use the get() function to update the translated information shown on the pages but the setAttributes() function instead.

Comment 1

[Germán Toro del Valle (:gtorodelvalle)]

Anything you would like to add, Doug, before adding it to the Dialer most wanted bugs? ;)

Comment 2

[Germán Toro del Valle (:gtorodelvalle)]

Once we agree on the right description, I will file similar bugs for the rest of the apps: Call Log (so the patch to this bug does not become too big), Call Screen and EmergencyCall.

Comment 3

[Doug Sherk (:drs) (inactive)]

I don't think that this is the kind of thing where we should sit down and specifically work on it unless we're asked by the l10n team or someone else.

I asked for this to be filed because the location in code that I was referring to [1] is a particularly bad offender, to the point that it may be worth fixing in a bug specifically for that just because of how bad it is.

As for the general move towards using the newest l10n API, we can include this incrementally with other work we do.

[1] https://github.com/mozilla-b2g/gaia/pull/24825#discussion-diff-18874070

Comment 4

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

Hey guys, thanks for filing it! :)

This document should help you here: https://developer.mozilla.org/en-US/Firefox_OS/Developing_Gaia/localization_code_best_practices

Comment 5

[BMO Automation]

Firefox OS is not being worked on