[fix] Add comments and nitpicks

Author: dee077

src/js/netjsongraph.render.js

@@ -93,7 +93,6 @@ class NetJSONGraphRender {
       "click",
       (params) => {
         const clickElement = configs.onClickElement.bind(self);
-        self.utils.setUrlFragments(self, params);
         if (params.componentSubType === "graph") {
           return clickElement(
             params.dataType === "edge" ? "link" : "node",
@@ -103,6 +102,7 @@ class NetJSONGraphRender {
         if (params.componentSubType === "graphGL") {
           return clickElement("node", params.data);
         }
+        self.utils.setUrlFragments(self, params);
         return params.componentSubType === "lines"
           ? clickElement("link", params.data.link)
           : !params.data.cluster && clickElement("node", params.data.node);
@@ -152,6 +152,7 @@ class NetJSONGraphRender {
       // Preserve original NetJSON node for sidebar use
       /* eslint-disable no-underscore-dangle */
       nodeResult._source = JSON.parse(JSON.stringify(node));
+      // Store the clicked node in this.selectedNode for easy access later without need for traverse
       self.utils.setSelectedNodeFromUrlFragments(self, fragments, node);
       return nodeResult;
     });
@@ -292,6 +293,7 @@ class NetJSONGraphRender {
           });
         }
       }
+      // Store the clicked node in this.selectedNode for easy access later without need for traverse
       self.utils.setSelectedNodeFromUrlFragments(self, fragments, node);
     });
     links.forEach((link) => {

src/js/netjsongraph.util.js

@@ -1207,6 +1207,12 @@ class NetJSONGraphUtil {
     return fragments;
   }
 
+  generateUrlFragments(fragments) {
+    return Object.values(fragments)
+      .map((urlParams) => urlParams.toString())
+      .join(";");
+  }
+
   setUrlFragments(self, params) {
     if (!self.config.urlFragments.show) return;
     const fragments = this.parseUrlFragments();
@@ -1227,11 +1233,7 @@ class NetJSONGraphUtil {
     if (zoom != undefined) {
       fragments[id].set("zoom", zoom);
     }
-    const newHash = Object.values(fragments)
-      .map((urlParams) => urlParams.toString())
-      .join(";");
-
-    window.location.hash = newHash;
+    window.location.hash = this.generateUrlFragments(fragments);
   }
 
   removeUrlFragment(self) {
@@ -1242,11 +1244,7 @@ class NetJSONGraphUtil {
     if (fragments[id]) {
       delete fragments[id];
     }
-    const newHash = Object.values(fragments)
-      .map((urlParams) => urlParams.toString())
-      .join(";");
-
-    window.location.hash = newHash;
+    window.location.hash = this.generateUrlFragments(fragments);
   }
 
   setSelectedNodeFromUrlFragments(self, fragments, node) {