From fde55a42809c04616ee2dda0e1c7b3948da8ed48 Mon Sep 17 00:00:00 2001 From: "Christian F." Date: Wed, 21 Aug 2024 20:47:44 +0200 Subject: [PATCH] - do not set text fields to white in DialogEditAbo. Use UIManager default colors. - minor cleanup --- CHANGELOG.md | 1 + src/main/java/mediathek/daten/ListeAbo.java | 1 - .../mediathek/gui/dialog/DialogEditAbo.form | 1 + .../mediathek/gui/dialog/DialogEditAbo.java | 18 ++++++++++-------- 4 files changed, 12 insertions(+), 9 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 4d5c79c2f0..28490baa81 100644 --- 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 index 32436e53bb..48cc15671c 100644 --- 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 index 52ed13fc8a..b40d3606c3 100644 --- a/src/main/java/mediathek/gui/dialog/DialogEditAbo.form +++ b/src/main/java/mediathek/gui/dialog/DialogEditAbo.form @@ -3,6 +3,7 @@
+ diff --git a/src/main/java/mediathek/gui/dialog/DialogEditAbo.java b/src/main/java/mediathek/gui/dialog/DialogEditAbo.java index 2ab2fcf970..1e496198f3 100644 --- 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 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)