2008-04-28 Jonathan Pobst <monkey@jpobst.com>

* ListBox.cs: Math.Min should be Math.Max. [Fixes bug #384183] svn path=/trunk/mcs/; revision=102051
mono · Apr 28, 2008 · 5c865e3 · 5c865e3
1 parent 1e72fad
commit 5c865e3
Show file tree

Hide file tree

Showing 2 changed files with 5 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,7 @@
+2008-04-28  Jonathan Pobst  <monkey@jpobst.com>
+
+	* ListBox.cs: Math.Min should be Math.Max.  [Fixes bug #384183]
+
 2008-04-28  George Giolfan  <georgegiolfan@yahoo.com>
 
 	* ThemeVisualStyles.cs: Added partial support for ScrollBar (based on the

diff --git a/mcs/class/Managed.Windows.Forms/System.Windows.Forms/ListBox.cs b/mcs/class/Managed.Windows.Forms/System.Windows.Forms/ListBox.cs
@@ -2083,7 +2083,7 @@ private bool UpdateVerticalScrollBar ()
 			if (canvas_size.Height > items_area.Height) {
 				show = true;
 				vscrollbar.Maximum = Items.Count - 1;
-				vscrollbar.LargeChange = Math.Min (items_area.Height / ItemHeight, 0);
+				vscrollbar.LargeChange = Math.Max (items_area.Height / ItemHeight, 0);
 			} else if (ScrollAlwaysVisible) {
 				show = true;
 				enabled = false;