Closed Bug 343836 Opened 18 years ago Closed 18 years ago

Day/Week View: All day events shown on wrong day or twice (box reuse issue?)

Categories

(Calendar :: Calendar Frontend, defect)

Product:
Calendar
Component:
Calendar Frontend
Platform:
x86
Windows XP
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
VERIFIED FIXED

People

(Reporter: dbo, Assigned: jminta)

Details

Attachments

(1 file)

delete backwards
1.44 KB, patch
jminta
: first-review+
Details | Diff | Splinter Review 
1. create two all day events on the same day
2. switch to next week (or day)
=> one event still appears (showing on the wrong day)
Confirmed with Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a1) Gecko/2006070607 Sunbird/0.3a2+

Playing around with back, forward and Go to Today you can get two event boxes for the same event too.
Assignee: nobody → base
Component: Lightning → Base
QA Contact: lightning → base
Version: unspecified → Trunk
The bugspam monkeys have struck again. They are currently chewing on default assignees for Calendar. Be afraid for your sanity!
Assignee: base → nobody
Component: Internal Components → Calendar Views
Flags: blocking0.3?
QA Contact: base → views
Summary: [multi] day view: multiple all day events on same day → Day/Week View: All day events shown on wrong day or twice (box reuse issue?)
Flags: blocking0.3? → blocking0.3+
Attached patch delete backwards — — Splinter Review 
It's a bad idea to splice() arrays while you're iterating through them.  We'll only get the first half of the boxes with the current code, but iterating backwards makes sure we get them all.
Assignee: nobody → jminta
Status: NEW → ASSIGNED
Attachment #230537 - Flags: first-review?(dmose)
Comment on attachment 230537 [details] [diff] [review]
delete backwards

ask mrbkap if flipping to --i instead makes a difference to spidermonkey. r=dmose
Attachment #230537 - Flags: first-review?(dmose) → first-review+
Patch checked in
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Verified with Sunbird/0.3a2+ (2006080207).
Verified with Thunderbird 1.5.0.5 (20060719) + Lightning/0.1+ (2006080207).
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: