Do not compose strings from parts

[trev]

Background

We currently have the following translatable strings:

We noticed you have both <strong>$tracking$</strong> and <strong>$acceptableAds$</strong> enabled.

and

If you prefer extra privacy, select the <strong>$acceptableAdsPrivacy$</strong> checkbox below.

The placeholders are replaced by other strings here, which are also used on the page by itself. This causes issues for languages where nouns needs to be declined and requires translators to use tricks. For example, in Russian "and $acceptableAds$" had to be changed into "and the feature $acceptableAds$" to make sense grammatically. Not all translators will notice the issue, and not all will know how to deal with it.

What to change

Do not use placeholders, just insert the strings here:

We noticed you have both <strong>Block additional tracking</strong> and <strong>Allow Acceptable Ads</strong> enabled.

and

If you prefer extra privacy, select the <strong>Only allow ads without third-party tracking</strong> checkbox below.

This gives translators more flexibility and avoids grammatical mistakes.

[saroyanm]

Updating the Strings now will require re-adding the translations in the Crowdin, I have couple of questions to Tamara before fixing this issue:

• Tamara, have you updated the Translation of this strings already (after #6163) ?
• How much effort it will require you if I update the strings the way it's suggested here?
• Do you agree that we shouldn't use placeholders ?

[trev]

This is definitely not something to do before Adblock Plus 3.0.2 release - we have enough issues as it is already.

[saroyanm]

Replying to trev:

This is definitely not something to do before Adblock Plus 3.0.2 release - we have enough issues as it is already.

Noted, I thought we want this before the release.

[Ross]

I'm guessing this issue and near-future translation updates would fix the acceptable ads notification appearing in multiple languages? Or should I create a new ticket for that?

Example when FF is set to DE:

We noticed you have both 'Blockieren von zusätzlichem Tracking' and 'Acceptable Ads erlauben?' enabled.

[greiner]

@Shikitita Which one is easier to manage? Ensuring that the same texts are used across multiple strings or ensuring that translators handle placeholders properly?

[saroyanm]

Replying to Ross:

I'm guessing this issue and near-future translation updates would fix the acceptable ads notification appearing in multiple languages? Or should I create a new ticket for that?

Example when FF is set to DE:

We noticed you have both 'Blockieren von zusätzlichem Tracking' and 'Acceptable Ads erlauben?' enabled.

Yes, the string is ​translated in the Crowdin, I'm just waiting for the Go(after all translations are ready) to import it to the ABPUI.

[Shikitita]

Replying to saroyanm:

Updating the Strings now will require re-adding the translations in the Crowdin, I have couple of questions to Tamara before fixing this issue:

• Do you agree that we shouldn't use placeholders ?

Replying to greiner:

@Shikitita Which one is easier to manage? Ensuring that the same texts are used across multiple strings or ensuring that translators handle placeholders properly?

This should answer also saroyanm's question:

Obviously I'd rather ensure the translators handle placeholders properly, as it's less prone to error in the sense that we'd be using the right translations for those two sections. If we don't, we'd have to make sure these strings aren't updated by our community and risk the terms being inconsistent with the right wording for those two options.

[trev]

Replying to Shikitita:

Obviously I'd rather ensure the translators handle placeholders properly,

How can you do that? Would you know that the sentence produced by replacing placeholders is grammatically incorrect?

[greiner]

Rejecting this ticket because we'd be replacing one issue with another one which is expected to be at least as difficult to deal with.