incubator-xap-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From jmarga...@apache.org
Subject svn commit: r506644 - /incubator/xap/trunk/codebase/src/xap/widgets/dojo/TreeTable.js
Date Mon, 12 Feb 2007 21:00:47 GMT
Author: jmargaris
Date: Mon Feb 12 13:00:46 2007
New Revision: 506644

URL: http://svn.apache.org/viewvc?view=rev&rev=506644
Log:
https://issues.apache.org/jira/browse/XAP-303
Thanks to Trevor Oldak and David Gennaco

Modified:
    incubator/xap/trunk/codebase/src/xap/widgets/dojo/TreeTable.js

Modified: incubator/xap/trunk/codebase/src/xap/widgets/dojo/TreeTable.js
URL: http://svn.apache.org/viewvc/incubator/xap/trunk/codebase/src/xap/widgets/dojo/TreeTable.js?view=diff&rev=506644&r1=506643&r2=506644
==============================================================================
--- incubator/xap/trunk/codebase/src/xap/widgets/dojo/TreeTable.js (original)
+++ incubator/xap/trunk/codebase/src/xap/widgets/dojo/TreeTable.js Mon Feb 12 13:00:46 2007
@@ -452,7 +452,11 @@
 	},
 	
 	_rowRemoved: function(row){
-		
+
+		if (this.isSelected(row)){
+			xap.util.ArrayHelper.removeElement(this._selectedItems, row);
+		}
+
 		//recurse through all children
 		for (var i = 0; i<row._rows.length;i++){
 			this._rowRemoved(row._rows[i]);
@@ -793,6 +797,11 @@
 	},
 	
 	setSelected : function (selected){
+	    // IMPORTANT we need to set _selected when a user event causes the row 
+	    // to be selected, it currently is only set when an Xml event (setting selected to true)
+	    // occurs
+	    // ALSO we need to write back to the Xml element when the user selects or de-selects
+	    // a row.
 		this._selected = selected;
 		if (this._table && selected){
 			this._table.selectRow(this);



Mime
View raw message