Discover: Lücken aufgefüllt, Fehler beseitigt
ClosedPublic

Authored by felf on Feb 4 2021, 12:18 AM.

Details

Summary

Ich hab das Ticket auf BKO gelesen und dachte, das geht schnell. :D
Und dann hab ich eben gleich alle Dateien angesehen.

Frederiks Änderungen von gerade eben sind auch drin.

Diff Detail

Repository
R883 Subversion
Lint
Lint Skipped
Unit
Unit Tests Skipped
felf requested review of this revision.Feb 4 2021, 12:18 AM
felf created this revision.
felf created this object with edit policy "Subscribers".

Wenn du einen SVN-Account hättest, ginge es noch schneller. ;)

Das andere Review habe ich nun auch wieder gesehen. Muss ich aber morgen machen. Das Anwenden und Backporten ist schon immer ein kleines Stück Arbeit.

messages/discover/plasma-discover-notifier.po
72

Hier sehe ich leider den Kontext nicht und kann so nichts dazu sagen.

messages/discover/plasma-discover.po
103–104

Hier wäre die "aktive" Formulierung schon OK, da das Original mit "Lists" beginnt. Ich glaube, das haben wir so gehandhabt.

769–773

Hast du Discover installiert und kannst das in der GUI finden?

felf added inline comments.Feb 6 2021, 3:49 PM
messages/discover/plasma-discover.po
103–104

Es war nicht einheitlich, weil es oben drúber noch einen dritten „List all“-String gab, der wie hier passiv formuliert war. Von mir aus ändere ich das zurück.

769–773

Ich habe es eben nochmal installiert, aber es findet keine Backends (ich bin auf Arch Linux) und bei den Erweiterungen tut sich nichts beim Browsen – die Listen bleiben leer. Ich kann es also nicht testen.

felf added inline comments.Feb 6 2021, 3:51 PM
messages/discover/plasma-discover-notifier.po
72

Ups, hier vergessen zu antworten.
In der Datei gibt es:

  • Updates
  • Upgrades
  • und zum Überdruss auch noch „Refresh …“.

Alle drei benutzen „Aktualisieren“ als Verb oder Substantiv, je nach String

schwarzer accepted this revision.Feb 10 2021, 10:43 PM

Committet.

messages/discover/plasma-discover.po
769–773

Hab's gerade angeschaut. Du hast recht.

This revision is now accepted and ready to land.Feb 10 2021, 10:43 PM
schwarzer closed this revision.Feb 10 2021, 10:43 PM