Page MenuHomePhabricator
Differential D12846

Calendar tooltips should always show start and end time of events.
ClosedPublic
Actions

Authored by lpriestley on May 14 2015, 7:18 PM.
Tags
None
Referenced Files
F18850334: D12846.id30900.diff
Thu, Oct 30, 3:54 PM
F18828159: D12846.id.diff
Fri, Oct 24, 2:40 PM
F18825700: D12846.diff
Thu, Oct 23, 11:40 PM
F18774148: D12846.id30901.diff
Thu, Oct 9, 3:28 PM
F18751036: D12846.id.diff
Sat, Oct 4, 9:36 AM
F18670606: D12846.id.diff
Sep 25 2025, 12:48 AM
F18659560: D12846.id.diff
Sep 23 2025, 1:23 PM
F18615511: D12846.id.diff
Sep 14 2025, 2:39 PM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
chad
Group Reviewers
Blessed Reviewers
Maniphest Tasks
T8191: Month day event tooltips should show start and end time only
Commits
Restricted Diffusion Commit
rP87bf532c4146: Calendar tooltips should always show start and end time of events.
Summary

Closes T8191, Calendar tooltips should always show start and end time of events.

Test Plan

Open month view, hover over, All day events, multi-day all day events, multi-day non-all-day events, and regular events. Tooltips should be "All day", "All day, <start> - <end>", "<start>-<end>", and "<start>-<end>", respectively.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

lpriestley updated this revision to Diff 30900.May 14 2015, 7:18 PM
lpriestley retitled this revision from to Calendar tooltips should always show start and end time of events..
lpriestley updated this object.
lpriestley edited the test plan for this revision. (Show Details)
lpriestley added a reviewer: epriestley.
lpriestley added a task: T8191: Month day event tooltips should show start and end time only.
Herald added reviewers: chad, Blessed Reviewers. · View Herald TranscriptMay 14 2015, 7:18 PM
Herald added subscribers: epriestley, Korvin. · View Herald Transcript
epriestley accepted this revision.May 14 2015, 7:24 PM
epriestley edited edge metadata.
epriestley added inline comments.
src/view/phui/calendar/PHUICalendarMonthView.php
304 ↗(On Diff #30900)

debugging code?

This revision is now accepted and ready to land.May 14 2015, 7:24 PM
lpriestley updated this revision to Diff 30901.May 14 2015, 7:34 PM
lpriestley marked an inline comment as done.
lpriestley edited edge metadata.

removing debugging code

Closed by commit rP87bf532c4146: Calendar tooltips should always show start and end time of events. (authored by lpriestley, committed by lpriestley). · Explain WhyMay 14 2015, 7:35 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
resources/
celerity/
4 lines
src/
view/
form/
control/
7 lines
phui/
calendar/
34 lines
webroot/
rsrc/
css/
phui/
calendar/
4 lines

Diff 30902

View Options

resources/celerity/map.php

Loading...
View Options

src/view/form/control/AphrontFormDateControlValue.php

Loading...
View Options

src/view/phui/calendar/PHUICalendarListView.php

Loading...
View Options

webroot/rsrc/css/phui/calendar/phui-calendar-month.css

Loading...
Phabricator · Built by Phacility