X-Git-Url: https://git.openstreetmap.org/rails.git/blobdiff_plain/7fa1041340e3f1ad1074bfd8376b39535b862a69..7a9e92386e7511061d2c2254919db167ad28bfa3:/app/views/export/start.rjs

diff --git a/app/views/export/start.rjs b/app/views/export/start.rjs
index 5e4a2b91c..c0fced025 100644
--- a/app/views/export/start.rjs
+++ b/app/views/export/start.rjs
@@ -12,23 +12,20 @@ page << <<EOJ
 
     box = new OpenLayers.Control.DrawFeature(vectors, OpenLayers.Handler.RegularPolygon, { 
       handlerOptions: {
-        keyMask: OpenLayers.Handler.MOD_CTRL,
         sides: 4,
         snapAngle: 90,
         irregular: true,
         persist: true,
-        callbacks: { done: boxComplete }
+        callbacks: { done: endDrag }
       }
     });
     map.addControl(box);
 
-    box.activate();
-
     map.events.register("moveend", map, mapMoved);
 
     openSidebar({ onclose: stopExport });
 
-    updateRegion(map.getExtent());
+    setBounds(map.getExtent());
 
     if (map.baseLayer.name == "Mapnik") {
       $("format_mapnik").checked = true;
@@ -51,36 +48,6 @@ page << <<EOJ
     map.removeLayer(vectors);
   }
 
-  function formatChanged() {
-    if ($("format_osm").checked) {
-      $("export_osm").style.display = "inline";
-    } else {
-      $("export_osm").style.display = "none";
-    }
-
-    if ($("format_mapnik").checked) {
-      $("mapnik_scale").value = roundScale(map.getScale());
-      $("export_mapnik").style.display = "inline";
-
-      mapnikScaleChanged();
-    } else {
-      $("export_mapnik").style.display = "none";
-    }
-
-    if ($("format_osmarender").checked) {
-      var zoom = Math.min(map.getZoom(), maxOsmarenderZoom());
-
-      $("osmarender_zoom").options.selectedIndex = zoom - 4;
-      $("export_osmarender").style.display = "inline";
-    } else {
-      $("export_osmarender").style.display = "none";
-    }
-  }
-
-  $("format_osm").onclick = formatChanged;
-  $("format_mapnik").onclick = formatChanged;
-  $("format_osmarender").onclick = formatChanged;
-
   function boundsChanged() {
     var epsg4326 = new OpenLayers.Projection("EPSG:4326");
     var bounds = new OpenLayers.Bounds($("minlon").value,
@@ -93,12 +60,10 @@ page << <<EOJ
     map.events.unregister("moveend", map, mapMoved);
     map.zoomToExtent(bounds);
 
-    box.handler.clear();
-    box.handler.feature = new OpenLayers.Feature.Vector(bounds.toGeometry());
-    box.handler.layer.addFeatures([box.handler.feature], [box.handler.style]);
-    box.handler.layer.drawFeature(box.handler.feature, box.handler.style);
+    clearBox();
+    drawBox(bounds);
 
-    mapnikSizeChanged();
+    validateControls();
   }
 
   $("maxlat").onchange = boundsChanged;
@@ -106,31 +71,61 @@ page << <<EOJ
   $("maxlon").onchange = boundsChanged;
   $("minlat").onchange = boundsChanged;
 
-  function mapMoved() {
-    updateRegion(map.getExtent());
-  }
+  function startDrag() {
+    $("drag_box").innerHTML='Drag a box on the map to select an area';
+
+    clearBox();
+    box.activate();
+  };
+
+  $("drag_box").onclick = startDrag;
+
+  function endDrag(bbox) {
+    var bounds = bbox.getBounds();
 
-  function boxComplete(box) {
     map.events.unregister("moveend", map, mapMoved);
-    updateRegion(box.getBounds());
+    setBounds(bounds);
+    drawBox(bounds);
+    box.deactivate();
+    validateControls();
+
+    $("drag_box").innerHTML = "Manually select a different area";
   }
 
-  function updateRegion(bounds) {
+  function mapMoved() {
+    setBounds(map.getExtent());
+    validateControls();
+  }
+
+  function setBounds(bounds) {
     var epsg4326 = new OpenLayers.Projection("EPSG:4326");
     var decimals = Math.pow(10, Math.floor(map.getZoom() / 3));
 
-    bounds.transform(map.getProjectionObject(), epsg4326);
+    bounds = bounds.clone().transform(map.getProjectionObject(), epsg4326);
 
     $("minlon").value = Math.round(bounds.left * decimals) / decimals;
     $("minlat").value = Math.round(bounds.bottom * decimals) / decimals;
     $("maxlon").value = Math.round(bounds.right * decimals) / decimals;
     $("maxlat").value = Math.round(bounds.top * decimals) / decimals;
+  }
+
+  function clearBox() {
+    vectors.destroyFeatures();
+  }
+
+  function drawBox(bounds) {
+    var feature = new OpenLayers.Feature.Vector(bounds.toGeometry());
+
+    vectors.addFeatures(feature);
+  }
+
+  function validateControls() {
+    var bounds = new OpenLayers.Bounds($("minlon").value, $("minlat").value, $("maxlon").value, $("maxlat").value);
 
     if (bounds.getWidth() * bounds.getHeight() > 0.25) {
       $("format_osm").disabled = true;
       $("format_osm").checked = false;
-
-      formatChanged();
+      $("export_osm").style.display = "none";
     } else {
       $("format_osm").disabled = false;
     }
@@ -140,7 +135,7 @@ page << <<EOJ
 
     $("mapnik_max_scale").innerHTML = roundScale(max_scale);
 
-    mapnikSizeChanged();
+    mapnikScaleChanged();
 
     for (var o = 0; o < $("osmarender_zoom").options.length; o++) {
       var option = $("osmarender_zoom").options[o];
@@ -157,6 +152,36 @@ page << <<EOJ
     }
   }
 
+  function formatChanged() {
+    if ($("format_osm").checked) {
+      $("export_osm").style.display = "inline";
+    } else {
+      $("export_osm").style.display = "none";
+    }
+
+    if ($("format_mapnik").checked) {
+      $("mapnik_scale").value = roundScale(map.getScale());
+      $("export_mapnik").style.display = "inline";
+
+      mapnikScaleChanged();
+    } else {
+      $("export_mapnik").style.display = "none";
+    }
+
+    if ($("format_osmarender").checked) {
+      var zoom = Math.min(map.getZoom(), maxOsmarenderZoom());
+
+      $("osmarender_zoom").options.selectedIndex = zoom - 4;
+      $("export_osmarender").style.display = "inline";
+    } else {
+      $("export_osmarender").style.display = "none";
+    }
+  }
+
+  $("format_osm").onclick = formatChanged;
+  $("format_mapnik").onclick = formatChanged;
+  $("format_osmarender").onclick = formatChanged;
+
   function maxMapnikScale() {
     var bounds = new OpenLayers.Bounds($("minlon").value, $("minlat").value, $("maxlon").value, $("maxlat").value);
     var epsg4326 = new OpenLayers.Projection("EPSG:4326");