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
F18034835: D16210.id38998.diff
Sun, Aug 3, 1:19 AM
F17968130: D16210.id38987.diff
Fri, Aug 1, 1:05 PM
F17967644: D16210.id38998.diff
Fri, Aug 1, 12:34 PM
F17967351: D16210.id38993.diff
Fri, Aug 1, 12:22 PM
F17949960: D16210.id.diff
Fri, Aug 1, 12:02 AM
F17803400: D16210.id.diff
Fri, Jul 25, 10:09 AM
F17792575: D16210.id38987.diff
Thu, Jul 24, 10:51 PM
F17762791: D16210.id38993.diff
Tue, Jul 22, 10:35 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