Changeset 9754 in josm for trunk/src/org/openstreetmap/josm/gui/io

Timestamp:

2016-02-07T19:03:59+01:00 (8 years ago)

Author:

bastiK

Message:

fixed: white cells in unsaved changed dialog (closes #12462)

Location:

trunk/src/org/openstreetmap/josm/gui/io

Files:

• LayerNameAndFilePathTableCell.java (modified) (1 diff)
• SaveLayersTableColumnModel.java (modified) (1 diff)

trunk/src/org/openstreetmap/josm/gui/io/LayerNameAndFilePathTableCell.java

@@ -30 +30 @@
 import org.openstreetmap.josm.tools.GBC;
 
+/**
+ * Display and edit layer name and file path in a <code>JTable</code>.
+ * 
+ * Note: Do not use the same object both as <code>TableCellRenderer</code> and
+ * <code>TableCellEditor</code> - this can mess up the current editor component
+ * by subsequent calls to the renderer (#12462).
+ */
 class LayerNameAndFilePathTableCell extends JPanel implements TableCellRenderer, TableCellEditor {
     private static final Color colorError = new Color(255, 197, 197);

trunk/src/org/openstreetmap/josm/gui/io/SaveLayersTableColumnModel.java

@@ -86 +86 @@
 
         // column 0 - layer name, save path editor
-        LayerNameAndFilePathTableCell lnafptc = new LayerNameAndFilePathTableCell();
+        LayerNameAndFilePathTableCell lnfpRenderer = new LayerNameAndFilePathTableCell();
+        LayerNameAndFilePathTableCell lnfpEditor = new LayerNameAndFilePathTableCell();
         col = new TableColumn(0); // keep in sync with SaveLayersModel#columnFilename
         col.setHeaderValue(tr("Layer Name and File Path"));
         col.setResizable(true);
-        col.setCellRenderer(lnafptc);
-        col.setCellEditor(lnafptc);
+        col.setCellRenderer(lnfpRenderer);
+        col.setCellEditor(lnfpEditor);
         col.setPreferredWidth(324);
         addColumn(col);