Changeset 6509 in josm for trunk/src/org/openstreetmap/josm/actions

Timestamp:

2013-12-22T18:18:43+01:00 (12 years ago)

Author:

Don-vip

Message:

fix #9459 - initialize default center view of MapView to last download location, if any, instead of (0,0)

Location:

trunk/src/org/openstreetmap/josm/actions

Files:

• AutoScaleAction.java (modified) (2 diffs)
• DownloadAction.java (modified) (1 diff)
• NewAction.java (modified) (1 diff)

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

@@ -31 +31 @@
 import org.openstreetmap.josm.gui.dialogs.LayerListDialog;
 import org.openstreetmap.josm.gui.dialogs.ValidatorDialog.ValidatorBoundingXYVisitor;
+import org.openstreetmap.josm.gui.download.DownloadDialog;
 import org.openstreetmap.josm.gui.layer.Layer;
 import org.openstreetmap.josm.tools.Shortcut;
@@ -244 +245 @@
         }
         else if (mode.equals("download")) {
-            if (!Main.pref.get("osm-download.bounds").isEmpty()) {
+            Bounds bounds = DownloadDialog.getSavedDownloadBounds();
+            if (bounds != null) {
                 try {
-                    v.visit(new Bounds(Main.pref.get("osm-download.bounds"), ";"));
+                    v.visit(bounds);
                 } catch (Exception e) {
-                    e.printStackTrace();
+                    Main.warn(e);
                 }
             }

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

@@ -26 +26 @@
  */
 public class DownloadAction extends JosmAction {
+    
+    /**
+     * Constructs a new {@code DownloadAction}.
+     */
     public DownloadAction() {
         super(tr("Download from OSM..."), "download", tr("Download map data from the OSM server."),

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

@@ -13 +13 @@
 import org.openstreetmap.josm.tools.Shortcut;
 
+/**
+ * Creates a blank new OSM data layer.
+ * @since 169
+ */
 public class NewAction extends JosmAction {
 
+    /**
+     * Constructs a {@code NewAction}.
+     */
     public NewAction() {
         super(tr("New Layer"), "new", tr("Create a new map layer."),