Page MenuHomePhabricator
Differential D16210

Build that thing someone posted a screenshot of on Facebook
ClosedPublic
Actions

Authored by epriestley on Jul 1 2016, 1:44 AM.
Tags
None
Referenced Files
F13213104: D16210.diff
Fri, May 17, 7:23 AM
F13211480: D16210.id38987.diff
Fri, May 17, 5:48 AM
F13209336: D16210.id38993.diff
Thu, May 16, 11:29 PM
F13176670: D16210.id38998.diff
Wed, May 8, 4:15 PM
Unknown Object (File)
Fri, May 3, 8:42 AM
Unknown Object (File)
Wed, Apr 24, 10:19 PM
Unknown Object (File)
Apr 12 2024, 11:33 PM
Unknown Object (File)
Mar 15 2024, 1:05 PM
View All Files
Subscribers
None

Details

Reviewers
chad
Commits
rPdc37789d530f: Build that thing someone posted a screenshot of on Facebook
Summary

Seemed kinda cool.

Test Plan

Screen Shot 2016-06-30 at 6.36.26 PM.png (423×493 px, 51 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 38987.Jul 1 2016, 1:44 AM
epriestley retitled this revision from to Build that thing someone posted a screenshot of on Facebook.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.Jul 1 2016, 1:44 AM

pasted_file (74×77 px, 2 KB)

epriestley added a comment.Jul 1 2016, 1:51 AM

maybe we need a bucket of water

for putting fires out :)

chad added a comment.Jul 1 2016, 2:23 AM

Tabs should have a means of setting a color/error/warning, I think? The token looks a little odd offhand.

epriestley updated this revision to Diff 38993.Jul 1 2016, 2:39 AM
epriestley edited edge metadata.
  • Use tab color instead.

Screen Shot 2016-06-30 at 7.39.09 PM.png (350×464 px, 43 KB)

chad accepted this revision.Jul 1 2016, 3:22 AM
chad edited edge metadata.
This revision is now accepted and ready to land.Jul 1 2016, 3:22 AM
Closed by commit rPdc37789d530f: Build that thing someone posted a screenshot of on Facebook (authored by epriestley, committed by epriestley). · Explain WhyJul 1 2016, 11:36 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
4 lines
applications/
differential/
controller/
160 lines
customfield/
18 lines
18 lines
edge/
58 lines
storage/
29 lines
view/
31 lines
diffusion/
view/
167 lines
infrastructure/
diff/
view/
171 lines
view/
phui/
19 lines

Diff 38998

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/differential/controller/DifferentialRevisionViewController.php

Loading...
View Options

src/applications/differential/customfield/DifferentialChildRevisionsField.php

Loading...
View Options

src/applications/differential/customfield/DifferentialParentRevisionsField.php

Loading...
View Options

src/applications/differential/edge/DifferentialStackGraph.php

Loading...
View Options

src/applications/differential/storage/DifferentialRevision.php

Loading...
View Options

src/applications/differential/view/DifferentialRevisionListView.php

Loading...
View Options

src/applications/diffusion/view/DiffusionHistoryTableView.php

Loading...
View Options

src/infrastructure/diff/view/PHUIDiffGraphView.php

Loading...
View Options

src/view/phui/PHUITabView.php

Loading...
Phabricator · Built by Phacility