Closed
Bug 311911
Opened 19 years ago
Closed 19 years ago
Allday events are not showing up in multiweek view
Categories
(Calendar :: Sunbird Only, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: sebo.moz, Assigned: mostafah)
Details
Attachments
(1 file)
7.63 KB,
patch
|
jminta
:
first-review+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8b5) Gecko/20051001 Firefox/1.4.1
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a1) Gecko/20051008 Mozilla Sunbird/0.2+
Happened during last week.
Reproducible: Always
Steps to Reproduce:
1.Create Allday event
2.Change to multiweek view
3.
Actual Results:
no allday events shown
Expected Results:
show allday events
Comment 1•19 years ago
|
||
Confirmed.
Regression range:
20051006 works
20051007 does not.
Bug 309000 was the only change in that range to touch multiweekView, cc'ing
gekacheka.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Comment 2•19 years ago
|
||
I think I found the solution:
With bug 309000 the property 'lastDateOfView' was renamed to 'endExDateOfView'
in multiweekView.js.
But 'lastDateOfView' is used in calendarWindow.js (line 668) in method
CalendarView.prototype.createEventBox().
After renameing all occurrences of 'endExDateOfView' back to 'lastDateOfView'
the all day event is displayed again in Multiweek View.
I don't know how to create a patch so please check this and create a patch.
Comment 3•19 years ago
|
||
http://lxr.mozilla.org/seamonkey/search?string=lastDateOfView still shows a few
hits. I think they all need to be replaced. gekacheka, can you look at this?
Comment 4•19 years ago
|
||
global replace. Fixes things for me.
Attachment #199733 -
Flags: first-review?(jminta)
Comment 5•19 years ago
|
||
Comment on attachment 199733 [details] [diff] [review]
s/lastDateOfView/endExDateOfView/g
r=jminta
Attachment #199733 -
Flags: first-review?(jminta) → first-review+
Comment 6•19 years ago
|
||
patch checked in
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•