It's not that leading or trailing whitespace in translated strings is
forbidden *in principle*, but I think it is unwanted in most of these
cases. Remove these spaces that could cause confusion, possibly useless
work for translators and/or ugly displays.
The only cases where the spaces seem to have been intentional are the
trailing ones in menu labels (Translations/*/menu.xml), in order to
obtain right alignment of keyboard shortcuts. However, there is only one
case in English where the spacing looks approximately correct
(File -> Reset). In all other cases, it looks awful. And this poor way
of obtaining right alignment gives bad results in other languages, where
translators use the same number of spaces as in English.
Bottom line: if right alignment of keyboard shortcuts is indeed desired
in the menu labels (I think it is), it should be done in a smarter
way...
This translated string has been removed from Translations/en/menu.xml in
commit ff87cf4257. Remove its translations
from the other Translations/*/menu.xml files that still have it.
Also remove <intl include="Translations/locale.xml"/> from
preferences.xml. Something equivalent based on readProperties() is done
in FlightGear (cf. FGLocale::FGLocale() in
flightgear/src/Main/locale.cxx).
Run script to identify missing/obsolete items:
de/options.xml is missing 192 translation(s).
de/menu.xml is missing 20 translation(s).
es/options.xml is missing 12 translation(s).
es/menu.xml is missing 78 translation(s).
fr/options.xml is missing 0 translation(s).
fr/menu.xml is missing 0 translation(s).
it/options.xml is missing 212 translation(s).
it/menu.xml is missing 78 translation(s).
nl/options.xml is missing 54 translation(s).
nl/menu.xml is missing 1 translation(s).
pl/options.xml is missing 81 translation(s).
pl/menu.xml is missing 76 translation(s).
pt/options.xml is missing 212 translation(s).
pt/menu.xml is missing 86 translation(s).