[Mono-bugs] [Bug 81397][Wis] Changed - [Patch] GroupBox Padding property does not re-draw child controls at correct position

[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 monkey at jpobst.com.

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

--- shadow/81397	2007-04-19 12:30:55.000000000 -0400
+++ shadow/81397.tmp.22213	2007-04-19 13:09:34.000000000 -0400
@@ -11,13 +11,13 @@
 AssignedTo: monkey at jpobst.com                            
 ReportedBy: robbel at gmail.com               
 QAContact: mono-bugs at ximian.com
 TargetMilestone: 1_2
 URL: 
 Cc: 
-Summary: GroupBox Padding property does not re-draw child controls at correct position
+Summary: [Patch] GroupBox Padding property does not re-draw child controls at correct position
 
 Please fill in this template when reporting a bug, unless you know what 
 you are doing.
 Description of Problem: Whenever I create a GroupBox that contains a 
 PictureBox as its child control and set the Padding property of the 
 former to something different than the default 0,0,0,0, then the child is 
@@ -59,6 +59,19 @@
 ------- Additional Comments From robbel at gmail.com  2007-04-19 12:30 -------
 Traced it back further to the groupBox.SuspendLayout and
 groupBox.ResumeLayout calls. If I comment them out, the pictureBox is
 always drawn (incorrectly) without any padding. If I leave those
 statements in, at least the first drawing is correct (until resize of
 the form)
+
+------- Additional Comments From monkey at jpobst.com  2007-04-19 13:09 -------
+Thanks for both of the test cases.  I was not making the PictureBox
+DockStyle = Fill, and therefore I hadn't been able to find the issue.
+
+I made some general fixes to GroupBox to implement DefaultPadding and
+that it's DisplayRectangle takes padding into consideration whereas
+Control's does not in r75957.
+
+However, the actual fix for this in DefaultLayout.  I am attaching the
+patch, as we are freezing for the 1.2.4 release today, and I'm not
+going to make a layout change at the last minute, as layout can be
+finicky.