From 7727ce46183a6fb4adaaf0e5b63cd06a6034a2a8 Mon Sep 17 00:00:00 2001
From: Robert Kaiser <kairo@kairo.at>
Date: Wed, 29 Oct 2008 18:52:25 +0100
Subject: [PATCH 1/1] add scale field in image settings, add extra button for
 drawing image right away#

---
 .../mandelbrot/content/image-settings.js      | 15 +++++++++++
 .../mandelbrot/content/image-settings.xul     | 26 +++++++++++++------
 .../chrome/mandelbrot/content/mandelbrot.js   |  2 +-
 .../locales/en-US/image-settings.dtd          |  1 +
 4 files changed, 35 insertions(+), 9 deletions(-)

diff --git a/xulapp/chrome/mandelbrot/content/image-settings.js b/xulapp/chrome/mandelbrot/content/image-settings.js
index d9af993..a907f5a 100644
--- a/xulapp/chrome/mandelbrot/content/image-settings.js
+++ b/xulapp/chrome/mandelbrot/content/image-settings.js
@@ -38,6 +38,8 @@
 var gColorPalette = [];
 
 function initSettings() {
+  recalcCoord('Cr', 'scale');
+  recalcCoord('Ci', 'scale');
 }
 
 function drawPreview() {
@@ -78,3 +80,16 @@ function drawPreview() {
   drawLine(0, [Cr_min, Cr_max, Ci_min, Ci_max],
               canvas, context, iterMax, algorithm);
 }
+
+function recalcCoord(coord, target) {
+  if (target == 'scale') {
+    document.getElementById(coord + '_scale').value =
+      parseFloat(document.getElementById(coord + '_max').value) -
+      parseFloat(document.getElementById(coord + '_min').value);
+  }
+  else if (target == 'max') {
+    document.getElementById('mandelbrot.last_image.' + coord + '_max').value =
+      parseFloat(document.getElementById(coord + '_min').value) +
+      parseFloat(document.getElementById(coord + '_scale').value);
+  }
+}
\ No newline at end of file
diff --git a/xulapp/chrome/mandelbrot/content/image-settings.xul b/xulapp/chrome/mandelbrot/content/image-settings.xul
index 04da656..c4c6092 100644
--- a/xulapp/chrome/mandelbrot/content/image-settings.xul
+++ b/xulapp/chrome/mandelbrot/content/image-settings.xul
@@ -46,9 +46,9 @@
             xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul"
             xmlns:html="http://www.w3.org/1999/xhtml"
             title="&imageSettings.title;"
-            dlgbuttons="accept,cancel"
-            ondialogaccept="callDrawImage();"
-            buttonlabelaccept="&DrawImageButton.label;">
+            buttons="accept,cancel,extra1"
+            ondialogextra1="callDrawImage();"
+            buttonlabelextra1="&DrawImageButton.label;">
 
   <prefpane id="imgSettingsPane" onpaneload="initSettings();">
     <script type="application/x-javascript"
@@ -77,19 +77,29 @@
       <hbox flex="1">
         <label value="&coord.min.label;" control="Cr_min"/>
         <textbox id="Cr_min" size="10"
-                 preference="mandelbrot.last_image.Cr_min"/>
+                 preference="mandelbrot.last_image.Cr_min"
+                 onchange="recalcCoord('Cr', 'scale');"/>
         <label value="&coord.max.label;" control="Cr_max"/>
         <textbox id="Cr_max" size="10"
-                 preference="mandelbrot.last_image.Cr_max"/>
+                 preference="mandelbrot.last_image.Cr_max"
+                 onchange="recalcCoord('Cr', 'scale');"/>
+        <label value="&coord.scale.label;" control="Cr_scale"/>
+        <textbox id="Cr_scale" size="10"
+                 onchange="recalcCoord('Cr', 'max');"/>
       </hbox>
       <description value="&coord.imag.label;"/>
       <hbox flex="1">
         <label value="&coord.min.label;" control="Ci_min"/>
         <textbox id="Ci_min" size="10"
-                 preference="mandelbrot.last_image.Ci_min"/>
+                 preference="mandelbrot.last_image.Ci_min"
+                 onchange="recalcCoord('Ci', 'scale');"/>
         <label value="&coord.max.label;" control="Ci_max"/>
         <textbox id="Ci_max" size="10"
-                 preference="mandelbrot.last_image.Ci_max"/>
+                 preference="mandelbrot.last_image.Ci_max"
+                 onchange="recalcCoord('Ci', 'scale');"/>
+        <label value="&coord.scale.label;" control="Ci_scale"/>
+        <textbox id="Ci_scale" size="10"
+                 onchange="recalcCoord('Ci', 'max');"/>
       </hbox>
     </groupbox>
 
@@ -107,7 +117,7 @@
                    preference="mandelbrot.image.height"/>
         </hbox>
       </groupbox>
-    
+
       <groupbox>
         <caption label="&preview.title;"/>
         <hbox flex="1" pack="center" align="center">
diff --git a/xulapp/chrome/mandelbrot/content/mandelbrot.js b/xulapp/chrome/mandelbrot/content/mandelbrot.js
index 0265497..cf4927a 100644
--- a/xulapp/chrome/mandelbrot/content/mandelbrot.js
+++ b/xulapp/chrome/mandelbrot/content/mandelbrot.js
@@ -137,7 +137,7 @@ function drawLine(line, dimensions, canvas, context, iterMax, algorithm) {
 
     if (img_y < canvas.height)
       setTimeout(drawLine, 0, img_y, dimensions, canvas, context, iterMax, algorithm);
-    else
+    else if (gStartTime)
       EndCalc();
 }
 
diff --git a/xulapp/chrome/mandelbrot/locales/en-US/image-settings.dtd b/xulapp/chrome/mandelbrot/locales/en-US/image-settings.dtd
index ea6c02b..1351acc 100644
--- a/xulapp/chrome/mandelbrot/locales/en-US/image-settings.dtd
+++ b/xulapp/chrome/mandelbrot/locales/en-US/image-settings.dtd
@@ -43,6 +43,7 @@
 <!ENTITY coord.imag.label      "Imaginary">
 <!ENTITY coord.min.label       "From">
 <!ENTITY coord.max.label       "To">
+<!ENTITY coord.scale.label     "Scale">
 
 <!ENTITY img.size.title        "Image Size">
 <!ENTITY img.width.label       "Width">
-- 
2.35.3