Document & add endDate handing to the UI #384
jkamenik
started this conversation in
Feature Requests
Replies: 1 comment
-
I would greatly appreciate the addition of some way to mark end dates as inclusive in events for purposes of the graphical view. I use the metadata endDate to surface events that are ending on a given day, and having to work around the "manually add +1 day to show as expected in graphical view" exclusive-date workaround is very redundant. My other option is setting specific start and end times, but then I have timestamps showing in graphical view for events that don't need timestamps because they're actually all-day. A property to set inclusivity would help make the multi-day behaviour more intuitive. |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
No where in the docs is
endDate
described. Nor is it in the UI. However, it works. The following two events both render according to spec.Thoughts on Non-inclusive end-dates
End dates/times are exclusive, while start dates/times are inclusive. This behavior is noted in the event parsing section of Full Calendar doc. And it is a also the iCal spec. And it makes sense for computers, but is not intuitive.
It is done this way so that an event's end date can be the same as the next event's start date and regardless of system precision there is no overlap. So, the first event ends at 12:29:59 on the 24th; which renders the way one expects However, the second ends 11:59:59 on the 23rd and doesn't render on the 24th.
The later behavior seems like a bug to most humans. Which is why most calendar apps handle this by making the event inclusive in the UI (i.e., from the 22 to the 24th) but generate the event exclusive (i.e.,
endDate: 2023-01-25
) in the document.Since we easily see the document first and foremost, it would likely be raised as a bug if we follow spec and made
endDate
exclusive since folks would have to remember it was exclusive.One through might be to add and use
endDateInclusive
and forallDay: true
events add 1 day before sending it for rendering. Or addendDateExclusive
as an alternative property toendDate
. ForallDay: false
still write outendDate
since it makes sense. And forallDay: true
useendDateExclusive
instead so it is more clear in the file what is happening.Just some thoughts.
Beta Was this translation helpful? Give feedback.
All reactions