Changeset 13777 in josm for trunk/src

Timestamp:

2018-05-16T21:30:47+02:00 (7 years ago)

Author:

wiktorn

Message:

Use setRememberWindowGeometry instead overriding setVisible

See: #16249

File:

• trunk/src/org/openstreetmap/josm/gui/preferences/imagery/AddImageryDialog.java (modified) (3 diffs)

trunk/src/org/openstreetmap/josm/gui/preferences/imagery/AddImageryDialog.java

@@ -18 +18 @@
 public class AddImageryDialog extends ExtendedDialog implements ContentValidationListener {
 
-    private final String saveGeometryEntryName;
-
     /**
      * Constructs a new AddImageryDialog.
@@ -33 +31 @@
         setMinimumSize(new Dimension(300, 400));
         panel.addContentValidationListener(this);
-        this.saveGeometryEntryName = panel.getClass().getName() + ".geometry";
+        setRememberWindowGeometry(
+                panel.getClass().getName() + ".geometry",
+                WindowGeometry.centerInWindow(Main.parent, new Dimension(400, 600))
+                );
     }
 
@@ -46 +47 @@
         buttons.get(0).setEnabled(isValid);
     }
-
-    @Override
-    public void setVisible(boolean visible) {
-        if (visible) {
-            new WindowGeometry(saveGeometryEntryName, WindowGeometry.centerInWindow(Main.parent,
-                    new Dimension(800, 600))).applySafe(this);
-        } else if (isShowing()) { // Avoid IllegalComponentStateException like in #8775
-            new WindowGeometry(this).remember(saveGeometryEntryName);
-        }
-        super.setVisible(visible);
-    }
 }