Author: toshok
Date: 2007-01-05 13:19:00 -0500 (Fri, 05 Jan 2007)
New Revision: 70547

Modified:
   trunk/mcs/class/Managed.Windows.Forms/System.Windows.Forms/ChangeLog
   trunk/mcs/class/Managed.Windows.Forms/System.Windows.Forms/DataGrid.cs
Log:
2007-01-05  Chris Toshok  <[EMAIL PROTECTED]>

        * DataGrid.cs: don't forceably rebind columns in SetDataSource
        unless our list manager has changed (i.e. unless we have reason to
        believe our columns have changed).  Fixes #80422.



Modified: trunk/mcs/class/Managed.Windows.Forms/System.Windows.Forms/ChangeLog
===================================================================
--- trunk/mcs/class/Managed.Windows.Forms/System.Windows.Forms/ChangeLog        
2007-01-05 18:13:04 UTC (rev 70546)
+++ trunk/mcs/class/Managed.Windows.Forms/System.Windows.Forms/ChangeLog        
2007-01-05 18:19:00 UTC (rev 70547)
@@ -1,3 +1,9 @@
+2007-01-05  Chris Toshok  <[EMAIL PROTECTED]>
+
+       * DataGrid.cs: don't forceably rebind columns in SetDataSource
+       unless our list manager has changed (i.e. unless we have reason to
+       believe our columns have changed).  Fixes #80422.
+
 2007-01-05  Jonathan Pobst  <[EMAIL PROTECTED]>
 
        * Control.cs: Move LayoutEngine.Layout to the OnLayout method, so

Modified: trunk/mcs/class/Managed.Windows.Forms/System.Windows.Forms/DataGrid.cs
===================================================================
--- trunk/mcs/class/Managed.Windows.Forms/System.Windows.Forms/DataGrid.cs      
2007-01-05 18:13:04 UTC (rev 70546)
+++ trunk/mcs/class/Managed.Windows.Forms/System.Windows.Forms/DataGrid.cs      
2007-01-05 18:19:00 UTC (rev 70547)
@@ -2325,9 +2325,9 @@
                        if (list_manager != null)
                                ConnectListManagerEvents ();
 
-                       BindColumns ();
+                       if (old_lm != list_manager) {
+                               BindColumns ();
 
-                       if (old_lm != list_manager) {
                                /* reset first_visiblerow to 0 here before
                                 * doing anything that'll requires us to
                                 * figure out if we need a scrollbar. */

_______________________________________________
Mono-patches maillist  -  [email protected]
http://lists.ximian.com/mailman/listinfo/mono-patches

Reply via email to