[Mono-bugs] [Bug 82154][Nor] Changed - TreeView: ExpandAll removes all nodes

[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 andyhume32 at yahoo.co.uk.

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

--- shadow/82154	2007-07-20 16:14:49.000000000 -0400
+++ shadow/82154.tmp.12649	2007-08-25 10:04:42.000000000 -0400
@@ -2,13 +2,13 @@
 Product: Mono: Class Libraries
 Version: 1.2
 OS: All
 OS Details: 
 Status: NEW   
 Resolution: 
-Severity: 
+Severity: Unknown
 Priority: Normal
 Component: Windows.Forms
 AssignedTo: toshok at ximian.com                            
 ReportedBy: gert.driesen at pandora.be               
 QAContact: mono-bugs at ximian.com
 TargetMilestone: ---
@@ -24,6 +24,48 @@
 2. follow the on-screen instructions.
 
 ------- Additional Comments From gert.driesen at pandora.be  2007-07-20 16:14 -------
 Created an attachment (id=19590)
 Repro
 
+
+------- Additional Comments From andyhume32 at yahoo.co.uk  2007-08-25 10:04 -------
+I'd checked that hitting the 'NumKeyPad*' key in FolderBrowserDialog 
+did expand-all on the tree so was surprised to see this bug -- and 
+reproduce it.
+
+I modified the repro code in three ways: adding a Reset button to 
+call the current ResetButton_Click method, making the Expand button 
+call a method that just calls "_treeView.ExpandAll ();" method, and 
+add shortcut-keys to buttons.
+
+With the 'new' Expand button, hitting it twice show the correct tree 
+momentarily[1], before ArgEx:
+Unhandled Exception: System.ArgumentOutOfRangeException: '-4' is not 
+a valid value for 'Value'. 'Value' should be between 'Minimum' 
+and 'Maximum'
+Parameter name: Value
+  at System.Windows.Forms.ScrollBar.set_Value (Int32 value) 
+[0x000d5] in ScrollBar.cs:568
+  at (wrapper remoting-invoke-with-check) 
+System.Windows.Forms.ScrollBar:set_Value (int)
+  at System.Windows.Forms.TreeView.ExpandAll () [0x0008c] in 
+TreeView.cs:764
+  at (wrapper remoting-invoke-with-check) 
+System.Windows.Forms.TreeView:ExpandAll ()
+  at MainForm.ExpandButton_Click (System.Object sender, 
+System.EventArgs e) [0x00000]
+[...cut...]
+
+I thought originally using the shortcut-keys (Alt+E etc) affected 
+the behaviour but it seems not.
+
+Also, hitting the 'NumKeyPad*' key seems to have no affect on this 
+issue -- e.g. hitting it before ExpandAll still shows the ExpandAll 
+bug.  Perhaps the difference is (as I understand it) in it calling 
+instead TreeNode.ExpandAll.
+(There a refresh bug when hitting 'NumKeyPad-' when the selected 
+node is off the screen which I'll report sometime).
+
+
+[1] In the original/British English meaning, "_for_ as short time"; 
+not the american "_in_ a short time" e.g. soon/presently.