Page MenuHomePhabricator

Make calendar intepret all-day dates in a more consistent way
ClosedPublic

Authored by epriestley on Oct 31 2016, 6:55 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Dec 17, 2:02 PM
Unknown Object (File)
Fri, Dec 13, 10:43 PM
Unknown Object (File)
Wed, Dec 11, 8:07 AM
Unknown Object (File)
Sat, Dec 7, 1:46 AM
Unknown Object (File)
Tue, Dec 3, 4:18 PM
Unknown Object (File)
Sun, Dec 1, 8:56 PM
Unknown Object (File)
Sun, Dec 1, 5:27 AM
Unknown Object (File)
Fri, Nov 29, 9:02 AM
Subscribers
None

Details

Summary

In ICS, an event on "Nov 1" starts on "2016-11-01" and ends on "2016-11-02".

This is convenient for computers, but this isn't what users expect to enter in date controls. They expect to enter "nov 1" to "Nov 1" for a one-day, all-day event. This is consistent with other applications.

Store the value the user entered, but treat it as the first second of the next day when actually using it if the event is an all day event.

Test Plan

Mucked around with multi-day all-day events, recurring all-day events, imports, etc. Couldn't catch any weird/unintuitive stuff anymore offhand.

(Previously, entering "Nov 1" to "Nov 2" created a one-day event, which was unclear.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Make calendar intepret all-day dates in a more consistent way.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Oct 31 2016, 7:05 PM
This revision was automatically updated to reflect the committed changes.