i18n: String uses only placeholders

[rakekniven]

Is this meant to be translated?

desktop/src/gui/owncloudgui.cpp

Line 434 in d2cfa24

allStatusStrings += tr("%1: %2").arg(folder->shortGuiLocalPath(), folderMessage);

Reported at Transifex.

Git blame brings me to @nilsding @mgallien

[nilsding]

my thought was that other translations might have a need to reorder that. if there's no need for that I can change that to a plain QString too.

[rakekniven]

The idea of reordering is good, but there is no comment on the meaning of the placeholders.
Is this about timestamps? Or files/folders?

[nilsding]

It's the folder to be synced, followed by a status message (e.g. Syncing 25MB (3 minutes left), Waiting to start syncing., Last sync was successful.)

[rakekniven]

Ok. Please see https://docs.nextcloud.com/server/latest/developer_manual/basics/front-end/l10n.html#provide-context-hints-for-translators and add you example as a comment.
This comment will appear at Transifex and help the translators. Nearly no one is searching/reading source code.

[nilsding]

got it, will add an example comment :)