Ticket #19366: 19366.bug_fixes.patch

src/org/openstreetmap/josm/gui/preferences/display/ColorPreference.java

@@ -264 +264 @@
         colorEdit = new JButton(tr("Choose"));
         colorEdit.addActionListener(e -> {
             int sel = colors.getSelectedRow();
+            if (sel < 0) return;
             sel = colors.convertRowIndexToModel(sel);
             ColorEntry ce = tableModel.getEntry(sel);
             JColorChooser chooser = new JColorChooser(ce.getDisplayColor());
@@ -273 +274 @@
                     JOptionPane.OK_CANCEL_OPTION,
                     JOptionPane.PLAIN_MESSAGE);
             if (answer == JOptionPane.OK_OPTION) {
-                colors.setValueAt(chooser.getColor(), sel, 1);
+                ce.info.setValue(chooser.getColor());
+                tableModel.fireTableRowsUpdated(sel, sel);
             }
         });
         defaultSet = new JButton(tr("Reset to default"));
@@ -283 +285 @@
             ColorEntry ce = tableModel.getEntry(sel);
             Color c = ce.info.getDefaultValue();
             if (c != null) {
-                colors.setValueAt(c, sel, 1);
+                ce.info.setValue(c);
+                tableModel.fireTableRowsUpdated(sel, sel);
             }
         });
         JButton defaultAll = new JButton(tr("Set all to default"));
@@ -293 +296 @@
                 ColorEntry ce = data.get(i);
                 Color c = ce.info.getDefaultValue();
                 if (c != null) {
-                    colors.setValueAt(c, i, 1);
+                    ce.info.setValue(c);
+                    tableModel.fireTableDataChanged();
                 }
             }
         });