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 ;