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
Unknown Object (File)
Mon, Apr 8, 9:22 AM
Unknown Object (File)
Tue, Apr 2, 9:11 AM
Unknown Object (File)
Mon, Apr 1, 6:00 AM
Unknown Object (File)
Fri, Mar 29, 6:41 AM
Unknown Object (File)
Mar 10 2024, 1:21 AM
Unknown Object (File)
Feb 18 2024, 10:51 AM
Unknown Object (File)
Feb 18 2024, 10:51 AM
Unknown Object (File)
Feb 18 2024, 10:50 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.