Zettelkasten-Team / Zettelkasten

Zettelkasten-Developer-Builds
http://zettelkasten.danielluedecke.de
GNU General Public License v3.0
733 stars 94 forks source link

Dialog „Aktuellere Sicherheitskopie vorhanden“ sollte intuitiver sein #361

Closed trutzig89182 closed 3 years ago

trutzig89182 commented 3 years ago

Wenn nach einem Absturz auf eine aktuellere Sicherheitskopie hingewiesen wird, gibt es die Möglichkeit „Ja“, „Nein“ und „Abrechen“ auszuwählen. Ja ist dabei als Default-Option blau hinterlegt. Zwar wird die Bedeutung der einzelnen Optionen im Text des Dialogfensters erklärt, aber sie ergibt sich nicht direkt aus der Button-Bezeichnung. Ewarten würde ich zudem eher, dass „Ja“ die Sicherungskopie öffnet, wenn Kern des Dialogfensters ist, mich auf diese hinzuweisen. Auch ist es von den Konsequenzen her sinnvoll, die Vorauswahl auf das Öffnen der Sicherheitskopie festzulegen, da dabei keine Daten verloren gehen (Kopie wird angelegt). Ich muss selbst bei dem Dialogfenster jedes mal kurz innehalten, weil ich auch schon mal fälschlicherweise die Sicherungskopie verworfen habe.

To Reproduce Open Zettelkasten after crash before new data could be saved.

Expected behavior More intuitive dialogue for the live backup.

Screenshots Sicherungskopie

Version information (please complete the following information):

trutzig89182 commented 3 years ago

Der Text ließe sich ja relativ einfach ändern. Sind Zeile 646–647 in: Zettelkasten/src/main/resources/de/danielluedecke/zettelkasten/resources/ZettelkastenView.properties

Mein Vorschlag wäre etwas in die Richtung:

newerBackupMsg=<html>F\u00FCr die zu \u00F6ffnende Datendatei existiert eine Sicherungskopie, die aktueller ist.<br><br>Das bedeutet, dass bei der letzten Verwendung des Zettelkastens vorgenommene<br>\u00C4nderungen beim Beenden nicht gesichert wurden.<br><br>M\u00F6chten Sie<br>- die aktuellere Sicherungskopie \u00F6ffnen oder mit der gespeicherten Datendatei fortfahren? <br><br>Wenn Sie unsicher sind, wie Sie vorgehen sollen, w\u00E4hlen Sie <b>Sicherungskopie öffnen</b> aus.<br>Von Ihrer Datendatei wird dann automatisch eine Kopie angelegt, sodass diese Daten nicht verloren gehen.</html>
newerBackupTitle=Aktuellere Sicherungskopie vorhanden

Optionen sollten da sein: „Sicherungskopie öffnen“, „Datendatei öffnen“, „Abbrechen“.

Allerdings müsste das natürlich auch in den unterschiedlichen Sprachen geschehen in: Zettelkasten/src/main/resources/de/danielluedecke/zettelkasten/resources/ZettelkastenView_en_GB.properties Zettelkasten/src/main/resources/de/danielluedecke/zettelkasten/resources/ZettelkastenView_es_ES.properties Zettelkasten/src/main/resources/de/danielluedecke/zettelkasten/resources/ZettelkastenView_pt_BR.properties

Die Buttons sind bisher über JOptionPane.YES_NO_CANCEL_OPTION gelöst. Das ist natürlich sehr praktisch und vielleicht darum der Grund, warum der Dialogtext da etwas drumherum gebogen ist. Zettelkasten/src/main/java/de/danielluedecke/zettelkasten/ZettelkastenView.java

RalfBarkow commented 3 years ago

Default sollte in diesem Fall wohl Nein sein, nicht wahr?

RalfBarkow commented 3 years ago

@trutzig89182 see How to Make Dialogs , a section in The Java Tutorial.

RalfBarkow commented 3 years ago

@trutzig89182 Hast Du eine Idee, wie man das in der Klasse Zettelkastenview fixen könnte? (s. obigen commit)

RalfBarkow commented 3 years ago

Cf. Changing Default of JOptionPane.YES_NO_OPTION while maintaining keyboard shortcuts?

trutzig89182 commented 3 years ago

Default „No“ wäre schon mal besser. Sinnvoller wäre es aber m.E. noch, wenn man nicht mit Ja/Nein-arbeitet, die man dann umständlich erklären muss. Ich werde am Wochenende mal versuchen ein bisschen reinzuschauen, kann aber nichts versprechen, da meine Java-Kenntnisse eher bescheiden sind.

trutzig89182 commented 3 years ago

Habe mal einen Aufschlag gemacht. Das Dialogfenster sieht jetzt so aus:

BackupOptions

Dabei ist die Belegung der Buttons geändert, so dass die Texte und Optionen in den anderen Sprachen notwendig geändert werden müssen, bevor man das einbauen kann.

Ich denke aber, wir sollten erst einmal abchecken, ob der Text so klar ist und ob man evtl noch was ändern sollte, bevor wir das in den anderen Sprachen anpassen.

Ich habe es mal in meine Fork commitet. Damit Du mal drüber schauen kannst. d879435ca93e1319075488d95801f006ec1ba08b

RalfBarkow commented 3 years ago

@trutzig89182 übernehme ich gerne! Vielen Dank.

Sprachen habe ich angepasst. Okay?

RalfBarkow commented 3 years ago

Aktueller Build: https://github.com/Zettelkasten-Team/Zettelkasten/actions/runs/931386194

trutzig89182 commented 3 years ago

Mit der Anpassung der Sprachen hatte ich auch angefangen, aber habe sie dann jetzt wieder rausgenommen. Habe noch einen Typo gefunden und den Code in ZettelkastenView.java etwas klarer strukturiert. Commit: d71b04458d7c9a9099403f162495b2dd5d10fa2e

Pull request für beides macht vermutlich keinen Sinn, oder?

RalfBarkow commented 3 years ago

@trutzig89182 Danke! Und bitte PR auf den release/v3.2.8