1079860 - [Loop]: Use WebApp's manifests instead of locales.ini

Status: VERIFIED FIXED

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

Description

[Cristian Rodriguez (:crdlc)]

See bug 1022767

Comment 1

[Cristian Rodriguez (:crdlc)]

Attachment: Github pull request

Comment 2

[Cristian Rodriguez (:crdlc)]

Comment on attachment 8501786 [details]
Github pull request

Hi Axel,

   Could you take a look at this patch? I've implemented these tasks and I would like to know if it is ok or I missed something.

1) Updated l10n library
2) Updated manifest.webapp and prepared for localization
3) Updated index.html to set the new paths to properties files
4) Deleted <br> from locales
5) Fixed the markup because there were a tag which wasn't closed and the localization failed

Thanks a lot for your time

Comment 3

[Francesco Lodolo [:flod] (OOO Oct 3–10)]

You're making assumptions on the language structure here

-loggedInAs = You're logged in as <br> {{username}}
+loggedInAs = You're logged in as

As in: you can't be sure that the username is going to work at the end of the sentence. You should leave a placeholder.

You're also reusing the "unknown" string. I'd suggest to have a specific string that talks about "unknown user".

Other doubt (leaving f? open): I don't think that we want "Firefox Hello" as a localizable string.

Comment 4

[Cristian Rodriguez (:crdlc)]

Comments addressed Francesco, maybe you would like to take a look again, thanks a lot for your time

Comment 5

[Francesco Lodolo [:flod] (OOO Oct 3–10)]

String-wise it looks good to me now (with the doubt about "Firefox Hello").

Comment 6

[Cristian Rodriguez (:crdlc)]

Firefox Hello has been removed 

https://github.com/mozilla-b2g/firefoxos-loop-client/pull/171/files#diff-964b329a1e6469f66cec3e4991fbb538L2

Now this is defined in js

https://github.com/mozilla-b2g/firefoxos-loop-client/pull/171/files#diff-88824f0fb2947afec9c4bc50f3224cfbR1

it implies no translations for branding as you suggested correctly IMHO



(In reply to Francesco Lodolo [:flod] from comment #5)
> String-wise it looks good to me now (with the doubt about "Firefox Hello").

Comment 7

[Francesco Lodolo [:flod] (OOO Oct 3–10)]

(In reply to Cristian Rodriguez (:crdlc) from comment #6)
> it implies no translations for branding as you suggested correctly IMHO

Exactly, but I want to see Pike's opinion on it ;-)

Comment 8

[Axel Hecht [:Pike]]

Comment on attachment 8501786 [details]
Github pull request

The branding question is really a product/legal decision.

There are basically three levels of choice, with different balance between policy and adaptibility

- hard-code the brand. This guarantees that the brand is literally appearing as the same sequence of glyphs. Depending on the code referencing the brand, you can add pre- and postfixes.
- expose the brand to localizers, but policy to keep it intact, unless there are explicit exceptions, granted on a one-off basis.
- have a localized brand. Requires quite a bit of skill on the localizer front to find a great brand, but also makes sure that the brand is best-suiting market.

Technically this looks good, and the l12y stuff flod mostly covered. I checked with gandalf if there's something better to recommend instead of the innerHTML, but that's WIP. No reason to hold back on that.

Great to see the update here to a current backbone. Here I can loop (hic) in help, and get good answers.

Comment 9

[Cristian Rodriguez (:crdlc)]

and also removed unused strings

Comment 10

[Borja Salguero [:borjasalguero]]

Comment on attachment 8501786 [details]
Github pull request

Thanks!! Great job here!

Comment 11

[Cristian Rodriguez (:crdlc)]

in master:

https://github.com/mozilla-b2g/firefoxos-loop-client/commit/433c206f125419e83e08d91435177803637cdf34

Comment 12

[Loli (:lolimartinezcr)]

Tested, German language in Loop Mobile client.

Two scenarios tested:
1. User changes to German/English and Spanish when  Loop is open.
2. Device with German/English and Spanish languaje selected and after Loop is installed.

Flame
user
v2.0
Gecko-05f9775
Gaia-d3e4da3
Loop version: master , dd4862e