When (forward) browsing the month views, a weekly recurring event gets displayed many times. Event-boxes for the new month get added, but the old boxes are not removed. Steps to reproduce: - Switch to month view - Create a weekly recurring event, no end date - Browse to the next month - Observer that the event is displayed twice, once for the currently displayed month, and once for the previous month (as the tooltip tells us) This makes using the month view with recurring events pretty hard, so marking as blocker. (If we can't find a solution, we could turn of the re-use of the grid. That would be a performance hit, but solve this problem)
Actually, all kinds of events stay on the view, not just recurring events. Sometimes the boxes are removed, but most are not. I can't tell what determines if a box is removed or not.
Created attachment 219373 [details] multiweek screenshot Event and day boxes in multiweek view are affected too. Also the box style gets screwed up. (See attached screenshot.) It works in the 2006-04-19-07 build and fails in the 2006-04-20-07 build. Bug 334704 was the only check in to mozilla/calendar during that time.
Created attachment 219377 [details] [diff] [review] patch This patch fixes Bug 334996 and Bug 334724. Therefore the patch on Bug 334724 is obsolete. We should check if the change to calIDuration did not regressed other places.
Comment on attachment 219377 [details] [diff] [review] patch The patch fixes the problem but is wrong. With this patch we always hit the "shouldn't ever happen" case in relayout(). So we need another solution.
Created attachment 219442 [details] [diff] [review] fix proper while The while loop to remove the old events was never getting called. Ignore the newDuration bit, since that's fixed in another bug.
Patch checked in. (the error i saw was bogus, a bug in my setup)