Author: dmorozov Date: 2007-06-25 03:30:31 -0400 (Mon, 25 Jun 2007) New Revision: 1284 Modified: trunk/sandbox/scrollable-grid/src/main/javascript/ClientUI/controls/grid/GridBody.js Log: Fix ensureVisible method's setting scroll position in grid Modified: trunk/sandbox/scrollable-grid/src/main/javascript/ClientUI/controls/grid/GridBody.js =================================================================== --- trunk/sandbox/scrollable-grid/src/main/javascript/ClientUI/controls/grid/GridBody.js 2007-06-22 18:08:16 UTC (rev 1283) +++ trunk/sandbox/scrollable-grid/src/main/javascript/ClientUI/controls/grid/GridBody.js 2007-06-25 07:30:31 UTC (rev 1284) @@ -482,8 +482,8 @@ this.setProgressCtrl(new ClientUI.common.box.SplashBox(null, this.getElement(), 300)); } this.splash.setSize(this.scrollBox.getViewportWidth(), this.scrollBox.getViewportHeight()); + this.splash.updateLayout(); this.splash.show(); - this.splash.updateLayout(); }, forceReRender: function() { if(ClientUILib.isIE && !ClientUILib.isIE7) { @@ -744,9 +744,9 @@ var visibleRows = parseInt(this.contentBox.getHeight() / this.defaultRowHeight, 10) + 1; if(this.grid.dataModel.getCount() > visibleRows) { var y = index*this.defaultRowHeight; + this.scrollBox.getElement().scrollTop = y; this.currentPos = 0; - this._onContentVScroll(y); - this.scrollBox.getElement().scrollTop = y; + this._onContentVScroll(this.scrollBox.getElement().scrollTop); } } },