From 0ae95d53385d45d3a4f266b1033cffa2b180302b Mon Sep 17 00:00:00 2001
From: BigEd <ed.spittles@gmail.com>
Date: Fri, 1 Oct 2010 13:02:03 +0000
Subject: [PATCH] [bug]rejig non-graphics mode

---
 index.html |  3 +++
 macros.js  |  3 ++-
 wires.js   | 41 ++++++++++++++++++++++++-----------------
 3 files changed, 29 insertions(+), 18 deletions(-)

diff --git a/index.html b/index.html
index 43f7398..1211292 100644
--- a/index.html
+++ b/index.html
@@ -51,6 +51,7 @@ Enter your own program into the array of RAM
 </span>
 <div class="frame" id="frame">
 <div class="chip" id="chip">
+<span id="waiting">Please wait while graphics is initialising...</span>
 <canvas class="chip" id="chipbg"></canvas>
 <canvas class="chip" id="overlay"></canvas>
 <canvas class="chip" id="hilite"></canvas>
@@ -88,8 +89,10 @@ Enter your own program into the array of RAM
   <input type="checkbox" name="2" id="updateShow2" onchange="updateShow(this.name,this.checked)" />(protection)
   </form>
   <form>
+  <input type="button" value="Hide Chip Layout" onclick="hideChipLayout()" />
   <input type="button" value="Clear Highlighting" onclick="clearHighlight()" />
   </form>
+  Animate during simulation: <input type="checkbox" id="animateModeCheckbox" onchange="updateChipLayoutAnimation(this.checked)" />
 </div>
 <div id="expertControlPanel" style="display:none">
   <form>
diff --git a/macros.js b/macros.js
index d4e6752..cda8ba2 100644
--- a/macros.js
+++ b/macros.js
@@ -132,7 +132,8 @@ function signalSet(n){
 function step(){
 	trace[cycle]= {chip: stateString(), mem: getMem()};
 	halfStep();
-	refresh();
+	if(animateChipLayout)
+		refresh();
 	cycle++;
 	chipStatus();
 }
diff --git a/wires.js b/wires.js
index 73e5b4f..9f8c470 100644
--- a/wires.js
+++ b/wires.js
@@ -41,7 +41,7 @@ var npwr = nodenames['vcc'];
 
 // some modes and parameters which can be passed in from the URL query
 var expertMode=false
-var drawGraphics = true;
+var animateChipLayout = true;
 var loadCanvas = true;
 
 /////////////////////////
@@ -99,7 +99,7 @@ function setupParams(){
 		} else if(name=="expert" && value.indexOf("t")==0){
 			updateExpertMode(true);
 		} else if(name=="graphics" && value.indexOf("f")==0){
-			setupNoGraphics();
+			hideChipLayout();
 		} else {
 			if(loglevel>0)
 				console.log('unrecognised parameters:',params);
@@ -108,15 +108,13 @@ function setupParams(){
 	}
 }
 
-function setupNoGraphics(){
-	// if user requests no graphics, we'll do no canvas operations at all
-	// which saves a lot of memory and allows us to run on small systems
-	loadCanvas=false;
-	// should the canvas later be loaded, we don't want to draw to it
-	// which speeds up simulation
-	drawGraphics=false;
-	// we'll also hide or shrink the graphics panel and replace it
-	hideChipLayout();
+function updateChipLayoutAnimation(isOn){
+	// simulation is much faster if we don't update the chip layout on every step
+	animateChipLayout=isOn;
+	if(animateChipLayout && !loadCanvas) {
+		loadCanvas=true;
+		showChipLayout();
+	}
 }
 
 function setupNodes(){
@@ -225,7 +223,7 @@ function hexdigit(n){return '0123456789ABCDEF'.charAt(n);}
 /////////////////////////
 
 function refresh(){
-	if(!drawGraphics)
+	if(!loadCanvas)
 		return;
 	ctx.clearRect(0,0,10000,10000);
 	for(i in nodes){
@@ -255,9 +253,7 @@ function hiliteNode(n){
 	}
 }
 
-
 function drawSeg(ctx, seg){
-	if(!drawGraphics) return;
 	var dx = 400;
 	ctx.beginPath();
 	ctx.moveTo(seg[0]+dx, 10000-seg[1])
@@ -364,7 +360,6 @@ function findNodeNumber(x,y){
 }
 
 function updateLoglevel(value){
-	console.log("updateLoglevel:",value,loglevel);
 	loglevel = value;
 	initLogbox(signalSet(loglevel));
 }
@@ -397,8 +392,12 @@ function showChipLayout(){
 	document.getElementById('chip').style.display = 'block';
 	document.getElementById('layoutControlPanel').style.display = 'block';
 	document.getElementById('nochip').style.display = 'none';
-	// experts see the control panel and can enable animation manually
-	drawGraphics=!expertMode;
+	// allow the display to update while we load the graphics
+	setStatus('loading graphics...');
+	setTimeout(showChipLayout_part2, 0);
+}
+
+function showChipLayout_part2(){
 	setupLayerVisibility();
 	setupBackground();
 	setupOverlay();
@@ -406,6 +405,8 @@ function showChipLayout(){
 	setupHitBuffer();
 	recenter();
 	refresh();
+	document.getElementById('waiting').style.display = 'none';
+	setStatus('Ready!');  // would prefer chipStatus but it's not idempotent
 	hilite.onmousedown = function(e){mouseDown(e);}
 }
 
@@ -414,8 +415,14 @@ function hideChipLayout(){
 	document.getElementById('chip').style.display = 'none';
 	document.getElementById('layoutControlPanel').style.display = 'none';
 	document.getElementById('nochip').style.display = 'block';
+	// if user requests no chip layout, we can do no canvas operations at all
+	// which saves a lot of memory and allows us to run on small systems
+	loadCanvas=false;
+	// cannot animate the layout if there is no canvas
+	animateChipLayout=false;
 }
 
+
 /////////////////////////
 //
 // Etc.