From 9ef6528d8e382482200e0f3cd6d33d95ab57a892 Mon Sep 17 00:00:00 2001
From: Tom Hughes <tom@compton.nu>
Date: Sun, 20 Apr 2008 10:31:03 +0000
Subject: [PATCH] Make sure the export button is enabled if we switch away from
 mapnik export, and that the mapnik image size is updated correctly.

---
 app/views/export/start.rjs | 27 +++++++++++++++------------
 1 file changed, 15 insertions(+), 12 deletions(-)

diff --git a/app/views/export/start.rjs b/app/views/export/start.rjs
index 7a323b05b..3f1d73d68 100644
--- a/app/views/export/start.rjs
+++ b/app/views/export/start.rjs
@@ -64,6 +64,7 @@ page << <<EOJ
     drawBox(bounds);
 
     validateControls();
+    mapnikSizeChanged();
   }
 
   $("maxlat").onchange = boundsChanged;
@@ -107,6 +108,8 @@ page << <<EOJ
     $("minlat").value = Math.round(bounds.bottom * decimals) / decimals;
     $("maxlon").value = Math.round(bounds.right * decimals) / decimals;
     $("maxlat").value = Math.round(bounds.top * decimals) / decimals;
+
+    mapnikSizeChanged();
   }
 
   function clearBox() {
@@ -129,13 +132,18 @@ page << <<EOJ
     } else {
       $("format_osm").disabled = false;
     }
-    
-    var max_zoom = maxOsmarenderZoom();
+
     var max_scale = maxMapnikScale();
 
-    $("mapnik_max_scale").innerHTML = roundScale(max_scale);
+    if ($("format_mapnik").checked && $("mapnik_scale").value < max_scale) {
+      $("export_commit").disabled = true;
+    } else {
+      $("export_commit").disabled = false;
+    }
 
-    mapnikScaleChanged();
+    $("mapnik_max_scale").innerHTML = roundScale(max_scale);
+  
+    var max_zoom = maxOsmarenderZoom();
 
     for (var o = 0; o < $("osmarender_zoom").options.length; o++) {
       var option = $("osmarender_zoom").options[o];
@@ -162,8 +170,6 @@ page << <<EOJ
     if ($("format_mapnik").checked) {
       $("mapnik_scale").value = roundScale(map.getScale());
       $("export_mapnik").style.display = "inline";
-
-      mapnikScaleChanged();
     } else {
       $("export_mapnik").style.display = "none";
     }
@@ -176,6 +182,8 @@ page << <<EOJ
     } else {
       $("export_osmarender").style.display = "none";
     }
+
+    validateControls();
   }
 
   $("format_osm").onclick = formatChanged;
@@ -228,12 +236,7 @@ page << <<EOJ
 
   function mapnikScaleChanged() {
     mapnikSizeChanged();
-
-    if ($("mapnik_scale").value < maxMapnikScale()) {
-      $("export_commit").disabled = true;
-    } else {
-      $("export_commit").disabled = false;
-    }
+    validateControls();
   }
 
   $("mapnik_scale").onchange = mapnikScaleChanged;
-- 
2.39.5