Differential D31354

messages/calendarsupport
ClosedPublic
Actions

Authored by daphipz on Sat, Oct 3, 10:05 PM.

Details

Reviewers
kde-i18n-de
ewaldmuller
bxela

Diff Detail

Repository
R883 Subversion
Lint
Lint Skipped
Unit
Unit Tests Skipped
daphipz requested review of this revision.Sat, Oct 3, 10:05 PM
daphipz created this revision.
ewaldmuller accepted this revision.Sun, Oct 4, 10:28 AM
ewaldmuller added a subscriber: ewaldmuller.

Passt, danke Philipp!

This revision is now accepted and ready to land.Sun, Oct 4, 10:28 AM
bxela requested changes to this revision.Sun, Oct 4, 7:53 PM
bxela added a subscriber: bxela.
bxela added inline comments.
messages/calendarsupport/calendarsupport6.po
496

Wenn ich mich in die Rolle eines Benutzers begebe und mir die Erklärungen in Klammern von diesem und vorherigen Eintrag durchlese, dann bin ich irritiert: Beide sagen das Gleiche aus. Verbesserungsvorschlag für aktuelle Eintrag: "Vertraulich (für Sie and bestimmte weitere Personen zugänglich)" → engl. "Confidential (accessible to you and certain further persons)"

521

s. o. bzgl. Erklärungstext zu „Vertraulich“

This revision now requires changes to proceed.Sun, Oct 4, 7:53 PM
daphipz updated this revision to Diff 87022.Mon, Oct 5, 3:08 AM
daphipz marked 2 inline comments as done.
daphipz added inline comments.
messages/calendarsupport/calendarsupport6.po
496

Das scheint mir ein Fehler im Code zu sein: https://invent.kde.org/pim/calendarsupport/-/blob/master/src/kcalprefs.kcfg

Dieser String sollte glaube ich glein wie der andere lauten (some others), dann müsste es passen.

daphipz marked 2 inline comments as done.Mon, Oct 5, 2:59 PM
daphipz added inline comments.
messages/calendarsupport/calendarsupport6.po
496

https://invent.kde.org/pim/calendarsupport/-/merge_requests/127

bxela requested changes to this revision.Mon, Oct 5, 8:25 PM

Diese Einreichung passt, trotzdem wähle ich hier "Request Changes", damit mein Kommentar "Kommentar zum MR127" nicht übersehen wird.

messages/calendarsupport/calendarsupport6.po
496

Finde ich nicht. Man kann auf eine Zeilennummer rechts klicken und dann auf „Link-Adresse kopieren" → Ergebnis z. B. https://invent.kde.org/pim/calendarsupport/-/blob/master/src/kcalprefs.kcfg#L20 → dann gelangt man per Klick auf so eine Verknüpfung direkt zur entsprechenden Stelle.

496

O. K., jetzt sehe ich, warum ich das alte engl. Original nicht finde. MR127 wurde zusammengeführt. Kommentar zum MR127: Kann es sein, das nicht
"(accessible to you and select others)", sondern
"(accessible to you and selected others)" beabsichtigt wurde?

This revision now requires changes to proceed.Mon, Oct 5, 8:25 PM
daphipz requested review of this revision.Tue, Oct 6, 10:06 AM
daphipz marked 4 inline comments as done.
daphipz added inline comments.
messages/calendarsupport/calendarsupport6.po
496

Kann es sein, das nicht "(accessible to you and select others)", sondern "(accessible to you and selected others)" beabsichtigt wurde?

"select" ist auch als Adjektiv nutzbar (ausgewählt): https://dict.leo.org/englisch-deutsch/select

496

dann gelangt man per Klick auf so eine Verknüpfung direkt zur entsprechenden Stelle.

Ich habe absichtlich auf die gesamte Datei verlinkt, weil der String dort drei Mal in den Blöcken für To-Dos, Journals und Events vorkommt :)

bxela accepted this revision.Tue, Oct 6, 6:49 PM

Passt, danke Philipp!

messages/calendarsupport/calendarsupport6.po
496

Dass „select“ auch als Adjektiv verwendet werden kann, wusste ich nicht, danke für den Tipp!
Wenn du meinst, dass „select others“ für Muttersprachler nicht komisch erscheint (kann ich nicht beurteilen) und die Nichtmuttersprachler auch sofort verstehen, dass „ausgewählte Andere“ gemein ist (ich wäre nicht drauf gekommen), dann lassen wir es so.

This revision is now accepted and ready to land.Tue, Oct 6, 6:49 PM
daphipz closed this revision.Wed, Oct 7, 11:46 AM
daphipz marked 2 inline comments as done.

Eingespielt mit https://websvn.kde.org/?revision=1749520&view=revision

Revision Contents
Changeset List

Diff 87022

View Options

messages/calendarsupport/calendarsupport6.po

Loading...