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
F19816700: D16816.id40494.diff
Thu, Mar 5, 7:51 AM
F19816696: D16816.id40494.diff
Thu, Mar 5, 7:50 AM
F19811851: D16816.diff
Wed, Mar 4, 2:45 AM
F19554532: D16816.diff
Jan 29 2026, 11:09 PM
F19554058: D16816.id.diff
Jan 29 2026, 5:04 PM
F19554053: D16816.id.diff
Jan 29 2026, 5:03 PM
F19511800: D16816.diff
Jan 12 2026, 7:54 AM
F19511799: D16816.diff
Jan 12 2026, 7:54 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.