My favorites | Sign in
Logo
          
New issue | Search
for
| Advanced search | Search tips
Issue 233: Improve display of multi-day events
2 people starred this issue and may be notified of changes. Back to list
Status:  Fixed
Owner:  ----
Closed:  Nov 2008
Type-Defect
Priority-Low
Milestone-1.0
Component-UI
Component-time


Sign in to add a comment
 
Reported by jdcohenesq, Jul 13, 2008
Multi-day events may display in an unexpected way.

The 12 July 2008 code sprint (and follow-up) changed the test for whether
an event occurs within a time range.  See  Issue178 .

  Old test:  Is the *start_time* within range?
  New test:  Is *any part of the event* within range?

The reason for the change is to improve handling of multi-day events.  

Consider OSCON, which starts July 21, ends July 25.

Under the old test, if you viewed the Calagator overview page on July 22,
23, 24, or 25, OSCON would not appear.
Under the new test, it will appear under Today's events, but display in an
unexpected way.  For instance on July 22, you will see a group of events
listed under "Today".  It will then list events that started earlier than
today, including their start times.  So it will look like this:

Today
  Monday          O'Reilly Open Source Convention
  Jul 21 2008     7:30am through Friday, July 25 at 2pm
                       Oregon Convention Center 

This is unexpected  because Today is July 22.  

This *not* a issue with *which* events are listed.  But it is an issue with
how the times are described on the Overview page.  There are similar issues
for the Browse Events page (and for the filtered events on that page).  


Comment 1 by jdcohenesq, Jul 13, 2008
This should be reflected in three views:
Events Overview
Events Browse unfiltered
Events Browse filtered
Comment 2 by jdcohenesq, Jul 19, 2008
r978 improves the display, but a better display may be possible.
Labels: -Milestone-Release1.0 Milestone-Post1.0
Comment 5 by jdcohenesq, Nov 15, 2008
I am reducing priority to Low, and changing status to Fixed because this issue has
not been raised again after various improvements to the display. 
Status: Fixed
Labels: -Priority-Medium Priority-Low
Sign in to add a comment

Hosted by Google Code