openjverein / jverein

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

Eigenschaften von einzelnen Mitgliedern über Kontextmenü bearbeiten #370

Open tolot27 opened 1 month ago

tolot27 commented 1 month ago

Es wäre richtig cool, wenn man die Eigenschaften eines einzelnen Mitglieds über das Kontextmenü der Mitgliederliste und den daraufhin erscheinenden Dialog bearbeiten könnte. Momentan kann dieser Dialog ja nur zum Setzen von Eigenschaften, insbesondere bei mehreren Mitgliedern verwendet werden. Wenn jedoch nur ein Mitglied ausgewählt ist, könnte er die bereits gesetzten Eigenschaften anzeigen und so ein echtes Bearbeiten ermöglichen.

tolot27 commented 1 month ago

Die Funktion bzw. das Vorhandensein der Checkboxen an den inneren Knoten (inkl. root) erschließt sich mir auch nicht so ganz. Für was sind sie da? Wenn sie wenigstens die States der darunterliegenden Checkboxen ändern würden (alle setzen, alle löschen bzw. zurücksetzen auf vorherigen State bevor man das erste Mal drauf klickt), wäre deren Daseinsberechtigung gerechtfertigt. 😏

image

JohannMaierhofer commented 1 month ago

Wenn jedoch nur ein Mitglied ausgewählt ist, könnte er die bereits gesetzten Eigenschaften anzeigen und so ein echtes Bearbeiten ermöglichen.

Ich denke, es könnte etwas verwirrend für die Anwender sein wenn der Menüpunkt bei einfacher Selektion ein anderes Verhalten hat als bei mehrfach Selektion. Ich würde darum davon abraten so etwas zu machen. Wenn, dann sollten es unterschiedliche Menüeinträge sein.

lenilsas commented 1 month ago

Ich finde auch, dass das Verhalten überall gleich sein sollte. Gibt es auch die Möglichkeit Checkboxes o.ä. mit drei Zuständen zu machen? Damit wäre es möglich Eigenschaften zu entfernen. Ich fände es auch sinnvoll wenn man mehrere Mitglieder-Eigenschaften gleichzeitig bearbeiten könnte. Auch ist es bisher nicht möglich Eigenschaften mit nur eine erlaubten Eintrag über das Contextmenu zu bearbeiten. Im Forum gab es auch die Frage nach einem Filter "ohne Eigenschaft", das währe auch mit einem 3-Zustände Checkobx möglich.

JohannMaierhofer commented 1 month ago

Für das gleichzeitige bearbeiten mehrerer Einträge gäbe es noch eine Alternative. Man könnte einen CSV Export der Eigenschaften selektierter Mitglieder implementieren. In der Tabellenkalkulations Software kann man leicht Änderungen vornehmen. Mit einem Import könnte man die neuen Werte wieder einlesen. Mit dem Import könnte man auch anderweitig generierte Tabellen einlesen. Das dürfte wahrscheinlich einfacher zu implementieren sein und wäre auch so nützlich. Es verbaut auch nicht den Weg es später innerhalb Jverein zu implementieren falls dann noch Bedarf besteht.

JohannMaierhofer commented 1 month ago

Das Problem mit dem Filter ohne Eigenschaft besteht natürlich weiter.

JohannMaierhofer commented 1 month ago

Die Funktion bzw. das Vorhandensein der Checkboxen an den inneren Knoten (inkl. root) erschließt sich mir auch nicht so ganz. Für was sind sie da? Wenn sie wenigstens die States der darunterliegenden Checkboxen ändern würden (alle setzen, alle löschen bzw. zurücksetzen auf vorherigen State bevor man das erste Mal drauf klickt), wäre deren Daseinsberechtigung gerechtfertigt. 😏

Es gibt schon ein alle setzen, alle löschen, allerdings scheint das nur sporadisch zu funktionieren. Manchmal bei klicken werden bei mir alle gesetzt. Alle löschen geht ohne Probleme allerdings ist dann aber ein Haken im Folder.

Ich werde die Checkboxen aber sowieso noch entfernen, darum sollte das jetzt egal sein.

JohannMaierhofer commented 1 month ago

Ich finde auch, dass das Verhalten überall gleich sein sollte. Gibt es auch die Möglichkeit Checkboxes o.ä. mit drei Zuständen zu machen? Damit wäre es möglich Eigenschaften zu entfernen.

Ich mache das auch noch. Mein Plan ist mit fünf Zuständen.

Die ersten drei Zustände werden automatisch bestimmt. Die anderen wählt man dann manuell.

Ich hoffe das ist so hilfreich.

JohannMaierhofer commented 1 month ago

Der erste Schritt ist in #399 gemacht.

JohannMaierhofer commented 2 weeks ago

Mit #426 unterstützt der Eigenschaften Dialog auch Eigenschaften Gruppen mit Pflicht und Max1.