Issue 1087: Calendar not responding properly to Option weekMode: "variable"
Status:  Invalid
Owner: ----
Closed:  Aug 2011
Reported by mike.win...@gmail.com, Aug 18, 2011
I've been doing some heavy modifications, this calendar is really well done. I think the only actual bug I have found in my travels so far is that the calendar first off does not have a default option for weekMode, so it was hard to find.

Second, when weekMode is set to "variable" instead of "fixed" (default), there are still 6 rows generated. In August 2011, the time of posting, there should only be 5 rows under weekMode: "variable"

Under function BasicView:
    function renderBasic(maxr, r, c, showNumbers)
        if(firstTime){
            buildSkeleton(maxr, showNumbers);

That buildSkeleton call should be
    buildSkeleton(rowCnt, showNumbers);

Making that change honors the actual specific row count, and will update the table properly when weekMode is set to "variable"

For me it's line 2191, but i've made a lot of modifications.. It should be somewhere near that area.



Aug 18, 2011
#1 mike.win...@gmail.com
Definitely should have done some further investigations before posting this.

The border styles I had applied made it look like a bug without the change, however maxr is the correct variable. There isn't a problem.

My apologies~ Ignore this thread.
Aug 24, 2011
Project Member #2 althaus.it
(No comment was entered for this change.)
Status: Invalid