Page MenuHomePhabricator

Probably fix some display issues with all-day events?
ClosedPublic

Authored by epriestley on Nov 7 2016, 6:44 PM.
Tags
None
Referenced Files
F19794067: D16815.diff
Thu, Feb 26, 6:22 PM
F19507246: D16815.diff
Jan 9 2026, 6:45 PM
F19507232: D16815.id40493.diff
Jan 9 2026, 6:43 PM
F19089288: D16815.id40490.diff
Dec 3 2025, 6:34 AM
F19076139: D16815.diff
Dec 1 2025, 11:39 AM
F18903358: D16815.id.diff
Nov 8 2025, 4:55 AM
F18893995: D16815.diff
Nov 7 2025, 1:31 PM
F18815024: D16815.id40490.diff
Oct 21 2025, 12:28 AM
Subscribers
None

Details

Summary

Ref T11801. These are pretty fiddly because users expect to see the end time for timed events ("10 AM - 11 AM" is ONE hour long) but not for all-day events ("Nov 2 - Nov 3" is TWO days long!)

We also want to store the thing the user actually entered so we don't lose data if they un-all-day the event later.

This may take a little more fiddling since it feels a little shaky, but I couldn't break this version immediately.

Test Plan

Imported a French holiday, got proper display in the UI.

Diff Detail

Repository
rP Phabricator
Branch
cal3
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 14381
Build 18728: Run Core Tests
Build 18727: arc lint + arc unit

Event Timeline

epriestley retitled this revision from to Probably fix some display issues with all-day events?.
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.Nov 7 2016, 6:50 PM
This revision was automatically updated to reflect the committed changes.