[bt-devel] AutoTIle/AutoCascade Bug

Thomas J. Philpot bt-devel@crosswire.org
Sat, 22 Apr 2000 12:47:46 -0500


--Boundary_(ID_q9X8n3AgTcOMhWKTG1c9zA)
Content-type: text/plain; CHARSET=US-ASCII
Content-transfer-encoding: 7BIT

Joachim,

While playing with BT 0.3 I noticed that I made a small mistake when I
suggested a patch for AutoTIle and AutoCascade.  BT issued a tile() or
cascade() command when you unchecked AutoTIle and AutoCascade.  The patch below
fixes this.  Now it only tiles/cascades automatically when AutoTile and
AutoCascade are checked.  Apply it to the CMDIArea.cpp file.


Tom

--Boundary_(ID_q9X8n3AgTcOMhWKTG1c9zA)
Content-type: text/plain; CHARSET=US-ASCII; name=cmdipatch.txt
Content-disposition: attachment; filename=cmdipatch.txt
Content-transfer-encoding: 7BIT

--- cmdiarea.cpp.bak	Tue Apr 11 15:57:30 2000
+++ cmdiarea.cpp	Sat Apr 22 12:33:20 2000
@@ -165,6 +165,7 @@
 	if (autoTileChecked) {
 		fixedGUIOption = Tile;	
 		windowMenu->setItemEnabled(ID_WINDOW_TILE, false);		
+		tile();
 	}
 	else {
 		fixedGUIOption = Nothing;	
@@ -172,7 +173,6 @@
 		windowMenu->setItemChecked(ID_WINDOW_AUTO_CASCADE, true);
 	};
 	
-	tile();
 }
 
 /**  */
@@ -184,13 +184,13 @@
 	if (autoCascadeChecked) {
 		fixedGUIOption = Cascade;	
 		windowMenu->setItemEnabled(ID_WINDOW_CASCADE, false);
+		cascade();
 	}
 	else {
 		fixedGUIOption = Nothing;	
 		windowMenu->setItemEnabled(ID_WINDOW_CASCADE, true);
 	};
 	
-	cascade();
 }
 
 /** Reimplementation */

--Boundary_(ID_q9X8n3AgTcOMhWKTG1c9zA)--