* DataGrid.cs: The large value for our vertical scrollbar should

depend on the number of visible rows, instead of trying to re-use the value of VisibleRowCount, which is the value of *current* visible rows, and can easily end messed up if new rows are added/removed. 2009-11-11 Carlos Alberto Cortez <calberto.cortez@gmail.com> svn path=/branches/mono-2-4/mcs/; revision=145945
mono · Nov 11, 2009 · f579415 · f579415
1 parent 2c8c108
commit f579415
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/mcs/class/Managed.Windows.Forms/System.Windows.Forms/ChangeLog b/mcs/class/Managed.Windows.Forms/System.Windows.Forms/ChangeLog
@@ -1,3 +1,10 @@
+2009-11-11  Carlos Alberto Cortez <calberto.cortez@gmail.com>
+
+	* DataGrid.cs: The large value for our vertical scrollbar should
+	depend on the number of visible rows, instead of trying to re-use the
+	value of VisibleRowCount, which is the value of *current* visible
+	rows, and can easily end messed up if new rows are added/removed.
+
 2009-11-11  Carlos Alberto Cortez <calberto.cortez@gmail.com>
 
 	* DataGrid.cs: When moving the scroll, *dont* ever end the current

diff --git a/mcs/class/Managed.Windows.Forms/System.Windows.Forms/DataGrid.cs b/mcs/class/Managed.Windows.Forms/System.Windows.Forms/DataGrid.cs
@@ -3318,7 +3318,10 @@ internal Rectangle ParentRowsArea {
 		}
 
 		int VLargeChange {
-			get { return VisibleRowCount; }
+			get { 
+				// the possible number of visible rows
+				return cells_area.Height / RowHeight;
+			}
 		}
 
 		#endregion Instance Properties