[Mono-bugs] [Bug 81742][Nor] Changed - [PATCH] MonthCalendar is not appearing.

[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=81742

--- shadow/81742	2007-05-26 14:19:49.000000000 -0400
+++ shadow/81742.tmp.24739	2007-05-26 14:57:04.000000000 -0400
@@ -5,16 +5,16 @@
 OS Details: 
 Status: NEW   
 Resolution: 
 Severity: Unknown
 Priority: Normal
 Component: Windows.Forms
-AssignedTo: toshok at ximian.com                            
+AssignedTo: rolfkvinge at ya.com                            
 ReportedBy: billholmes54 at gmail.com               
 QAContact: mono-bugs at ximian.com
-TargetMilestone: ---
+TargetMilestone: 1_2
 URL: 
 Cc: 
 Summary: [PATCH] MonthCalendar is not appearing.
 
 MonthCalendar is not appearing in the simple example attached.
 
@@ -28,6 +28,24 @@
 proposed patch
 
 
 ------- Additional Comments From billholmes54 at gmail.com  2007-05-26 14:19 -------
 The proposed patch feels a little too simple to me.  But it may give
 someone who knows what that are doing a good start. 
+
+------- Additional Comments From monkey at jpobst.com  2007-05-26 14:57 -------
+It's the right idea, but you're right, it's a bit too simple.  :)
+
+Control's constructor will set the size to MonthCalendar.DefaultSize,
+however it is returning (0,0).  This is because it multiplies the size
+by MonthCalendar.calendar_dimensions, which is set to (1,1) in
+MonthCalendar's constructor, but that isn't called until after
+Control's constructor.
+
+Using
+Size calendar_dimensions = new Size (1, 1);
+instead of setting it in the constructor produces better results,
+however it doesn't quite match MS, so some other tweaks may be needed.
+
+Since MonthCalendar is rolf's baby, assigning to him to look at.
+
+Thanks for the report!