Page MenuHomePhabricator

In Maniphest, don't render the task graph drawing if we're only showing parents/children
ClosedPublic

Authored by epriestley on Nov 7 2016, 7:08 PM.
Tags
None
Referenced Files
F19511800: D16816.diff
Mon, Jan 12, 7:54 AM
F19511799: D16816.diff
Mon, Jan 12, 7:54 AM
F19511797: D16816.diff
Mon, Jan 12, 7:52 AM
F19511773: D16816.diff
Mon, Jan 12, 7:31 AM
F19511757: D16816.diff
Mon, Jan 12, 7:18 AM
F19371289: D16816.id40494.diff
Fri, Dec 26, 4:32 PM
F18968679: D16816.diff
Nov 15 2025, 3:05 AM
F18763988: D16816.id.diff
Oct 7 2025, 3:30 AM
Subscribers
None

Details

Summary

Ref T4788. I thought I implemented this, but actualy didn't.

When we're in the "mid-sized" fallback mode (graph has more than 100 nodes, but not more than than 100 parents/children), don't actually draw the graph. It's almost always uninteresting and huge.

Instead, this just renders a list of direct parents, then the task, then the direct children, which is pretty straightforward.

Test Plan

Set limit to 5, saw mid-sized fallback graph with no actual graph drawing.

Diff Detail

Repository
rP Phabricator
Branch
graph1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 14385
Build 18733: Run Core Tests
Build 18732: arc lint + arc unit

Event Timeline

epriestley retitled this revision from to In Maniphest, don't render the task graph drawing if we're only showing parents/children.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Nov 7 2016, 7:09 PM
This revision was automatically updated to reflect the committed changes.