Disable the export button until a format is selected
authorTom Hughes <tom@compton.nu>
Thu, 15 Mar 2012 14:50:43 +0000 (14:50 +0000)
committerTom Hughes <tom@compton.nu>
Thu, 15 Mar 2012 14:50:43 +0000 (14:50 +0000)
app/views/export/start.js.erb

index 1134d16d83ce3790a555d766fd3ccd7268e5134c..e433a498fdbb532ccd1fa86ed69257545a6894f2 100644 (file)
@@ -214,15 +214,15 @@ function validateControls() {
   }
 
   var max_scale = maxMapnikScale();
+  var disabled = true;
 
-  if ($("#format_osm").prop("checked") && bounds.getWidth() * bounds.getHeight() > <%= MAX_REQUEST_AREA %>) {
-    $("#export_commit").prop("disabled", true);
-  } else if ($("#format_mapnik").prop("checked") && $("#mapnik_scale").val() < max_scale) {
-    $("#export_commit").prop("disabled", true);
-  } else {
-    $("#export_commit").prop("disabled", false);
+  if ($("#format_osm").prop("checked")) {
+    disabled = bounds.getWidth() * bounds.getHeight() > <%= MAX_REQUEST_AREA %>;
+  } else if ($("#format_mapnik").prop("checked")) {
+    disabled = $("#mapnik_scale").val() < max_scale;
   }
 
+  $("#export_commit").prop("disabled", disabled);
   $("#mapnik_max_scale").html(roundScale(max_scale));
 }