1526638 - [fr] Fluent: Move localizations from Variants to Macros

Status: RESOLVED FIXED

(Mozilla Localizations :: fr / French, enhancement)

Description

[Francesco Lodolo [:flod]]

Not exactly sure how, but we missed that French is using Variants. Noticed this morning because a string was changed. Filing a bug for visibility.

With Fluent Syntax 0.8 Variants are deprecated and we need to migrate these translations to macros.

This task is going to be managed by l10n-drivers directly, please don't make changes.

Comment 1

[Francesco Lodolo [:flod]]

Attachment: bug1526638.patch

I've set "lowercase" as the default, since it's used 4 out of 5 times.

Also, since -fxaccount-brand-name is a term, there's no risk of it being used as a stand-alone title, it can be used only in another message.

Comment 2

[Théo Chevalier]

Comment on attachment 9042721 [details] [diff] [review]
bug1526638.patch

`options-use-sync` needs lowercase as well, I added a suggestion in Pontoon yesterday to do just that, and left it as a suggestion for the other localizers to notice when approving, but let’s probably just change it here to avoid confusion.

Looks good otherwise!

Note that we’re using variants quite heavily on Firefox Monitor as well.

Comment 3

[Francesco Lodolo [:flod]]

Attachment: bug1526638.patch

Comment 4

[Francesco Lodolo [:flod]]

(In reply to Théo Chevalier [:tchevalier] (please use NI) from comment #2)

Note that we’re using variants quite heavily on Firefox Monitor as well.

Firefox Monitor needs to be fixed as well, you can do it directly in Pontoon.

It looks like you started using it only recently? Because we converted locales there too
https://github.com/mozilla/blurts-server/pull/668/files

Comment 5

[Francesco Lodolo [:flod]]

(In reply to Francesco Lodolo [:flod] from comment #4)

It looks like you started using it only recently?

That was a dumb question, since we added those strings this week. I'll need to keep an eye on ru and uk to make sure they use the new syntax.

For clarity, both work at the moment, but the variantList is already deprecated and will stop working at some point.

Comment 6

[Francesco Lodolo [:flod]]

https://hg.mozilla.org/l10n-central/fr/rev/62840c724ca00ce903f9bddd86dee1a22ce0fbbe