-
ErCiccione[m]
Comment from rbrunner on weblate:
-
ErCiccione[m]
The whole group of strings along the lines of "x <time unit> ago*, all from the Utils.js source file, run into translation problems for German: The code implements a pattern of generating a number plus an English string (e.g. 10 minutes ago) which does not work cleanly for German, where it should be "vor" plus number plus "<time units>", e.g. vor 10 Minuten
-
ErCiccione[m]
Is it possible to do something tofix that?
-
moneromooo
<joke>10 Minuten (*vor)</joke>
-
rbrunner
Well, I think what I proposed now, which results in outputs like "10 Minuten seither" is not a pretty, but a usable compromise for the time being, without any hastened code changes
-
ErCiccione[m]
Yeah but the problem probably exists for other languages as well, would be better to fix it somehow. But i understand it's tricky to find a good format in this case
-
moneromooo
Not really. With QML, you can have reference to argument indices, so you can do, eg: print("%1 %2", seconds, idiomatic_expression). German can turn "%1 %2" to "%2 %1".
-
moneromooo
Actually, don't even need that. Just "%u ago" would work, as it'd be changed to "vor %u".
-
moneromooo
The "Minuten" also has to go as a %s.
-
moneromooo
Since I assume you also have hours, etc.
-
rbrunner
Yes, string + placeholder for the number to go in would work nicely. It needs several strings (seconds, hours, days), but only 1 placeholder for the number.
-
ErCiccione[m]
Cool, good to know. Hopefully somebody will make the change. No need to have it for this release, but after, definitely.
-
xmr-pr
rating89us opened pull request #2914: Redesign wallet mode selection
-
xmr-pr
-
selsta
.merges
-
xmr-pr
Merge queue empty
-
selsta
.merge+ #2910 #2911 #2912 #2913
-
xmr-pr
...
-
selsta
ErCiccione[m]: does the lupdate command take long?
-
selsta
I tried something out but now it’s taking like 10+ mins to update
-
» selsta waiting patiently
-
ErCiccione[m]
No, it should be almost immediate
-
selsta
hmm it does not work
-
selsta
apparently lupdate supports plurals
-
ErCiccione[m]
try `lupdate -pro monero-wallet-gui.pro -ts translations/monero-core.ts -no-obsolete`
-
selsta
I’m interested if weblate also supports it
-
ErCiccione[m]
it does afaik
-
selsta
yea it does not work on my system somehow
-
selsta
will downgrade my Qt version
-
ErCiccione[m]
weird
-
selsta
hmm does not work :(
-
selsta
-
selsta
and then it gets stuck
-
selsta
same without -pluralonly
-
selsta
ErCiccione[m]: can you test how this looks in weblate?
monero-project/monero-gui #2915
-
selsta
if I understand it correctly we should be able to specify the correct plurals for every language (including english)
-
selsta
even if they have 3+ plural forms
-
selsta
-
xmr-pr
selsta opened pull request #2915: Utils: simplify for translations
-
xmr-pr
-
ErCiccione[m]
I think i had that error in past, but i don't remember why
-
ErCiccione[m]
i will test that PR on my weblate instance later today or tomorrow. About the plurals, i don't know much about it, i have to look it up, but weblate seem to be already using them.
-
selsta
ty