Button Korrektur für alle Dialoge (#114)

Buttons für alle Dialoge unter de.jost_net.JVerein.gui.dialogs angepasst: - Großschrift - Icons für Ok, Abbrechen etc. eingefügt - Entfernen/Löschen Icon durch user-trash-full.png ersetzt. Dieses wird auch in Hibiscus dafür verwendet
openjverein · Jan 23, 2024 · 11028df · 11028df
1 parent 26b3d02
commit 11028df
Show file tree

Hide file tree

Showing 20 changed files with 89 additions and 84 deletions.
diff --git a/src/de/jost_net/JVerein/gui/dialogs/BuchungenMitgliedskontenZuordnungVorschauDialog.java b/src/de/jost_net/JVerein/gui/dialogs/BuchungenMitgliedskontenZuordnungVorschauDialog.java
@@ -117,7 +117,7 @@ public void handleAction(Object context)
       {
         close();
       }
-    }, null, false, "window-close.png");
+    }, null, false, "process-stop.png");
     b.paint(parent);
   }
 

diff --git a/src/de/jost_net/JVerein/gui/dialogs/BuchungsjournalSortDialog.java b/src/de/jost_net/JVerein/gui/dialogs/BuchungsjournalSortDialog.java
@@ -17,6 +17,7 @@
 
 package de.jost_net.JVerein.gui.dialogs;
 
+import org.eclipse.swt.SWT;
 import org.eclipse.swt.widgets.Composite;
 import org.eclipse.swt.widgets.Event;
 import org.eclipse.swt.widgets.Listener;
@@ -60,7 +61,7 @@ public BuchungsjournalSortDialog(int position)
     super(position);
 
     setTitle("Buchungsjournal-Sortierung");
-    setSize(300, 200);
+    setSize(300, 140);
   }
 
   @Override
@@ -69,22 +70,22 @@ protected void paint(Composite parent) throws Exception
     LabelGroup options = new LabelGroup(parent, "Ihre Auswahl");
     options.addInput(this.getSortierung());
     ButtonArea b = new ButtonArea();
-    b.addButton("weiter", new Action()
+    b.addButton("Weiter", new Action()
     {
       @Override
       public void handleAction(Object context)
       {
         close();
       }
-    });
-    b.addButton("abbrechen", new Action()
+    }, null, false, "go-next.png");
+    b.addButton("Abbrechen", new Action()
     {
       @Override
       public void handleAction(Object context)
       {
         throw new OperationCanceledException();
       }
-    });
+    }, null, false, "process-stop.png");
     b.paint(parent);
   }
 

diff --git a/src/de/jost_net/JVerein/gui/dialogs/DokumentDialog.java b/src/de/jost_net/JVerein/gui/dialogs/DokumentDialog.java
@@ -53,7 +53,7 @@ protected void paint(Composite parent) throws Exception
   {
     getPart().paint(parent);
     ButtonArea b = new ButtonArea();
-    b.addButton("speichern", new Action()
+    b.addButton("Speichern", new Action()
     {
       @Override
       public void handleAction(Object context)
@@ -70,15 +70,15 @@ public void handleAction(Object context)
         }
         close();
       }
-    });
-    b.addButton("abbrechen", new Action()
+    }, null, true, "ok.png");
+    b.addButton("Abbrechen", new Action()
     {
       @Override
       public void handleAction(Object context)
       {
         throw new OperationCanceledException();
       }
-    });
+    }, null, false, "process-stop.png");
     b.paint(parent);
   }
 

diff --git a/src/de/jost_net/JVerein/gui/dialogs/EigenschaftenAuswahlDialog.java b/src/de/jost_net/JVerein/gui/dialogs/EigenschaftenAuswahlDialog.java
@@ -120,7 +120,7 @@ public void handleAction(Object context)
         }
         close();
       }
-    });
+    }, null, true, "ok.png");
     buttons.paint(parent);
   }
 

diff --git a/src/de/jost_net/JVerein/gui/dialogs/ExportDialog.java b/src/de/jost_net/JVerein/gui/dialogs/ExportDialog.java
@@ -135,7 +135,7 @@ public void handleAction(Object context)
       {
         close();
       }
-    }, null, false, "stop-circle.png");
+    }, null, false, "process-stop.png");
     group.addButtonArea(buttons);
 
     getShell()

diff --git a/src/de/jost_net/JVerein/gui/dialogs/FamilienmitgliedEntfernenDialog.java b/src/de/jost_net/JVerein/gui/dialogs/FamilienmitgliedEntfernenDialog.java
@@ -82,7 +82,7 @@ protected void paint(Composite parent) throws Exception
     lgBank.addLabelPair("BIC", control.getBic());
     // lgBank.addLabelPair("Kontoinhaber", control.getKontoinhaber());
     ButtonArea b = new ButtonArea();
-    b.addButton("weiter", new Action()
+    b.addButton("Weiter", new Action()
     {
 
       @Override
@@ -130,16 +130,16 @@ public void handleAction(Object context)
           return;
         }
       }
-    });
-    b.addButton("abbrechen", new Action()
+    }, null, true, "go-next.png");
+    b.addButton("Abbrechen", new Action()
     {
 
       @Override
       public void handleAction(Object context)
       {
         throw new OperationCanceledException();
       }
-    });
+    }, null, false, "process-stop.png");
     b.paint(parent);
   }
 

diff --git a/src/de/jost_net/JVerein/gui/dialogs/ImportDialog.java b/src/de/jost_net/JVerein/gui/dialogs/ImportDialog.java
@@ -138,7 +138,7 @@ public void handleAction(Object context)
       {
         throw new OperationCanceledException();
       }
-    }, null, false, "stop-circle.png");
+    }, null, false, "process-stop.png");
     group.addButtonArea(buttons);
     getShell()
         .setMinimumSize(getShell().computeSize(WINDOW_WIDTH, SWT.DEFAULT));

diff --git a/src/de/jost_net/JVerein/gui/dialogs/KontoauszugZuordnungDialog.java b/src/de/jost_net/JVerein/gui/dialogs/KontoauszugZuordnungDialog.java
@@ -113,7 +113,7 @@ public void handleAction(Object context)
         ueberschr = true;
         close();
       }
-    }, null, false, "undo.png");
+    }, null, false, "user-trash-full.png");
     buttons.addButton("Abbrechen", new Action()
     {
 

diff --git a/src/de/jost_net/JVerein/gui/dialogs/MailEmpfaengerAuswahlDialog.java b/src/de/jost_net/JVerein/gui/dialogs/MailEmpfaengerAuswahlDialog.java
@@ -49,7 +49,7 @@ public MailEmpfaengerAuswahlDialog(MailControl control, int position)
     super(position);
     this.control = control;
     setTitle("Mail-Empfänger");
-    setSize(1000, 450);
+    setSize(650, 450);
   }
 
   @Override
@@ -92,23 +92,17 @@ public void handleAction(Object context) throws ApplicationException
         }
       }
     });
-
-    b.addButton("aktive Mitglieder", new Action()
+    
+    b.addButton("Alle", new Action()
     {
       @Override
       public void handleAction(Object context)
       {
         try
         {
-          Date stichtag = new Date();
           for (Object o : control.getMitgliedMitMail().getItems(false))
           {
-            Mitglied m = (Mitglied) o;
-            if (m.getAdresstyp().getJVereinid() == 1
-                && m.isAngemeldet(stichtag))
-            {
-              control.getMitgliedMitMail().setChecked(o, true);
-            }
+            control.getMitgliedMitMail().setChecked(o, true);
           }
         }
         catch (RemoteException e)
@@ -118,22 +112,16 @@ public void handleAction(Object context)
       }
     });
 
-    b.addButton("inaktive Mitglieder", new Action()
+    b.addButton("Keinen", new Action()
     {
       @Override
       public void handleAction(Object context)
       {
         try
         {
-          Date stichtag = new Date();
           for (Object o : control.getMitgliedMitMail().getItems(false))
           {
-            Mitglied m = (Mitglied) o;
-            if (m.getAdresstyp().getJVereinid() == 1
-                && !m.isAngemeldet(stichtag))
-            {
-              control.getMitgliedMitMail().setChecked(o, true);
-            }
+            control.getMitgliedMitMail().setChecked(o, false);
           }
         }
         catch (RemoteException e)
@@ -142,18 +130,20 @@ public void handleAction(Object context)
         }
       }
     });
-
-    b.addButton("alle Adressen", new Action()
+    
+    b.addButton("Aktive Mitglieder", new Action()
     {
       @Override
       public void handleAction(Object context)
       {
         try
         {
+          Date stichtag = new Date();
           for (Object o : control.getMitgliedMitMail().getItems(false))
           {
             Mitglied m = (Mitglied) o;
-            if (m.getAdresstyp().getJVereinid() != 1)
+            if (m.getAdresstyp().getJVereinid() == 1
+                && m.isAngemeldet(stichtag))
             {
               control.getMitgliedMitMail().setChecked(o, true);
             }
@@ -166,7 +156,7 @@ public void handleAction(Object context)
       }
     });
 
-    b.addButton("aktive Mitglieder und Adressen", new Action()
+    b.addButton("Inaktive Mitglieder", new Action()
     {
       @Override
       public void handleAction(Object context)
@@ -177,9 +167,8 @@ public void handleAction(Object context)
           for (Object o : control.getMitgliedMitMail().getItems(false))
           {
             Mitglied m = (Mitglied) o;
-            if (m.getAdresstyp().getJVereinid() != 1
-                || (m.getAdresstyp().getJVereinid() == 1
-                    && m.isAngemeldet(stichtag)))
+            if (m.getAdresstyp().getJVereinid() == 1
+                && !m.isAngemeldet(stichtag))
             {
               control.getMitgliedMitMail().setChecked(o, true);
             }
@@ -192,10 +181,7 @@ public void handleAction(Object context)
       }
     });
 
-    b.paint(parent);
-
-    ButtonArea c = new ButtonArea();
-    c.addButton("alle", new Action()
+    b.addButton("Alle Adressen", new Action()
     {
       @Override
       public void handleAction(Object context)
@@ -204,7 +190,11 @@ public void handleAction(Object context)
         {
           for (Object o : control.getMitgliedMitMail().getItems(false))
           {
-            control.getMitgliedMitMail().setChecked(o, true);
+            Mitglied m = (Mitglied) o;
+            if (m.getAdresstyp().getJVereinid() != 1)
+            {
+              control.getMitgliedMitMail().setChecked(o, true);
+            }
           }
         }
         catch (RemoteException e)
@@ -214,16 +204,23 @@ public void handleAction(Object context)
       }
     });
 
-    c.addButton("keinen", new Action()
+    b.addButton("Aktive Mitglieder und Adressen", new Action()
     {
       @Override
       public void handleAction(Object context)
       {
         try
         {
+          Date stichtag = new Date();
           for (Object o : control.getMitgliedMitMail().getItems(false))
           {
-            control.getMitgliedMitMail().setChecked(o, false);
+            Mitglied m = (Mitglied) o;
+            if (m.getAdresstyp().getJVereinid() != 1
+                || (m.getAdresstyp().getJVereinid() == 1
+                    && m.isAngemeldet(stichtag)))
+            {
+              control.getMitgliedMitMail().setChecked(o, true);
+            }
           }
         }
         catch (RemoteException e)
@@ -232,7 +229,11 @@ public void handleAction(Object context)
         }
       }
     });
-    c.addButton("übernehmen", new Action()
+
+    b.paint(parent);
+
+    ButtonArea c = new ButtonArea();
+    c.addButton("Übernehmen", new Action()
     {
 
       @Override
@@ -255,16 +256,16 @@ public void handleAction(Object context)
         }
         close();
       }
-    });
+    }, null, true, "ok.png");
 
-    c.addButton("abbrechen", new Action()
+    c.addButton("Abbrechen", new Action()
     {
       @Override
       public void handleAction(Object context)
       {
         throw new OperationCanceledException();
       }
-    });
+    }, null, false, "process-stop.png");
 
     c.paint(parent);
   }

diff --git a/src/de/jost_net/JVerein/gui/dialogs/MailVorlagenAuswahlDialog.java b/src/de/jost_net/JVerein/gui/dialogs/MailVorlagenAuswahlDialog.java
@@ -53,7 +53,7 @@ protected void paint(Composite parent) throws Exception
     control.getMailVorlageTable().paint(parent);
 
     ButtonArea b = new ButtonArea();
-    b.addButton("verwenden", new Action()
+    b.addButton("Verwenden", new Action()
     {
 
       @Override
@@ -69,16 +69,16 @@ public void handleAction(Object context) throws ApplicationException
         }
         close();
       }
-    });
-    b.addButton("ohne Mail-Vorlage", new Action()
+    }, null, true, "ok.png");
+    b.addButton("Ohne Mail-Vorlage", new Action()
     {
 
       @Override
       public void handleAction(Object context)
       {
         close();
       }
-    });
+    }, null, true, "go-next.png");
     b.paint(parent);
   }
 

diff --git a/src/de/jost_net/JVerein/gui/dialogs/MailVorschauDialog.java b/src/de/jost_net/JVerein/gui/dialogs/MailVorschauDialog.java
@@ -157,7 +157,7 @@ public void handleAction(Object context)
       {
         close();
       }
-    });
+    }, null, false, "process-stop.png");
     b.paint(parent);
   }
-Original file line number
+Diff line change
@@ Expand Up / @@ -120,7 +120,7 @@ public void handleAction(Object context) @@
             }
             close();
           }
-        });
+        }, null, true, "ok.png");
         buttons.paint(parent);
       }
@@ Expand Down @@