You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@aries.apache.org by zo...@apache.org on 2010/07/03 15:24:05 UTC

svn commit: r960219 - in /incubator/aries/trunk/samples/goat/goat-web/src/main/resources/web/goat: ElementLayoutManager.js elements/TwistieSection.js

Author: zoe
Date: Sat Jul  3 13:24:05 2010
New Revision: 960219

URL: http://svn.apache.org/viewvc?rev=960219&view=rev
Log:
ARIES-319 comments out logs

Modified:
    incubator/aries/trunk/samples/goat/goat-web/src/main/resources/web/goat/ElementLayoutManager.js
    incubator/aries/trunk/samples/goat/goat-web/src/main/resources/web/goat/elements/TwistieSection.js

Modified: incubator/aries/trunk/samples/goat/goat-web/src/main/resources/web/goat/ElementLayoutManager.js
URL: http://svn.apache.org/viewvc/incubator/aries/trunk/samples/goat/goat-web/src/main/resources/web/goat/ElementLayoutManager.js?rev=960219&r1=960218&r2=960219&view=diff
==============================================================================
--- incubator/aries/trunk/samples/goat/goat-web/src/main/resources/web/goat/ElementLayoutManager.js (original)
+++ incubator/aries/trunk/samples/goat/goat-web/src/main/resources/web/goat/ElementLayoutManager.js Sat Jul  3 13:24:05 2010
@@ -78,7 +78,7 @@ doLayout: function(){	
 			element.x=0;
 			element.y=currentYPos;
 			
-			console.log("Invoking element.render" + this.owningComponent.id + " " + elementName);
+			//console.log("Invoking element.render" + this.owningComponent.id + " " + elementName);
 			
 			element.render();
 			

Modified: incubator/aries/trunk/samples/goat/goat-web/src/main/resources/web/goat/elements/TwistieSection.js
URL: http://svn.apache.org/viewvc/incubator/aries/trunk/samples/goat/goat-web/src/main/resources/web/goat/elements/TwistieSection.js?rev=960219&r1=960218&r2=960219&view=diff
==============================================================================
--- incubator/aries/trunk/samples/goat/goat-web/src/main/resources/web/goat/elements/TwistieSection.js (original)
+++ incubator/aries/trunk/samples/goat/goat-web/src/main/resources/web/goat/elements/TwistieSection.js Sat Jul  3 13:24:05 2010
@@ -60,8 +60,6 @@ dojo.declare("goat.elements.TwistieSecti
 	
 			this.twistieHandleGroup.connect("onclick",dojo.hitch(this,"twistieHandler"));
 			
-			console.log("ZZX: New TA instance, type: " + type + " component: " + this.component.id); 
-
 			this.addsub=dojo.subscribe("goat.relationshipaggregator.add."+this.component.id+"."+this.type, this, this.onAdd);
 			this.removesub=dojo.subscribe("goat.relationshipaggregator.remove."+this.component.id+"."+this.type, this, this.onRemove);		
 		},
@@ -75,7 +73,6 @@ dojo.declare("goat.elements.TwistieSecti
 			return this.height;
 		},
 		render: function(){
-			console.log("RENDERING...");
 			this.component.group.remove(this.twistieGroup);
 			if(this.built) {
 				this.twistieHandleGroup.remove(this.twistieHandle);
@@ -86,12 +83,13 @@ dojo.declare("goat.elements.TwistieSecti
 			this.createTwistie();
 			this.createText();
 
+			// This may not be the secionn that has asked for a component refresh. If is is not and it is open
+			// we need to redraw as the section may have moved as a result of something else above it 
+			// being closed. If this IS the the section that requested the resize we shoudl ne need to re-add, but do anyway. 
 			if(this.isOpen) {
-				this.removeItemsFromDisplay();
 				this.addItemsToDisplay();
 			}
 			this.component.group.add(this.twistieGroup);
-			console.log("Rendered, x = " + this.x + "y = " + this.y);
 		},
 		update: function(value){
 		},
@@ -183,7 +181,7 @@ dojo.declare("goat.elements.TwistieSecti
     		}else{
         		this.removeItemsFromDisplay();
     		}
-    		console.log("twistie requesting refresh of component due to change in state/content");
+
     		this.component.refresh();
 		},
 		createText: function(){
@@ -195,7 +193,6 @@ dojo.declare("goat.elements.TwistieSecti
 		},
 
 		createTwistie: function(){
-			console.log("in create twistie, x = " + this.x + " y = " + this.y);
     		var pys = this.y;
     		var pym = pys+5;
     		var pye = pys+10;
@@ -216,6 +213,7 @@ dojo.declare("goat.elements.TwistieSecti
 			}
 			return false;
 		}
+	
 
 
 });