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/03/30 02:02:04 UTC

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

Author: mturyn
Date: Thu Mar 29 19:02:04 2007
New Revision: 523908

URL: http://svn.apache.org/viewvc?view=rev&rev=523908
Log:
XAP-297: Under IE, table columns grow when headers clicked.

A clean-up method for IE set header and/or first-row cell widths directly; it should have set the outer widths instead to take borders into account.

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=523908&r1=523907&r2=523908
==============================================================================
--- incubator/xap/trunk/codebase/src/xap/widgets/dojo/TreeTable.js (original)
+++ incubator/xap/trunk/codebase/src/xap/widgets/dojo/TreeTable.js Thu Mar 29 19:02:04 2007
@@ -199,9 +199,15 @@
 			
 			// TODO: if the user wants a column to fill the remaining width they should set that column to 100%
 			if (dojo.render.html.ie){
-				this.headerRow.childNodes[i].style.width  = width;
+				dojo.html.setMarginBox(
+						this.headerRow.childNodes[i].style.width,
+						{width: width}
+										) ;
 				if (firstRow && firstRow.childNodes[i]){
-					firstRow.childNodes[i].style.width = width;
+					dojo.html.setMarginBox(
+						firstRow.childNodes[i].style.width,
+						{width: width}
+												) ;					
 				}
 			}
 			else{