Trimmed whitespace out of translations in AboutSystem

[dancxjo]

The whitespace is a hinderance to automatic translation.

[bonefish]

I don't think I understand the purpose of this patch. What do you mean by automatic translation? Like babelfish?

[mmlr]

I think the point was to pull the whitespace chars out of the to be translated strings so that translators don't need to take them into account. I guess he meant things like the translation assistant site or similar?

[dancxjo]

Yes. That's exactly right. "Polish" is content but "\n\nPolish\n" is content + styling/layout. The two should really be separated for effective translation.

[humdinger]

Unfortunately, I saw this ticket and dancxjo's patch only after my recent commit. Should we still apply the idea behind the patch, i.e. moving the control characters out of the localizing string? It sounds right to me, OTOH, I don't know if it's that much a hassle for a translator to add a "\n" or "\t" to a translated string.

[bonefish]

Replying to humdinger:

Unfortunately, I saw this ticket and dancxjo's patch only after my recent commit. Should we still apply the idea behind the patch, i.e. moving the control characters out of the localizing string? It sounds right to me, OTOH, I don't know if it's that much a hassle for a translator to add a "\n" or "\t" to a translated string.

I don't mind either way. It's definitely a translation issue (which I won't touch) and will probably break all existing translations (if any).

[pulkomandy]

I'd like to use ICU/Locale kit directly to get the proper localized language names. No need to hand-translate them when there is already a convenient database.

And if something break translations, better fix it now than later, before there are many translations to fix.

[pulkomandy]

Hello, This patch does not apply properly to the current sourcecode. Could you update it ?

[mmadia]

patched against hrev39395

[mmadia]

Here's a new patch. I wasn't sure if it'd be better to commit this now, or to hold off until after the next release.

[siarzhuk]

Looks like this one can be closed as wontfix. Using control characters in translated strings is a common practice this time. Many translations for AboutSystem already performed so this patch has no sense this time, I think.

Are there any objections? What do you think, Adrien?

[pulkomandy]

Yes, I think this makes the code needlessly complicated, and the current status doesn't clutter the translations that much...