search

openjverein / jverein

Open JVerein - Open Source Vereinsverwaltung
https://openjverein.github.io
GNU General Public License v3.0
43 stars 17 forks source link

Automatische Höhenanpassung in Dialogen #181

Closed MSchmalzl closed 8 months ago

MSchmalzl commented 8 months ago

Die Höhe von Dialogen wird automatisch an den Inhalt angepasst.

willuhn commented 8 months ago

Hast du getestet, ob das überall funktioniert? Immer dann, wenn der Inhalt de Dialogs eine variable Höhe haben kann, weil er z.B. eine Tabelle enthält dann funktioniert ein setSize(..., SWT.DEFAULT) nicht, weil SWT dann nicht weiss, wie hoch die Tabelle angezeigt werden soll. Mit dem Effekt, dass der Dialog so hoch wird, wie die ganze ungescrollte Tabelle. Wenn die nur eine Zeile enthält, ist der Dialog dann zu flach und wenn die Tabelle viele Zeilen enthält, wird der Dialog höher als der Bildschirm.

JohannMaierhofer commented 8 months ago

Der Kommentar ist gut. Ich habe nicht alle überprüft. Der MitgliedskontoAuswahlDialog könnte ein solcher Fall sein. Da sollte man die Größe so lassen wie sie war. Sonst fällt mir noch ein der MailVorlagenAuswahlDialog, MailEmpfängerDialog, EigenschaftenAuswahlDialog, SammelüberweisungAuswahlDialog

MSchmalzl commented 8 months ago

Die, die ich testen konnte habe ich getestet. Dürften so 95% gewesen sein. Da waren auch welche mit Tabellen dabei. Nachdem ich in der Testumgebung nicht so viele Daten habe ist das nicht aufgefallen.

Ich bessere nach.

MSchmalzl commented 8 months ago

Habe die Änderungen in Dialogen mit Tabellen rückgängig gemacht. Bitte mal kontrollieren ob es so passt.

JohannMaierhofer commented 8 months ago

Die Änderungen sind meiner Meinung nach ok. Den SplitBuchungDialog und den DokumentDialog habe ich zum Testen nicht gefunden wo die aufgerufen werden. Werden die überhaupt verwendet? Der About Dialog schaut bei mir etwas seltsam groß aus, war aber auch vorher schon so. Siehe unten. Mir ist der aktuelle KontoAuswahlDialog etwas zu klein, ich muss immer scrollen. Könnte man den auf 350 vergrößern. Ich denke man könnte aber auch gleich SWT.DEFAULT nehmen weil wohl keiner so viele Konten hat als dass sie nicht auf eine Bildschirmseite passen würden.

Bild7

MSchmalzl commented 8 months ago

Den SplitBuchungDialog und den DokumentDialog habe ich zum Testen nicht gefunden wo die aufgerufen werden. Werden die überhaupt verwendet?

Den SplitbuchungDialog habe ich auch nicht gefunden. Denke der wird nicht mehr verwendet. Den DokumentDialog gibt es meiner Meinung nur wenn man die Dokumentenarchivierung verwendet. https://openjverein.gitbook.io/doku/sonstiges/dokumente

Der About Dialog schaut bei mir etwas seltsam groß aus, war aber auch vorher schon so. Siehe unten.

Finde auch dass das komisch aussieht. Habe aber nicht herausgefunden warum das so ist.

Mir ist der aktuelle KontoAuswahlDialog etwas zu klein, ich muss immer scrollen. Könnte man den auf 350 vergrößern. Ich denke man könnte aber auch gleich SWT.DEFAULT nehmen weil wohl keiner so viele Konten hat als dass sie nicht auf eine Bildschirmseite passen würden.

Habe ich geändert.

dippeal commented 8 months ago

Die von mir geprüften Dialoge sehen gut aus. Willst du den "Über" Dialog noch in der Höhe anpassen? Ist es möglich/sinnvoll die Dialoge in der Mitte des Hauptfensters zu öffnen?

MSchmalzl commented 8 months ago

Die von mir geprüften Dialoge sehen gut aus. Willst du den "Über" Dialog noch in der Höhe anpassen?

Habe ich noch gemacht.

Ist es möglich/sinnvoll die Dialoge in der Mitte des Hauptfensters zu öffnen?

Ich hatte bisher noch nicht das Problem das ich über einen nicht richtig positionierten Dialog gestolpert bin. Hätte von daher die Änderung derzeit nicht weiter vorangetrieben.