- do not set text fields to white in DialogEditAbo. Use UIManager def…

…ault colors. - minor cleanup
mediathekview · Aug 21, 2024 · fde55a4 · fde55a4
1 parent 0ce2f36
commit fde55a4
Show file tree

Hide file tree

Showing 4 changed files with 12 additions and 9 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -2,6 +2,7 @@
 - **BUGFIX:** MV aktualisiert nun vor dem Download mittels `-dq` oder `--download-quit` Parameter die Filmliste.
 - **BUGFIX:** FlatLaf Regression gelöst, die manchmal zu einer inkorrekten Darstellung der Oberfläche führte.
 - **BUGFIX(Linux/macOS):** Im "Film Speichern"-Dialog wurde der Zielpfad im Dark Mode weiß dargestellt.
+- **BUGFIX:** Im "Neues Abo anlegen"-Dialog werden Textfelder im Dark Mode nicht mehr weiß hinterlegt.
 - **FEATURE(Linux/Windows):** Das Aussehen von FlatLaf kann in `.mediathek3/flatlaf` angepasst werden.
 - **FEATURE:** Selektierte Einträge in `Suchhistorie bearbeiten`-Dialog können nun mittels Entfernen-Taste gelöscht werden.
 

diff --git a/src/main/java/mediathek/daten/ListeAbo.java b/src/main/java/mediathek/daten/ListeAbo.java
@@ -67,7 +67,6 @@ public void addAbo(String aboname, String filmSender, String filmThema, String f
         datenAbo.setPsetName("");
 
         DialogEditAbo dialogEditAbo = new DialogEditAbo(MediathekGui.ui(), datenAbo, false);
-        dialogEditAbo.setTitle("Neues Abo anlegen");
         dialogEditAbo.setVisible(true);
         if (dialogEditAbo.successful()) {
             if (!aboExistiertBereits(datenAbo)) {

diff --git a/src/main/java/mediathek/gui/dialog/DialogEditAbo.form b/src/main/java/mediathek/gui/dialog/DialogEditAbo.form
@@ -3,6 +3,7 @@
 <Form version="1.3" maxVersion="1.3" type="org.netbeans.modules.form.forminfo.JDialogFormInfo">
   <Properties>
     <Property name="defaultCloseOperation" type="int" value="2"/>
+    <Property name="title" type="java.lang.String" value="Neues Abo anlegen"/>
   </Properties>
   <SyntheticProperties>
     <SyntheticProperty name="formSizePolicy" type="int" value="1"/>

diff --git a/src/main/java/mediathek/gui/dialog/DialogEditAbo.java b/src/main/java/mediathek/gui/dialog/DialogEditAbo.java
@@ -66,7 +66,7 @@ public DialogEditAbo(final JFrame parent, DatenAbo aktA, boolean isMultiEditMode
         // Zielpfad ========================
         ArrayList<String> pfade = daten.getListeAbo().getPfade();
         if (!pfade.contains(aktAbo.getZielpfad())) {
-            pfade.add(0, aktAbo.getZielpfad());
+            pfade.addFirst(aktAbo.getZielpfad());
         }
         comboboxPfad.setModel(new DefaultComboBoxModel<>(pfade.toArray(new String[0])));
         comboboxPfad.setEditable(true);
@@ -119,10 +119,11 @@ public void setVisible(boolean vis) {
 
     private void checkPfad() {
         String s = ((JTextComponent) comboboxPfad.getEditor().getEditorComponent()).getText();
-        if (!s.equals(FilenameUtils.checkDateiname(s, false /*pfad*/))) {
-            comboboxPfad.getEditor().getEditorComponent().setBackground(MVColor.DOWNLOAD_FEHLER.color);
+        final var editor = comboboxPfad.getEditor().getEditorComponent();
+        if (!s.equals(FilenameUtils.checkDateiname(s, false))) {
+            editor.setBackground(MVColor.DOWNLOAD_FEHLER.color);
         } else {
-            comboboxPfad.getEditor().getEditorComponent().setBackground(Color.WHITE);
+            editor.setBackground(UIManager.getColor("TextField.background"));
         }
     }
 
@@ -371,21 +372,21 @@ public EmptyTextDocListener(JTextField tf) {
         @Override
         public void insertUpdate(DocumentEvent e) {
             final boolean isEmpty = tf.getText().isBlank();
-            tf.setBackground(isEmpty ? Color.red : Color.white);
+            tf.setBackground(isEmpty ? Color.red : UIManager.getColor("TextField.background"));
             jButtonBeenden.setEnabled(!isEmpty);
         }
 
         @Override
         public void removeUpdate(DocumentEvent e) {
             final boolean isEmpty = tf.getText().isBlank();
-            tf.setBackground(isEmpty ? Color.red : Color.white);
+            tf.setBackground(isEmpty ? Color.red : UIManager.getColor("TextField.background"));
             jButtonBeenden.setEnabled(!isEmpty);
         }
 
         @Override
         public void changedUpdate(DocumentEvent e) {
             final boolean isEmpty = tf.getText().isBlank();
-            tf.setBackground(isEmpty ? Color.red : Color.white);
+            tf.setBackground(isEmpty ? Color.red : UIManager.getColor("TextField.background"));
             jButtonBeenden.setEnabled(!isEmpty);
         }
     }
@@ -400,6 +401,7 @@ private void initComponents() {
         jButtonHelp = new javax.swing.JButton();
 
         setDefaultCloseOperation(javax.swing.WindowConstants.DISPOSE_ON_CLOSE);
+        setTitle("Neues Abo anlegen");
 
         javax.swing.GroupLayout jPanelExtraLayout = new javax.swing.GroupLayout(jPanelExtra);
         jPanelExtra.setLayout(jPanelExtraLayout);
@@ -439,7 +441,7 @@ private void initComponents() {
                 .addContainerGap())
         );
 
-        layout.linkSize(javax.swing.SwingConstants.HORIZONTAL, new java.awt.Component[] {jButtonAbbrechen, jButtonBeenden});
+        layout.linkSize(javax.swing.SwingConstants.HORIZONTAL, jButtonAbbrechen, jButtonBeenden);
 
         layout.setVerticalGroup(
             layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)