fix: fixing the child length (#1038)

anandtiwary · jake-bassett · web-flow · commit f2452c672bc5 · 2021-07-29T22:41:55.000-07:00
* fix: fixing the child length

* refactor: fixing formatting

Co-authored-by: Jake &lt;45181984+jake-bassett@users.noreply.github.com&gt;
diff --git a/projects/observability/src/shared/components/topology/d3/layouts/custom-tree-layout.ts b/projects/observability/src/shared/components/topology/d3/layouts/custom-tree-layout.ts
@@ -9,7 +9,7 @@ export class CustomTreeLayout extends TreeLayout {
     const nodeWidth = this.getNodeWidth(rootHierarchyNode);
     const nodeHeight = this.getNodeHeight(rootHierarchyNode);
 
-    this.updateLayout(rootHierarchyNode, 0, -1, nodeWidth * 1, nodeHeight);
+    this.updateLayout(rootHierarchyNode, 0, -1, nodeWidth * 1.2, nodeHeight * 1.2);
   }
 
   private updateLayout(
@@ -28,6 +28,6 @@ export class CustomTreeLayout extends TreeLayout {
       this.updateLayout(node, nodeRowIndex + index, nodeColumnIndex + 1, cellWidth, cellHeight);
     });
 
-    return (hierarchyNode.children?.length ?? 0) + 1;
+    return hierarchyNode.children?.length ?? 1;
   }
 }

Original file line number	Diff line number	Diff line change
`@@ -9,7 +9,7 @@ export class CustomTreeLayout extends TreeLayout {`
`9`	`9`	`const nodeWidth = this.getNodeWidth(rootHierarchyNode);`
`10`	`10`	`const nodeHeight = this.getNodeHeight(rootHierarchyNode);`
`11`	`11`
`12`		`- this.updateLayout(rootHierarchyNode, 0, -1, nodeWidth * 1, nodeHeight);`
	`12`	`+ this.updateLayout(rootHierarchyNode, 0, -1, nodeWidth * 1.2, nodeHeight * 1.2);`
`13`	`13`	`}`
`14`	`14`
`15`	`15`	`private updateLayout(`
`@@ -28,6 +28,6 @@ export class CustomTreeLayout extends TreeLayout {`
`28`	`28`	`this.updateLayout(node, nodeRowIndex + index, nodeColumnIndex + 1, cellWidth, cellHeight);`
`29`	`29`	`});`
`30`	`30`
`31`		`- return (hierarchyNode.children?.length ?? 0) + 1;`
	`31`	`+ return hierarchyNode.children?.length ?? 1;`
`32`	`32`	`}`
`33`	`33`	`}`