Invalid number of rows rendered in Month view about gwt-cal HOT 9 CLOSED

Carlos, i'm forwarding this one to you, if you don't mind taking a look

A quick reference check showed both gwt datepicker 
(http://screencast.com/t/OGQzNGUzYj) and gxt datepicker 
(http://screencast.com/t/ZWM5ZGQ3) both render 6 row regardless. The real 
google calendar renders 5 when 
needed but the date picker it uses actually always renders 6 rows as well.  If 
it is easier it might be safe to just 
always render 6 rows

Hey Brad, 

I don't mind at all. I'll look into this today.

A Bug fix has been checked-in. Apparently the problem was that we were 
calculating
the number of required rows for the first date shown in the calendar instead of 
a
date ON the actual month of interest displayed in the Month View. This was not 
caused
by the MonthViewHelper#monthViewRequiredRows logic but by the date it was 
passed for
the calculation.

Because for some months the first date shown fell within the same month, the 
code
would correctly display the number of rows. For month views showing days from 
the
previous month, it was the number of rows for the PREVIOUS month what was being
displayed and hence the problem.

Snapshots from Firefox Hosted, Firefox Native, Chrome Native (all in Ubuntu 
x64) and
Internet Explorer 7 (Windows Vista 32-bit) are attached for the months shown 
with the
error when the issue was logged (Feb 2010 & May 2010).

CM

awesome. thanks Carlos!

Hello d.tangren,

Thank you for your feedback. If you can please let me know whether you find the 
issue
solved in your own environment so I can properly change the status of the 
defect I
would really appreciate it. Thanks for reporting the issue.

Best,

CM

P.S. I am jealous I don't have a osx snow leopard to test myself. ;)

Just tested. Works great! Thanks Carlos.