Page MenuHomePhabricator
Differential D10884

Add ReallyMajorEvent to PHUITimelineView
ClosedPublic
Actions

Authored by chad on Nov 20 2014, 12:47 AM.
Tags
None
Referenced Files
F18938528: D10884.id26138.diff
Nov 11 2025, 3:54 AM
F18921245: D10884.id.diff
Nov 9 2025, 1:54 PM
F18904875: D10884.diff
Nov 8 2025, 11:35 AM
F18879429: D10884.id26147.diff
Nov 6 2025, 6:27 PM
F18844831: D10884.id.diff
Oct 29 2025, 7:47 AM
F18826246: D10884.diff
Oct 24 2025, 3:10 AM
F18638205: D10884.diff
Sep 17 2025, 10:01 AM
F18623901: D10884.diff
Sep 15 2025, 6:02 PM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
btrahan
Commits
Restricted Diffusion Commit
rPd908e7673345: Add ReallyMajorEvent to PHUITimelineView
Summary

For actions like "Close" that are in theory stopping the timeline, we should display some disruption to the line itself.

Test Plan

Tested in UIExamples

Screen_Shot_2014-11-19_at_4.45.23_PM.png (552×760 px, 84 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad updated this revision to Diff 26138.Nov 20 2014, 12:47 AM
chad retitled this revision from to Add ReallyMajorEvent to PHUITimelineView.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added reviewers: epriestley, btrahan.
Herald added subscribers: epriestley, Korvin. · View Herald TranscriptNov 20 2014, 12:47 AM
chad edited the test plan for this revision. (Show Details)Nov 20 2014, 12:47 AM
Harbormaster completed remote builds in B3108: Diff 26138.Nov 20 2014, 12:47 AM
epriestley accepted this revision.Nov 21 2014, 12:25 AM
epriestley edited edge metadata.

Not totally sold on this yet but let's give it a shot.

This revision is now accepted and ready to land.Nov 21 2014, 12:25 AM
chad added a comment.Nov 21 2014, 12:32 AM

Yeah I read through as few really long Github Issues the other day and it caught my eye (as it was intended). When you're quickly scanning a long task it's a nice detail that helps orient you. https://github.com/twbs/bootstrap/issues/13823

Closed by commit rPd908e7673345: Add ReallyMajorEvent to PHUITimelineView (authored by chad, committed by chad). · Explain WhyNov 21 2014, 12:33 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
resources/
celerity/
6 lines
src/
applications/
uiexample/
examples/
25 lines
view/
phui/
41 lines
webroot/
rsrc/
css/
phui/
9 lines

Diff 26147

View Options

resources/celerity/map.php

Loading...
View Options

src/applications/uiexample/examples/PHUITimelineExample.php

Loading...
View Options

src/view/phui/PHUITimelineEventView.php

Loading...
View Options

webroot/rsrc/css/phui/phui-timeline-view.css

Loading...
Phabricator · Built by Phacility