Changeset 2132 in josm for trunk/src/org

Timestamp:

2009-09-14T21:00:54+02:00 (15 years ago)

Author:

Gubaer

Message:

fixed #3478: Remember size of download dialog

File:

• trunk/src/org/openstreetmap/josm/actions/DownloadAction.java (modified) (2 diffs)

trunk/src/org/openstreetmap/josm/actions/DownloadAction.java

@@ -4 +4 @@
 import static org.openstreetmap.josm.tools.I18n.tr;
 
-import java.awt.GridBagLayout;
+import java.awt.BorderLayout;
+import java.awt.Dimension;
 import java.awt.event.ActionEvent;
 import java.awt.event.KeyEvent;
 
-import javax.swing.JDialog;
 import javax.swing.JOptionPane;
 import javax.swing.JPanel;
 
 import org.openstreetmap.josm.Main;
+import org.openstreetmap.josm.gui.ExtendedDialog;
 import org.openstreetmap.josm.gui.download.DownloadDialog;
 import org.openstreetmap.josm.gui.download.DownloadDialog.DownloadTask;
-import org.openstreetmap.josm.tools.GBC;
 import org.openstreetmap.josm.tools.Shortcut;
+import org.openstreetmap.josm.tools.WindowGeometry;
 
 /**
@@ -35 +36 @@
     }
 
+    /**
+     * Creates the download dialog
+     * @return the downlaod dialog
+     */
+    protected ExtendedDialog createUploadDialog() {
+        dialog = new DownloadDialog();
+        JPanel downPanel = new JPanel(new BorderLayout());
+        downPanel.add(dialog, BorderLayout.CENTER);
+
+        final String prefName = dialog.getClass().getName()+ ".geometry";
+        ExtendedDialog dialog = new ExtendedDialog(Main.parent, tr("Download"), new String[] {tr("OK"), tr("Cancel")}) {
+            @Override
+            public void setVisible(boolean visible) {
+                if (visible) {
+                    new WindowGeometry(
+                            prefName,
+                            WindowGeometry.centerInWindow(Main.parent, new Dimension(1000,600))
+                    ).apply(this);
+                } else {
+                    new WindowGeometry(this).remember(prefName);
+                }
+                super.setVisible(visible);
+            }
+        };
+        dialog.setContent(downPanel, false /* don't use a scroll pane inside the dialog */);
+        dialog.setButtonIcons(new String[] {"ok", "cancel"});
+        return dialog;
+    }
+
     public void actionPerformed(ActionEvent e) {
-        dialog = new DownloadDialog();
-
-        JPanel downPanel = new JPanel(new GridBagLayout());
-        downPanel.add(dialog, GBC.eol().fill(GBC.BOTH));
-
-        JOptionPane pane = new JOptionPane(downPanel, JOptionPane.PLAIN_MESSAGE, JOptionPane.OK_CANCEL_OPTION);
-        JDialog dlg = pane.createDialog(Main.parent, tr("Download"));
-        dlg.setResizable(true);
-        dialog.setOptionPane(pane);
-
-        if (dlg.getWidth() > 1000) {
-            dlg.setSize(1000, dlg.getHeight());
-        }
-        if (dlg.getHeight() > 600) {
-            dlg.setSize(dlg.getWidth(),600);
-        }
-
+        ExtendedDialog dlg = createUploadDialog();
         boolean finish = false;
         while (!finish) {
-            dlg.setVisible(true);
+            dlg.showDialog();
             Main.pref.put("download.newlayer", dialog.newLayer.isSelected());
-            if (pane.getValue() instanceof Integer && (Integer)pane.getValue() == JOptionPane.OK_OPTION) {
+            if (dlg.getValue() == 1 /* OK */) {
                 Main.pref.put("download.tab", Integer.toString(dialog.getSelectedTab()));
                 for (DownloadTask task : dialog.downloadTasks) {