[Mono-bugs] [Bug 78555][Min] New - [PATCH] PropertyGrid: set_PropertySort() does not update toolbar

[bugzilla-daemon at bugzilla.ximian.com]

Please do not reply to this email- if you want to comment on the bug, go to the
URL shown below and enter your comments there.

Changed by atsushi at ximian.com.

http://bugzilla.ximian.com/show_bug.cgi?id=78555

--- shadow/78555	2006-06-01 03:08:57.000000000 -0400
+++ shadow/78555.tmp.23878	2006-06-01 03:08:57.000000000 -0400
@@ -0,0 +1,68 @@
+Bug#: 78555
+Product: Mono: Class Libraries
+Version: 1.1
+OS: 
+OS Details: 
+Status: NEW   
+Resolution: 
+Severity: 
+Priority: Minor
+Component: Windows.Forms
+AssignedTo: peter at novonyx.com                            
+ReportedBy: atsushi at ximian.com               
+QAContact: mono-bugs at ximian.com
+TargetMilestone: ---
+URL: 
+Cc: 
+Summary: [PATCH] PropertyGrid: set_PropertySort() does not update toolbar
+
+Setting PropertyGrid.PropertySort should update sorting related toolbar
+items. Those toolbar buttons are updated only when we explicitly click the
+toolbar buttons.
+
+Example repro:
+
+using System;
+using System.Drawing;
+using System.Windows.Forms;
+
+public class Test : Form
+{
+        public static void Main ()
+        {
+                Application.Run (new Test ());
+        }
+
+        public Test ()
+        {
+                Size = new Size (300, 400);
+                PropertyGrid pg = new PropertyGrid ();
+                pg.Dock = DockStyle.Fill;
+                pg.SelectedObject = new Control ();
+                Controls.Add (pg);
+                Button b = new Button ();
+                b.Dock = DockStyle.Bottom;
+                b.Click += delegate (object o, EventArgs e) {
+                        pg.PropertySort = PropertySort.Alphabetical;
+                };
+                Controls.Add (b);
+        }
+}
+
+1. Build and run the above.
+2. Push the button at the bottom of the form. It logically sets
+PropertySort as Alphabetical.
+
+
+Actual Results:
+
+It does not update the toolbar buttons.
+
+Expected Results:
+
+It should reflect the toolbar status.
+
+Additional Information:
+
+Patch being attached. I also removed some additional lines from
+toolbar_ButtonClick since set_PropertySort() handles them.