Closed Bug 329800 Opened 18 years ago Closed 18 years ago

Agendas today/tomorrow/soon filters end date should exclude 00:00:00 of next day

Categories

(Calendar :: Lightning Only, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: ssitter, Assigned: jminta)

Details

Attachments

(1 file)

Agendas today/tomorrow/soon filters end date should exclude 00:00:00 of next day

At the moment the end date of the today/tomorrow/soon filter rule includes 00:00:00 of next day. For example the 'Today' filter includes all events that start between 'now' and 'tomorrow 00:00:00'. As the result all-day events and events that start at 00:00:00 are shown in the wrong section.

Steps to Reproduce:
(Depending on your local timezone you need the patch on bug 329647 first.)
1) Create an all-day event or an event that start at 00:00:00 tomorrow.
2) Check the Agenda

Actual Results:
Event is shown in 'Today' section.

Expected Results:
Event is shown in 'Tomorrow' section.

You get similar results if you schedule the event for the day after tomorrow with 'Tomorrow' and 'Soon' section.

Reference:
function refreshPeriodDates() in
http://lxr.mozilla.org/mozilla/source/calendar/lightning/content/agenda-tree.js#383
Attached patch change <= to <Splinter Review
Patch changes the <= comparison to just <, which is the same as == -1.
Assignee: nobody → jminta
Status: NEW → ASSIGNED
Attachment #214639 - Flags: first-review?(dmose)
Comment on attachment 214639 [details] [diff] [review]
change <= to <

r=dmose; thanks for the patch!
Attachment #214639 - Flags: first-review?(dmose) → first-review+
Patch checked in.
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.