Collapse whitespace around color tags in translations

[RasmusKD]

Information

This PR fixes double spaces in translated messages.

A color tag is invisible once rendered, so a translation with whitespace on both sides of one shows a double space in chat. Example from the Danish locale:

flyMode=<primary>Set flytilstand <secondary> {0} <primary>for {1}<primary>.

The space before <secondary> and the space after it both survive rendering, so this shows as Set flytilstand aktiveret for WhoToldYou.

I compared every locale's rendered strings (tags stripped) against the English source: the same mistake appears in roughly 400 strings across 38 locales (pt 35, tr 26, he 25, uk 23, pt_BR 22, ... da 3). Fixing that string by string on Crowdin is not realistic, and new translations keep reintroducing it, so this handles it at render time instead: translated formats collapse whitespace surrounding a primary/secondary tag to a single space before hitting the format cache. Strings with the tag flush against a word on either side are unchanged.

Checked the transformation against the three affected Danish strings (single spaced afterwards) and a clean string (byte-identical), and I'm running the patched build on a Danish-locale server.

[pop4959]

This fix should be made in crowdin. If you submit fixes to the strings there they will be accepted. Your example shows the translation is wrong, we do not want to be trimming random spaces from translatables where they may be necessary/intended.

[RasmusKD]

Fair enough, fixed the Danish ones on Crowdin. The reason I tried code was scale: comparing every locale's rendered strings against the English source found this in ~370 strings across 37 locales, and new translations keep reintroducing it. Crowdin's built-in double-space QA check would catch new ones.

[RasmusKD]

Done as suggested: batch-submitted the fixes on Crowdin, 359 suggestions across 37 languages (a couple were already suggested by others). They'll need proofreader approval per language.