Page MenuHomePhabricator
Differential D16290

Respect 12 hour vs 24 hour time formats in Calendar day views
ClosedPublic
Actions

Authored by epriestley on Jul 13 2016, 5:27 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Apr 9, 5:12 AM
Unknown Object (File)
Mon, Apr 1, 3:23 AM
Unknown Object (File)
Mon, Apr 1, 3:23 AM
Unknown Object (File)
Mon, Apr 1, 3:23 AM
Unknown Object (File)
Thu, Mar 28, 5:55 PM
Unknown Object (File)
Thu, Mar 28, 2:22 PM
Unknown Object (File)
Thu, Mar 28, 2:21 PM
Unknown Object (File)
Thu, Mar 28, 2:16 PM
View All Files
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T9202: Calendar UI doesn't honor user time format preferences in day view
T10932: Daily view of calendar uses 12h (am/pm) marking of time, although preferences set to 24h
Commits
rP8062423271c7: Respect 12 hour vs 24 hour time formats in Calendar day views
Summary

Fixes T9202.

Test Plan
  • Viewed day in 12-hour, saw "8:00 PM".
  • Viewed day in 24-hour, saw "16:00".

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 39180.Jul 13 2016, 5:27 PM
epriestley retitled this revision from to Respect 12 hour vs 24 hour time formats in Calendar day views.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
epriestley added a task: T10932: Daily view of calendar uses 12h (am/pm) marking of time, although preferences set to 24h.
epriestley updated this object.Jul 13 2016, 5:28 PM
epriestley added a task: T9202: Calendar UI doesn't honor user time format preferences in day view.
chad accepted this revision.Jul 13 2016, 8:04 PM
chad edited edge metadata.
This revision is now accepted and ready to land.Jul 13 2016, 8:04 PM
epriestley added a comment.Jul 13 2016, 8:05 PM

I guess 8:00 PM is actually 20:00 but that's a transcription error rather than an actual bug.

Closed by commit rP8062423271c7: Respect 12 hour vs 24 hour time formats in Calendar day views (authored by epriestley, committed by epriestley). · Explain WhyJul 13 2016, 10:39 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
resources/
celerity/
4 lines
src/
view/
phui/
calendar/
2 lines
webroot/
rsrc/
js/
application/
calendar/
4 lines

Diff 39186

View Options

resources/celerity/map.php

Loading...
View Options

src/view/phui/calendar/PHUICalendarDayView.php

Loading...
View Options

webroot/rsrc/js/application/calendar/behavior-day-view.js

Loading...
Phabricator · Built by Phacility