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

Schöne Icons für KontoAuswahlDialog #111

Merged

Conversation

JohannMaierhofer
Copy link

@JohannMaierhofer JohannMaierhofer commented Jan 21, 2024

Habe auch "keine Konten" in "alle Konten" umbenannt. Man wählt ja mit dem Button die Anzeige der Buchungen aller Konten aus.
Screenshot_20240121_170017

Habe auch "keine Konten" in "alle Konten" umbenannt. Man wählt ja mit dem Button alle Konten aus.
@willuhn
Copy link
Member

willuhn commented Jan 22, 2024

Für "Übernehmen" und "Abbrechen" gibt es eigentlich bereits Standard-Icons, die direkt von Jameica bereitgestellt werden. "ok.png" und "process-stop.png". Ich würde eher die nehmen - dann ist es einheitlich. Da JVerein auch den Classpath von Jameica sieht, kannst du die Icons einfach verwenden - ohne die Datein in JVerein kopieren zu müssen.

PS: Kannst du den ersten Buchstaben von "übernehmen" und "abbrechen" bitte noch groß schreiben? An anderen Stellen ist es auch groß geschrieben.

@dippeal dippeal added the enhancement New feature or request label Jan 22, 2024
@MSchmalzl
Copy link

PS: Kannst du den ersten Buchstaben von "übernehmen" und "abbrechen" bitte noch groß schreiben? An anderen Stellen ist es auch groß geschrieben.

Das wäre auch noch so eine Sache die wir im ganzen Plug-in mal angehen sollten. Es mag zwar von der Rechtschreibung her korrekt sein, ich kenne aber kein Programm welches die Menüpunkte mit einem Kleinbuchstaben beginnt. Ich bin also auch für die Großschreibung des ersten Buchstabens.

@dippeal
Copy link
Member

dippeal commented Jan 22, 2024

PS: Kannst du den ersten Buchstaben von "übernehmen" und "abbrechen" bitte noch groß schreiben? An anderen Stellen ist es auch groß geschrieben.

Das wäre auch noch so eine Sache die wir im ganzen Plug-in mal angehen sollten. Es mag zwar von der Rechtschreibung her korrekt sein, ich kenne aber kein Programm welches die Menüpunkte mit einem Kleinbuchstaben beginnt. Ich bin also auch für die Großschreibung des ersten Buchstabens.

+1

@willuhn
Copy link
Member

willuhn commented Jan 22, 2024

Zumindest für "Abbrechen" gibt es sogar einen fertigen Button direkt in Jameica. Ein "buttons.addButton(new Cancel())" reicht.

@JohannMaierhofer
Copy link
Author

Ich stimme dem auch zu. Texte in Buttons müssen groß geschrieben werden.
Der gleiche Fehler besteht auch in:
KontoauszugZuordnungDialog, BuchungenMitgliedskontenZuordnungDialog, BuchungsartZuordnungDialog und ProjektAuswahldialog.
Ich werde diese auch anpassen.
Bezüglich der Icons habe ich die gleichen ja auch in diesen Dialogen eingebaut. Die waren in BuchungenMitgliedskontenZuordnungDialog bereits verwendet und ich wollte zumindest in JVerein konsistent sein. Aber es ist schon besser mit Hibiscus einheitlich zu sein.
Ich werde die vorgeschlagenen Icons verwenden.

PS: Auch die Icons im Baum ganz links sollte man anschauen. Mir fällt immer das unterschiedliche Icon für den Ordner auf. Die sind ja auch noch gleichzeitig sichtbar. Da müsste ein Experte mal alle Icons mit gleicher Funktionalität im Baum an Hibiscus Icons anpassen.

JohannMaierhofer added a commit to JohannMaierhofer/jverein that referenced this pull request Jan 22, 2024
@MSchmalzl MSchmalzl merged commit 2051de0 into openjverein:master Jan 22, 2024
@JohannMaierhofer JohannMaierhofer deleted the kontoauswahldialogicons branch January 22, 2024 09:41
willuhn pushed a commit that referenced this pull request Jan 26, 2024
* Feature: Unterdrücke Buchungsarten in Dialogen

Das Feature unterdrückt in Auswahldialogen Buchungsarten die seit mehr als 2 Jahren nicht benutzt wurden.
Das Feature ist in den Einstellungen auswählbar.

* Konfigurierbares Intervall

Im Feature zur Unterdrückung ungenutzter Buchungsarten ist jetzt der Zeitraum in Monaten einstellbar.

* Exception entfernt in BuchungZuordnungDialog

Exception entfernt in BuchungZuordnungDialog wenn abbrechen gewählt wird.
Deprecated Long ersetzt.

* Abbruch ohne Meldung

Abbruch gibt keine Meldung mehr aus so wie es auch bei vielen anderen Dialogen ist.
Funktioniert analog bei ESC und dem Schließen Icon oben rechts.
Icons zu den Buttons hinzugefügt analog Mitglieder Konto Auswahl Dialog.

* replace isInstance with instanceof

* Update BuchungBuchungsartZuordnungAction.java

Vorschlag von Willuhn implementiert

* Entfernen Button implementiert

* Button text groß und Icons wie für #111

* Entfernen Button Icon wie Löschen Icon in Hibiscus

* Abort Handling vereinfacht

* Kommentar von dippeal implementiert

* Leerzeichen eingefügt
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.

4 participants