Dolphin auf Vordermann gebracht
AbandonedPublic

Authored by schwarzer on Feb 18 2021, 11:16 PM.

Details

Summary

Wie schon in meiner Mail erwähnt, gibt es hier einen „Control button“, der sich auf den Dropdown bezieht, der bei ausgeblendeter Menüzeile erscheint. Das Ding heißt aber eigentlich Show menu/Menü anzeigen. Vielleicht hier mal den Entwickler anschreiben, warum er das hier anders nennt?

Diff Detail

Repository
R883 Subversion
Lint
Lint Skipped
Unit
Unit Tests Skipped
There are a very large number of changes, so older changes are hidden. Show Older Changes
felf requested review of this revision.Feb 18 2021, 11:16 PM
felf created this object with edit policy "Subscribers".
schwarzer added inline comments.
messages/dolphin/dolphin.po
437

In der Tat. ... Ist im Original ja schon etwas seltsam. ... Das muss ich mir mal in Dolphin anschauen, um das zu verstehen.

547

sodass?

589

die die
vornahmen -> vorgenommen haben?
Vermutlich würde ich hier auch den Nebensatz nicht einschieben: "Dies nimmt die letzte Änderung zurück, die Sie ..." Liest sich etwas leichter, oder?

607

Leerzeichen vor filename-Tag

669

"dieselben" ein Wort

752

schaltet

... muss hier erstmal pausieren.

felixernst added a subscriber: felixernst.EditedFeb 20 2021, 3:59 PM

Hallo!

Vielen Dank, dass sie sich um die Übersetzungen, insbesondere die der Hilfstexte, kümmern.

Wie schon in meiner Mail erwähnt, gibt es hier einen „Control button“, der sich auf den Dropdown bezieht, der bei ausgeblendeter Menüzeile erscheint. Das Ding heißt aber eigentlich Show menu/Menü anzeigen. Vielleicht hier mal den Entwickler anschreiben, warum er das hier anders nennt?

Im Programmcode heißt der Knopf Control button und wenn ich mich nicht irre, war das auch die Beschriftung des Knopfes, als ich diese Hilfstexte schrieb. Deswegen die fälschliche Bezeichnung. Dort ist also kein tieferer Sinn versteckt und ich sollte auch die englischen Texte aktualisieren. Die Benennung Control Menu wird auch im Programmcode bald zu HamburgerMenu verändert, sodass dann nirgends mehr das Wort Control auftaucht: https://invent.kde.org/system/dolphin/-/merge_requests/71

Leider wird momentan allgemein der Nutzen solcher "What's This?"-Hilfstexte in Frage gestellt, da manche davon ausgehen, dass Nutzer nicht in der Lage seien, sie aufzurufen (https://phabricator.kde.org/T9986). Ich arbeite gerade daran, diese Nutzungsprobleme zu tilgen, indem ich sie von den normalen Tooltips aus aufrufbar mache. Dazu werde ich die nächsten Tage einen Prototypen vorstellen. Generell besteht über die Art, wie Hilfe angeboten werden sollte, Uneinigkeit.

Um auf den Punkt zu kommen: Wenn es dumm läuft, dann werden diese Hilfstexte ganz entfernt oder müssen evtl. angepasst werden. Ich dachte diese Warnung sollte ich aussprechen, sodass sie beurteilen können, inwiefern ihre weiteren Bemühungen hier teilweise ins Leere laufen könnten.

Edit: Der Prototyp ist gut angekommen.

felf updated this revision to Diff 83435.Mar 6 2021, 6:30 PM

Mergekonflikte mit Upstream bereinigt. Ansonsten keine neuen Änderungen außer zweier entfernter Doppelpunkte.

lueck added a subscriber: lueck.Mar 12 2021, 6:04 AM
In D29895#677458, @felf wrote:

Mergekonflikte mit Upstream bereinigt. Ansonsten keine neuen Änderungen außer zweier entfernter Doppelpunkte.

Die Kommentare/Anmerkungen von schwarzer hast Du ignoriert?

lueck added a comment.Mar 12 2021, 6:12 AM

@felf :
Bitte "svn diff -x -U100 l10n-kf5/de >/tmp/foo.diff" benutzen, dann wird hier auch der vollständige englische Text der Nachricht angezeigt.
Danke

So, bin jetzt soweit durch. An dieser Stelle möchte ich noch einmal anmerken, dass es sinnvoller ist, kleine Happen zum Review zu stellen.

  1. ist das für den Reviewer angenehmer
  2. ist das für den Bearbeiter angenehmer, da die Reviews kleiner ausfallen
  3. werden systematische Fehler schneller geklärt und sind dann in den folgenden kleinen Review bereits behoben
  4. beim Einspielen muss der Patch nach Stable portiert werden, was immer auch Handarbeit bedeutet (und somit lieber in kleinen Happen vorkommt)

Also bitte ruhig etwas früher kleinere Happen zum Review stellen. Es bringt ja nichts, wenn man eine Datei in einem Sprung auf 100% bringt und das Review dann Wochen oder Monate liegt und immer wieder aktualisiert werden muss, weil die Entwickler Dinge im Code herumschieben.
Dann lieber in mehreren überschaubaren Schritten, damit die geleistete Arbeit schneller beim Benutzer ankommt. Denn dafür sitzen wir hier. :)

messages/dolphin/dolphin.po
806

Hier fehlt ein Wort.

839

Order -> Ordner

882

Die - Dies?

884

sie -> Sie

924

Wort fehlt.

944

Wort fehlt.

952

Order -> Ordner

973

Verzeichnis -> Ordner?

1067

und und

1091

Ordner

1113

öffnet

1261

wir -> wie

1283

usw.

1549

Was bedeutet "Abgehen" hier?

4482

kürzlich

4487

Ordner

4515

Wenn ich mich recht entsinne, hatten wir hierzu einmal eine Diskussion und waren so verblieben, dass wir die normalen drei Punkte verwenden, um den "Textsatz"-Anteil nicht weiter zu erhöhen. Oder irre ich mich da?

4525

"denselben" ein Wort

4623

Ordner

4643

Korrigiert mich, wenn ich falsch liege, aber funktionieren diese Platzhalter nicht nur in der Doku?

4646

mehrerer

messages/dolphin/org.kde.dolphin.appdata.po
38

Ordnern

72

einen einen

93

Kommandos -> Befehle?

94

Ordner

Hallo!

Vielen Dank, dass sie sich um die Übersetzungen, insbesondere die der Hilfstexte, kümmern.

Wie schon in meiner Mail erwähnt, gibt es hier einen „Control button“, der sich auf den Dropdown bezieht, der bei ausgeblendeter Menüzeile erscheint. Das Ding heißt aber eigentlich Show menu/Menü anzeigen. Vielleicht hier mal den Entwickler anschreiben, warum er das hier anders nennt?

Im Programmcode heißt der Knopf Control button und wenn ich mich nicht irre, war das auch die Beschriftung des Knopfes, als ich diese Hilfstexte schrieb. Deswegen die fälschliche Bezeichnung. Dort ist also kein tieferer Sinn versteckt und ich sollte auch die englischen Texte aktualisieren. Die Benennung Control Menu wird auch im Programmcode bald zu HamburgerMenu verändert, sodass dann nirgends mehr das Wort Control auftaucht: https://invent.kde.org/system/dolphin/-/merge_requests/71

Leider wird momentan allgemein der Nutzen solcher "What's This?"-Hilfstexte in Frage gestellt, da manche davon ausgehen, dass Nutzer nicht in der Lage seien, sie aufzurufen (https://phabricator.kde.org/T9986). Ich arbeite gerade daran, diese Nutzungsprobleme zu tilgen, indem ich sie von den normalen Tooltips aus aufrufbar mache. Dazu werde ich die nächsten Tage einen Prototypen vorstellen. Generell besteht über die Art, wie Hilfe angeboten werden sollte, Uneinigkeit.

Um auf den Punkt zu kommen: Wenn es dumm läuft, dann werden diese Hilfstexte ganz entfernt oder müssen evtl. angepasst werden. Ich dachte diese Warnung sollte ich aussprechen, sodass sie beurteilen können, inwiefern ihre weiteren Bemühungen hier teilweise ins Leere laufen könnten.

Edit: Der Prototyp ist gut angekommen.

@felixernst
für die Zukunft ist es vielleicht sinnig, bei solchen Testballons Übersetzungen auszuschließen. Entweder durch Weglassen von i18n() oder durch Anpassungen in der messages.sh oder so. Dazu gerne im Vorfeld auf kde-i18n-doc@k.o melden und fragen, wie du dazu vorgehen kannst.

@felixernst
für die Zukunft ist es vielleicht sinnig, bei solchen Testballons Übersetzungen auszuschließen. Entweder durch Weglassen von i18n() oder durch Anpassungen in der messages.sh oder so. Dazu gerne im Vorfeld auf kde-i18n-doc@k.o melden und fragen, wie du dazu vorgehen kannst.

Als ich sie geschrieben habe, hätte ich nicht vermutet, dass es Leute gibt, die etwas dagegen haben könnten, auf diese Weise Hilfe anzubieten. Es war auch die erste Sache, die ich überhaupt für KDE gemacht habe, also bitte ich da eine gewisse Ahnungslosigkeit zu entschuldigen. Wenn ich in Zukunft etwas schreibe, dass evtl. unnötige Übersetzungsarbeit bedeutet, werde ich mich dabei geschickter verhalten. In jedem Fall scheint sich die Sache aber jetzt sowieso geklärt zu haben und diese Übersetzungen werden in Zukunft Nutzern dabei helfen die Funktionen von Dolphin in ihrer Muttersprache zu entdecken und zu verstehen. Hoffentlich sogar besser denn je mit https://invent.kde.org/frameworks/kxmlgui/-/merge_requests/45.

felf marked 28 inline comments as done.Mar 13 2021, 7:04 PM

Die Kommentare/Anmerkungen von schwarzer hast Du ignoriert?

Es kann gut sein, dass ich zu dem Zeitpunkt noch gar keine Rückmeldungen habe einfliessen lassen, weil der Review noch nicht abgeschlossen war. In dem Diff, den ich gleich einreiche, sind alle Kommentare, die mir jetzt hier angezeigt wurden, verarbeitet. Es gibt ein paar wenige Ausnahmen, bei denen das Done-Flag entsprechend nicht gesetzt ist.

Also bitte ruhig etwas früher kleinere Happen zum Review stellen. Es bringt ja nichts, wenn man eine Datei in einem Sprung auf 100% bringt und das Review dann Wochen oder Monate liegt und immer wieder aktualisiert werden muss, weil die Entwickler Dinge im Code herumschieben.

Meistens (so auch in diesem Fall) fange ich mit einer Datei an, wenn mir bei der normalen Benutzung ein Fehler auffällt. Aber dann gucke ich meistens gleich die ganze Datei einmal durch, damit sich ein Merge Request „lohnt“.

Dann lieber in mehreren überschaubaren Schritten, damit die geleistete Arbeit schneller beim Benutzer ankommt. Denn dafür sitzen wir hier. :)

Das kann ich natürlich nachvollziehen, aber dann muss *ich* mir ja merken, welche Teile ich schon gemacht habe und welche nicht. ;-)

messages/dolphin/dolphin.po
806

Ich habe Sie durch über ersetzt, dann wirkt der Satz allgemein weniger holprig.

1549

Das Abgehen (Abschreiten) von einzelnen Punkten auf einem Pfad.

4643

Kann gut sein. Ich habe das wohl durcheinander gebracht, weil das hier zwar ein Hilfetext ist, aber der trotzdem in der Anwendung auftaucht und nicht im Handbuch. Ich ándere es in Fließtext.

felf updated this revision to Diff 83436.Mar 13 2021, 7:08 PM
felf marked 2 inline comments as done.
felf set the repository for this revision to R883 Subversion.

Neu: jetzt mit -U100.

Offensichtlich ist hier was schief gelaufen mit dem diff
Die Änderungen in dolphin.po fehlen und aus org.kde.dolphin.appdata.po wurden Nachrichten entfernt, die in dolphin.po enthalten sind.

felf updated this revision to Diff 83438.Mar 29 2021, 9:57 PM

Ich habe nochmal svn up gemacht und meine Änderungen mit deinen Korrekturen gemergt. Hier ist der diff davon mit -x -U100.

lueck accepted this revision.Mar 30 2021, 3:18 AM

Einige "Verzeichnisse" sollten noch in "Ordner" geändert werden, ansonsten ist das OK

messages/dolphin/dolphin.po
407

Verzeichnisse -> Ordner

792

Verzeichnis -> Ordner

977–980

Verzeichnis -> Ordner
Wurzelverzeichnis -> Basisordner?

4626

Order -> Ordner

messages/dolphin/org.kde.dolphin.appdata.po
52

Verzeichnissen -> Ordnern

80

Verzeichnisse -> Ordner

This revision is now accepted and ready to land.Mar 30 2021, 3:18 AM
MrPepe requested changes to this revision.Mar 30 2021, 10:08 AM
MrPepe added a subscriber: MrPepe.
MrPepe added inline comments.
messages/dolphin/org.kde.dolphin.appdata.po
52

geteilte Ansicht -> geteilten Ansicht

This revision now requires changes to proceed.Mar 30 2021, 10:08 AM
felf updated this revision to Diff 83440.Mar 30 2021, 10:46 AM
felf marked an inline comment as done.

Verzeichnis→Ordner

Allerdings gefällt „Wurzelordner“ meinem Sprachgefühl irgendwie so gar nicht. Das Wort wird auch sonst nirgends verwendet – anders als Wurzelverzeichnis.

In D29895#677545, @felf wrote:

Verzeichnis→Ordner

Allerdings gefällt „Wurzelordner“ meinem Sprachgefühl irgendwie so gar nicht. Das Wort wird auch sonst nirgends verwendet – anders als Wurzelverzeichnis.

Ich hatte ja auch "Basisordner" (wird bereits 199 mal verwendet) und nicht "Wurzelordner" vorgeschlagen.

felf updated this revision to Diff 83441.Mar 30 2021, 7:42 PM

Neu: Jetzt auch mit Basisordner.
Ich war gestern Abend wohl schon geistig umnachtet. Kein Wunder, wenn man was im Bett erledigt. Jetzt sollte alles stimmen.

lueck accepted this revision.Mar 31 2021, 4:08 AM
lueck commandeered this revision.Mar 31 2021, 4:37 AM
lueck edited reviewers, added: felf; removed: lueck.

ist drin, danke

oneway added a subscriber: oneway.Apr 26 2021, 7:37 PM
oneway added inline comments.
messages/dolphin/dolphin.po
4042

Datumsformat?

lueck added inline comments.Apr 27 2021, 4:53 AM
messages/dolphin/dolphin.po
4042

Zur Auswahl steht bei Datumsstil oder Datumsformat:
Relativ (z. B. „Vor 30 Minuten“)
oder
Absolut (z. B. „“27.04.21 08:43“)

Da passt Datumsformat vielleicht besser?

felf added inline comments.Apr 30 2021, 10:01 PM
messages/dolphin/dolphin.po
4042

Gegenargument: wenn das Datum weiter zurück liegt, wird nicht „vor x Monaten“ angezeigt, sondern wieder das Datum wie bei „absolut“ verwendet.

Ist hier alles drin und geklärt?

schwarzer added inline comments.May 22 2021, 11:40 AM
messages/dolphin/dolphin.po
437

So, habe mir das nun angeschaut.
Was damit gemeint ist, dass hier die "Was ist das?"-Hilfe verwendet wurde, um diesen Text angezeigt zu bekommen. Der Text beschreibt dann, wie man eine Suchleiste anzeigen kann und schlägt vor, dass man die "Was ist das?"-Hilfe eben auf dieser Suchleiste verwendet kann, um weitere Informationen zu erhalten.

Also vielleicht sowas wie "Sie können die Hilfefunktion auch direkt auf der Suchleiste verwenden, um weitere Hinweise zur Benutzung zu beommen." ... Oder so. Fühlt sich noch nicht ganz rund an aber ich denke, in die Richtung geht das.

schwarzer accepted this revision.Jun 13 2021, 7:02 PM

Ist drin.

Aus irgendeinem Grund kann ich das Review hier nicht schließen.

bcooksley changed the visibility from "All Users" to "Public (No Login Required)".Sep 2 2021, 6:07 AM
schwarzer commandeered this revision.Sep 2 2021, 6:13 AM
schwarzer abandoned this revision.
schwarzer edited reviewers, added: lueck; removed: schwarzer.

Ist bereits drin.