trunk: Night Color ergänzt und korrigiert
ClosedPublic

Authored by felf on Apr 1 2020, 7:49 PM.

Details

Summary

Ich habe so weit ich weiß kein SVN-Schreibrecht.

Diff Detail

Repository
R883 Subversion
Lint
Lint Skipped
Unit
Unit Tests Skipped
felf requested review of this revision.Apr 1 2020, 7:49 PM
felf created this revision.
felf created this object with edit policy "Subscribers".
felf updated this revision to Diff 79084.Apr 1 2020, 7:57 PM
felf updated this revision to Diff 79085.

Ich habe zu erst deinen Kommentar nicht gesehen und wollte vorschlagen "Durchgangszeit" auf "Übergangszeit" zu ändern. Also ja ich bin dafür.

Was hältst du von "werden" statt "sind" in "Normale Farben sind wiederhergestellt um %1"?
Auch würde ich "zurückgestellt" durch "zurückgesetzt" ersetzen; in "Nachtfarben werden zurückgestellt ab %1".
Und in "Mit Nachtfarben werden die Farben auf dem Bildschirm wärmer eingestellt, um" würde ich "eingestellt" durch "dargestellt" austauschen.


I would also change "Durchgangszeit" to "Übergangszeit" as you suggested.

What do you think about changing "sind" to "werden" in "Normale Farben sind wiederhergestellt um %1"?
Also I'd suggest using "zurückgesetzt" instead of "zurückgestellt" in "Nachtfarben werden zurückgestellt ab %1".
And in "Mit Nachtfarben werden die Farben auf dem Bildschirm wärmer eingestellt, um" I'd change "eingestellt" to "dargestellt".

felf added a comment.Apr 1 2020, 9:51 PM

Was hältst du von "werden" statt "sind" in "Normale Farben sind wiederhergestellt um %1"?

Da es sich um einen längeren Vorgang handelt, der um %1 Uhr abgeschlossen ist, würde ich „sind“ behalten wollen. „Werden“ suggeriert, dass der Prozess um %1 Uhr beginnt.

Auch würde ich "zurückgestellt" durch "zurückgesetzt" ersetzen; in "Nachtfarben werden zurückgestellt ab %1".

Ich hätte ja am liebsten „zurückgefahren“ genommen, aber das ist nicht alltagssprachtauglich. ;-)
Ich vermisse in dem Diff irgendwie einen anderen Kommentar von mir: ich hätte gerne die Satzstruktur umgedreht, damit ein natürlicher(er) Satz daraus wird:
„Nachtfarben werden ab %1 zurückgesetzt“
Was hälst du davon?

Und in "Mit Nachtfarben werden die Farben auf dem Bildschirm wärmer eingestellt, um" würde ich "eingestellt" durch "dargestellt" austauschen.

Klingt gut.

Ein paar Anmerkungen von mir ...

messages/kde-workspace/kcm_nightcolor.po
122

Einschalten?

154

Vielleicht "Der Morgen" damit es eindeutiger ist?

159

Weißt du, was das für ein Durchgang ist? Passt vielleicht Übergang besser?

175

Nachtfarben werden ab %1 zurückgesetzt?

felf marked 3 inline comments as done.Apr 1 2020, 10:59 PM
felf added inline comments.
messages/kde-workspace/kcm_nightcolor.po
159

Transition time: die Zeit, in der von Kalt auf Warm übergegangen wird. Man kann eine Dauer einstellen, in der der Übergang fließend durchgeführt wird.
Ich würde am liebsten auch den Pluran verwenden, weil es zwei getrennte Übergänge sind (Kalt→Warm, Warm→Kalt), die sich hier aufgrund der fehlerhaften Einstellung überschneiden.

felf updated this revision to Diff 79095.Apr 1 2020, 11:02 PM

Review eingearbeitet.

Ja dann aber bitte ganz richtig; für "Normale Farben sind wiederhergestellt um %1" Futur II: "Normale Farben werden um %1 wiederhergestellt sein" X-)

„Nachtfarben werden ab %1 zurückgesetzt“ finde ich gut!

schwarzer added inline comments.Apr 2 2020, 7:28 AM
messages/kde-workspace/kcm_nightcolor.po
159

Du hast hier meinen Segen fuer den Plural.

felf marked an inline comment as done.Apr 2 2020, 6:29 PM

Ja dann aber bitte ganz richtig; für "Normale Farben sind wiederhergestellt um %1" Futur II: "Normale Farben werden um %1 wiederhergestellt sein" X-)

Das ginge nach meinem laienhaften Sprachverständnis aber nur, wenn es sich um einen konkreten, tatsächlich stattfindenden Vorgang handelt. Dies ist nur eine allgemeine Aussage.

„Nachtfarben werden ab %1 zurückgesetzt“ finde ich gut!

Dann sind wir glaub ich durch mit allen Anmerkungen.

schwarzer accepted this revision.Apr 2 2020, 8:13 PM

Jup. Danke. :)
Kannst du selbst einspielen?

This revision is now accepted and ready to land.Apr 2 2020, 8:13 PM
felf added a comment.Apr 2 2020, 9:30 PM

Ween du mir sagst, wie… das ist das erste Mal, dass ich bei Phabricator so weit gekommen bin. :)
(Und früher hab ich das nicht selbst gemacht.)

In D28493#640433, @felf wrote:

Ween du mir sagst, wie… das ist das erste Mal, dass ich bei Phabricator so weit gekommen bin. :)
(Und früher hab ich das nicht selbst gemacht.)

Da erwischst du mich aber auch etwas auf dem falschen Fuss. Ich habe Phabricator auch noch nciht wirklich benutzt. :D ... Aber wenn ich das richtig verstehe, muss man seine lokalen Aenderungen einspielen. Das geht nicht ueber Phabricator. Oder irre ich mich da?

davidre added a subscriber: davidre.Apr 3 2020, 7:50 AM

Ich habe arc noch nicht mit svn benutzt, aber ich glaube arc commit sollte funktionieren? https://secure.phabricator.com/book/phabricator/article/arcanist_diff/#subversion

felf closed this revision.Apr 4 2020, 11:05 AM
schwarzer reopened this revision.Apr 4 2020, 12:14 PM
This revision is now accepted and ready to land.Apr 4 2020, 12:14 PM
schwarzer closed this revision.Apr 5 2020, 7:51 PM

Committed