Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sollbuchungen Menu Update #255

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

JohannMaierhofer
Copy link

@JohannMaierhofer JohannMaierhofer commented Jul 3, 2024

Im Sollbuchungen View habe ich das Context Menu erweitert. Bei einem doppel Klick auf eine Sollbuchung hätte ich erwartet, dass man die Sollbuchung editieren kann. Statt dessen wird aber das Mitglied geöffnet.
Ich habe jetzt die doppel Klick Action auch auf das Context Menü gelegt und zusätzlich die Möglichkeit die Sollbuchung zu editieren oder zu löschen. Man muss dann nicht immer erst in das Mitgliedkonto des Mitglieds gehen.
Der Check für Löschen ist wie im Mitgliedskonto des Mitglieds.

Menü alt:
Bildschirmfoto_20240703_153042

Menü neu:
Bildschirmfoto_20240707_110126

PS: Das Löschen Icon habe ich auch im Context Menü im Mitgliedskonto geändert damit die Menüs gleich aussehen.
Bildschirmfoto_20240703_154122

@JohannMaierhofer JohannMaierhofer added the enhancement New feature or request label Jul 3, 2024
@dippeal
Copy link
Member

dippeal commented Jul 7, 2024

Kleinigkeit, aber wenn man an der Datei schon dran ist würde ich das "Menu neu" folgendermaßen aufbauen:

  1. Sollbuchung bearbeiten
  2. Sollbuchung löschen
  3. Trennstrich
  4. Mitglied anzeigen
  5. Rechnung erstellen
  6. Mahnung erstellen
    (Die drei Punkte hinter Rechnung und Mahnung ist wenig aussagekräftig.)

@JohannMaierhofer
Copy link
Author

Menü ist geändert, schaut jetzt besser aus.
Bildschirmfoto_20240707_110126

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants