Bug 515201

Hi! Thanks for reporting this bug.
I fixed the typo in "Kurzbefehle einrichten", but was unable to find the other string in the source code. It was removed in this commit in April 2024: https://invent.kde.org/plasma/plasma-welcome/-/commit/09de5f3df0337d7a436639346958bccdda839aa7

What OS/version of plasma-welcome are you running? Run "plasma-welcome --version" in a terminal to find out.
I run Fedora 43 and plasma-welcome version 6.5.5 and don't see this string in the UI.

The typo fix is currently under review and will land in the next couple of days: https://phabricator.kde.org/D30839

(In reply to Philipp Kiemle from comment #1)
> Hi! Thanks for reporting this bug.
> I fixed the typo in "Kurzbefehle einrichten", but was unable to find the
> other string in the source code. It was removed in this commit in April
> 2024:
> https://invent.kde.org/plasma/plasma-welcome/-/commit/
> 09de5f3df0337d7a436639346958bccdda839aa7
> 
> What OS/version of plasma-welcome are you running? Run "plasma-welcome
> --version" in a terminal to find out.
> I run Fedora 43 and plasma-welcome version 6.5.5 and don't see this string
> in the UI.
> 
> The typo fix is currently under review and will land in the next couple of
> days: https://phabricator.kde.org/D30839

I see, I was looking at the .po file (https://invent.kde.org/plasma/plasma-welcome/-/blob/master/po/de/plasma-welcome.po), where the translation is still listed, but it is no longer needed.

On another note, I have noticed that the Time & Date portion of the krunner demo is not working, because "Time" was translated as "Uhrzeit", when only "Zeit" is an accepted keyword. Either the translation in plasma-welcome has to be changed to match the keyword, or the kdeplasma-addons translations has to changed to add "Uhrzeit" as a secondary keyword, if that is even currently possible.

> the translation is still listed, but it is no longer needed.

This is intended, as it re-adds the translation automatically should the string ever be used again. It saves us translators from doing the same things over and over again ;)

> On another note, I have noticed that the Time & Date portion of the krunner demo is not working, because "Time" was translated as "Uhrzeit", when only "Zeit" is an accepted keyword. Either the translation in plasma-welcome has to be changed to match the keyword, or the kdeplasma-addons translations has to changed to add "Uhrzeit" as a secondary keyword, if that is even currently possible.

I changed it to "Zeit in Tokyo", as I think changing the actual krunner keyword (which people may have been accustomed to) would be more disruptive than changing it here in the slideshow. AFAIK, it is not possible to add multiple keywords for krunner queries, unless the module itself parses it somehow. (see https://invent.kde.org/plasma/kdeplasma-addons/-/blob/cfc690ef32c498bc34ce8f482bfae3e07f3e243c/runners/datetime/datetimerunner.cpp#L25 as an example for such regex parsing). I may file a bug report to krunner, requesting this feature ;)

(In reply to Philipp Kiemle from comment #3)
> > the translation is still listed, but it is no longer needed.
> 
> This is intended, as it re-adds the translation automatically should the
> string ever be used again. It saves us translators from doing the same
> things over and over again ;)
> 
> > On another note, I have noticed that the Time & Date portion of the krunner demo is not working, because "Time" was translated as "Uhrzeit", when only "Zeit" is an accepted keyword. Either the translation in plasma-welcome has to be changed to match the keyword, or the kdeplasma-addons translations has to changed to add "Uhrzeit" as a secondary keyword, if that is even currently possible.
> 
> I changed it to "Zeit in Tokyo", as I think changing the actual krunner
> keyword (which people may have been accustomed to) would be more disruptive
> than changing it here in the slideshow. AFAIK, it is not possible to add
> multiple keywords for krunner queries, unless the module itself parses it
> somehow. (see
> https://invent.kde.org/plasma/kdeplasma-addons/-/blob/
> cfc690ef32c498bc34ce8f482bfae3e07f3e243c/runners/datetime/datetimerunner.
> cpp#L25 as an example for such regex parsing). I may file a bug report to
> krunner, requesting this feature ;)

Regarding the old translation, keeping it makes sense, but the 3 individual dots should then probably be changed to an ellipsis to match the other translations nonetheless.