From 9acdc0bc2e58b3b084b756a78f0a1d7991fa752a Mon Sep 17 00:00:00 2001 From: Knut Sveidqvist Date: Wed, 14 Sep 2022 10:58:59 +0200 Subject: [PATCH] Cleanup fixing som lingering issues --- src/diagrams/c4/c4Renderer.js | 1 - src/diagrams/class/classRenderer.js | 2 -- src/diagrams/mindmap/mindmapRenderer.js | 4 +--- 3 files changed, 1 insertion(+), 6 deletions(-) diff --git a/src/diagrams/c4/c4Renderer.js b/src/diagrams/c4/c4Renderer.js index c3402d0b9..dceca2887 100644 --- a/src/diagrams/c4/c4Renderer.js +++ b/src/diagrams/c4/c4Renderer.js @@ -616,7 +616,6 @@ export const draw = function (_text, id, _version, diagObj) { globalBoundaryMaxY = conf.diagramMarginY; const title = diagObj.db.getTitle(); - const c4type = diagObj.db.getC4Type(); // TODO: @knsv: remove this? let currentBoundarys = diagObj.db.getBoundarys(''); // switch (c4type) { // case 'C4Context': diff --git a/src/diagrams/class/classRenderer.js b/src/diagrams/class/classRenderer.js index 612a7d879..c1236afea 100644 --- a/src/diagrams/class/classRenderer.js +++ b/src/diagrams/class/classRenderer.js @@ -157,8 +157,6 @@ export const draw = function (text, id, _version, diagObj) { securityLevel === 'sandbox' ? select(sandboxElement.nodes()[0].contentDocument.body) : select('body'); - // TODO: @knsv doc is not used, bug? - const doc = securityLevel === 'sandbox' ? sandboxElement.nodes()[0].contentDocument : document; // Fetch the default direction, use TD if none was found const diagram = root.select(`[id='${id}']`); diff --git a/src/diagrams/mindmap/mindmapRenderer.js b/src/diagrams/mindmap/mindmapRenderer.js index 37cce58bf..1519dc406 100644 --- a/src/diagrams/mindmap/mindmapRenderer.js +++ b/src/diagrams/mindmap/mindmapRenderer.js @@ -192,9 +192,7 @@ function layoutMindmap(node, conf) { }); // Merge the trees into a single tree - const result = mergeTrees(node, trees); - // TODO: @knsv The function is not called bug? - eachNode; + mergeTrees(node, trees); return node; } /**