You are viewing a plain text version of this content. The canonical link for it is here.
Posted to xap-commits@incubator.apache.org by mt...@apache.org on 2007/05/18 20:09:06 UTC

svn commit: r539586 - /incubator/xap/trunk/codebase/src/xap/widgets/dojo/TreeTable.js

Author: mturyn
Date: Fri May 18 13:09:05 2007
New Revision: 539586

URL: http://svn.apache.org/viewvc?view=rev&rev=539586
Log:
Small fix (in setting the column width in extendLastColumn()) that fixes a bug under IE---dragging wasn't working well.

Still have the problem overflowing the rightmost header under IE if the table header is
narrower than the table widget's full width.

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=539586&r1=539585&r2=539586
==============================================================================
--- incubator/xap/trunk/codebase/src/xap/widgets/dojo/TreeTable.js (original)
+++ incubator/xap/trunk/codebase/src/xap/widgets/dojo/TreeTable.js Fri May 18 13:09:05 2007
@@ -736,6 +736,7 @@
 		// will need.  If we don't do this, space ends up getting
 		// added to or subtracted from (it seems) the first cell,
 		// which ends up with a width of "100%":
+		this.headerTable.style.tableLayout = "fixed" ;
 		this.headerTable.style.width = "auto" ;
 		this.table.style.width = "auto" ;			
 
@@ -746,16 +747,17 @@
 		if(len > 0){
 			var newWidth = dojo.html.getMarginBox(headerChildren[len-1]).width ;
 			newWidth += changeInPixels ;
-			headerChildren[lastIdx].style.width = newWidth+"px";				
+			headerChildren[lastIdx].style.width = newWidth+"px";						
+			
 			var sizer = this.columnsSizer ;
 			// Put all the extra space in the last column:
 			sizer.children[lastIdx].sizeShare = newWidth ;
 			sizer.children[lastIdx].sizeActual = newWidth ;
-			this._columns[lastIdx]._explicitColumnWidth
-						 = newWidth + "px" ;
+			this._columns[lastIdx].setStatedWidth(newWidth) ;
 		}
-		// Match headers to body columns:
+		// Match body to to them changed header column:
 		this._fixColumnWidths(lastIdx,lastIdx) ;
+
 		
 		// Fix these widths (in the sense of "set them");
 		// leaving them free seems to hurt scrolling:
@@ -765,7 +767,6 @@
 			= this.headerTable.scrollWidth +"px" ;
 			
 		this._renewColumnSizers();
-
 	},
 
 	
@@ -1477,9 +1478,11 @@
 		
 		// user grabbed the last sizer:
 		if( this._grabbedTheLastSizer && this._lastTableRightPx ){
+		
 			// The last sizer corresponds to a non-column;
 			// it defines the right boundary of the table:
 			var currentTableRightPx = this._lastSizer.offsetLeft ;
+		
 			var deltaTableRightPx 
 				= currentTableRightPx 
 					- this._lastTableRightPx ;