[richfaces-svn-commits] JBoss Rich Faces SVN: r1284 - trunk/sandbox/scrollable-grid/src/main/javascript/ClientUI/controls/grid.

[richfaces-svn-commits at lists.jboss.org]

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);
 			}
 		}
 	},