Remove dependency weak-key. Use path key to forcefully rerender when …

…changed instead (#13)
5monkeys · Feb 3, 2022 · 1a76967 · 1a76967
1 parent 9f1266c
commit 1a76967
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 44 deletions.
diff --git a/packages/djedi-json/package-lock.json b/packages/djedi-json/package-lock.json
diff --git a/packages/djedi-json/package.json b/packages/djedi-json/package.json
@@ -24,8 +24,7 @@
   },
   "dependencies": {
     "classnames": "^2.3.1",
-    "lodash-es": "^4.17.21",
-    "weak-key": "^1.0.2"
+    "lodash-es": "^4.17.21"
   },
   "peerDependencies": {
     "react": ">=16.8.0",

diff --git a/packages/djedi-json/src/core/Renderer/EditorTree.tsx b/packages/djedi-json/src/core/Renderer/EditorTree.tsx
@@ -1,5 +1,4 @@
 import React from 'react';
-import key from 'weak-key';
 
 import { useCMS } from '../../contexts/cms';
 import Editable from '../Editable';
@@ -22,21 +21,16 @@ const EditorTree: React.FC<{ tree: NodeTreeItem; path?: string[] }> = ({ tree, p
   if (!Config) {
     return null;
   }
-  // const [childKey] = Object.entries(Config.content).find(([, t]) => t.type == 'input/children');
 
   const { children } = tree.content;
-  // const localPath = [...path, index.toString(), 'content', 'children'];
 
   return (
     <Editable config={Config} tree={tree} path={path}>
       {Array.isArray(children) &&
-        children?.map((child, i) => (
-          <EditorTree
-            tree={child}
-            key={key(child)}
-            path={[...path, 'content', 'children', i.toString()]}
-          />
-        ))}
+        children?.map((child, i) => {
+          const k = [...path, 'content', 'children', i.toString()];
+          return <EditorTree tree={child} key={k.join('.')} path={k} />;
+        })}
     </Editable>
   );