1206102 - Make the TP list names more flexible from a l10n standpoint

Status: RESOLVED FIXED

(Firefox :: Settings UI, defect, P1)

Description

[Panos Astithas (he/him) [:past] (please ni?)]

Bug 1177085 introduced a block list dialog that presents some UI challenges for localizers, due to the length of the strings. Paolo suggested that we could split the strings now, so that future improvements to the text layout could be independent of the localization efforts (hence easy to uplift).

Comment 1

[Panos Astithas (he/him) [:past] (please ni?)]

Attachment: Patch v1

Paolo, is this what you had in mind?

Comment 2

[Panos Astithas (he/him) [:past] (please ni?)]

Comment on attachment 8662953 [details] [diff] [review]
Patch v1

Francesco, are we correct in understanding that splitting the name in two before the uplift will be beneficial to localizers in case we find a nice way to format the text later?

Comment 3

[Francesco Lodolo [:flod]]

Comment on attachment 8662953 [details] [diff] [review]
Patch v1

Review of attachment 8662953 [details] [diff] [review]:
-----------------------------------------------------------------

Yes, it would definitely help. In that case we could fix the JS in mozilla-aurora and leave mozNameTemplate unused until it can be removed in mozilla-central.

::: browser/locales/en-US/chrome/browser/preferences/preferences.properties
@@ +41,2 @@
>  # LOCALIZATION NOTE (mozstdName, etc.): These labels appear in the tracking
>  # protection block lists dialog. They are the names of the block lists.

I'd be more explicit with the comment, just to avoid people trying to change the structure of the sentences.

# LOCALIZATION NOTE (mozstdName, etc.): These labels appear in the tracking
# protection block lists dialog, mozNameTemplate is used to create the final
# string. Note that in the future these two strings (name, desc) could be
# displayed on two different lines.

Comment 4

[Panos Astithas (he/him) [:past] (please ni?)]

Attachment: Patch v2

Thanks, I've made the suggested comment change.

Comment 5

[:Paolo Amadini]

Comment on attachment 8662971 [details] [diff] [review]
Patch v2

Review of attachment 8662971 [details] [diff] [review]:
-----------------------------------------------------------------

Yeah, this is what I had in mind for the localization.

I guess I'd simplify some of the code but it's not a big deal, feel free to land as is with the strings for now, we can always iterate later.

Comment 6

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/c4bc60ef00fc

Comment 7

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/c4bc60ef00fc

Comment 8

[Panos Astithas (he/him) [:past] (please ni?)]

No need for QA here.