@yuneta/gobj-ui 7.4.4 → 7.4.5

package/dist/gobj-ui.cjs.js

@@ -55708,7 +55708,6 @@ function refresh_minimap(gobj) {
 		return;
 	}
 	if (graph_get_plugin(gobj, "minimap")) return;
-	(0, _yuneta_gobj_js.log_warning)(`${(0, _yuneta_gobj_js.gobj_short_name)(gobj)}: minimap: adding for ${n} nodes (threshold ${threshold})`);
 	graph_add_plugin(gobj, "minimap", {
 		size: [200, 140],
 		position: "bottom-left",
@@ -55732,12 +55731,6 @@ function refresh_minimap(gobj) {
 			}
 		}
 	});
-	try {
-		graph.draw();
-	} catch (e) {
-		(0, _yuneta_gobj_js.log_error)(`${(0, _yuneta_gobj_js.gobj_short_name)(gobj)}: cannot draw the minimap: ${e}`);
-	}
-	(0, _yuneta_gobj_js.log_warning)(`${(0, _yuneta_gobj_js.gobj_short_name)(gobj)}: minimap: plugin is ${graph_get_plugin(gobj, "minimap") ? "up" : "MISSING"}`);
 }
 /************************************************************
 *  The innerHTML of ONE node, in the given theme and highlight state.
@@ -56530,30 +56523,32 @@ function ac_load_data(gobj, event, kw, src) {
 			break;
 		}
 	}
-	if (do_links && priv.graph) graph_draw(gobj).then(() => {
-		create_links(gobj);
+	if (do_links && priv.graph) {
+		try {
+			refresh_minimap(gobj);
+		} catch (e) {
+			(0, _yuneta_gobj_js.log_error)(`${(0, _yuneta_gobj_js.gobj_short_name)(gobj)}: refresh_minimap failed: ${e}`);
+		}
 		graph_draw(gobj).then(() => {
-			graph_layout(gobj).then(() => {
-				if (priv.edit_mode) graph_add_plugin(gobj, "history");
-				else graph_remove_plugin(gobj, "history");
-				try {
-					refresh_minimap(gobj);
-				} catch (e) {
-					(0, _yuneta_gobj_js.log_error)(`${(0, _yuneta_gobj_js.gobj_short_name)(gobj)}: refresh_minimap failed: ${e}`);
-				}
-				if (priv._pending_focus_topic !== null) {
-					let ft = priv._pending_focus_topic;
-					priv._pending_focus_topic = null;
-					graph_focus_topic(gobj, ft);
-				}
-				if (priv._pending_find !== null) {
-					let term = priv._pending_find;
-					priv._pending_find = null;
-					publish_find_result(gobj, term, graph_find_nodes(gobj, term));
-				}
+			create_links(gobj);
+			graph_draw(gobj).then(() => {
+				graph_layout(gobj).then(() => {
+					if (priv.edit_mode) graph_add_plugin(gobj, "history");
+					else graph_remove_plugin(gobj, "history");
+					if (priv._pending_focus_topic !== null) {
+						let ft = priv._pending_focus_topic;
+						priv._pending_focus_topic = null;
+						graph_focus_topic(gobj, ft);
+					}
+					if (priv._pending_find !== null) {
+						let term = priv._pending_find;
+						priv._pending_find = null;
+						publish_find_result(gobj, term, graph_find_nodes(gobj, term));
+					}
+				});
 			});
 		});
-	});
+	}
 	return 0;
 }
 /************************************************************

package/dist/gobj-ui.es.js

@@ -55695,7 +55695,6 @@ function refresh_minimap(gobj) {
 		return;
 	}
 	if (graph_get_plugin(gobj, "minimap")) return;
-	log_warning(`${gobj_short_name(gobj)}: minimap: adding for ${n} nodes (threshold ${threshold})`);
 	graph_add_plugin(gobj, "minimap", {
 		size: [200, 140],
 		position: "bottom-left",
@@ -55719,12 +55718,6 @@ function refresh_minimap(gobj) {
 			}
 		}
 	});
-	try {
-		graph.draw();
-	} catch (e) {
-		log_error(`${gobj_short_name(gobj)}: cannot draw the minimap: ${e}`);
-	}
-	log_warning(`${gobj_short_name(gobj)}: minimap: plugin is ${graph_get_plugin(gobj, "minimap") ? "up" : "MISSING"}`);
 }
 /************************************************************
 *  The innerHTML of ONE node, in the given theme and highlight state.
@@ -56517,30 +56510,32 @@ function ac_load_data(gobj, event, kw, src) {
 			break;
 		}
 	}
-	if (do_links && priv.graph) graph_draw(gobj).then(() => {
-		create_links(gobj);
+	if (do_links && priv.graph) {
+		try {
+			refresh_minimap(gobj);
+		} catch (e) {
+			log_error(`${gobj_short_name(gobj)}: refresh_minimap failed: ${e}`);
+		}
 		graph_draw(gobj).then(() => {
-			graph_layout(gobj).then(() => {
-				if (priv.edit_mode) graph_add_plugin(gobj, "history");
-				else graph_remove_plugin(gobj, "history");
-				try {
-					refresh_minimap(gobj);
-				} catch (e) {
-					log_error(`${gobj_short_name(gobj)}: refresh_minimap failed: ${e}`);
-				}
-				if (priv._pending_focus_topic !== null) {
-					let ft = priv._pending_focus_topic;
-					priv._pending_focus_topic = null;
-					graph_focus_topic(gobj, ft);
-				}
-				if (priv._pending_find !== null) {
-					let term = priv._pending_find;
-					priv._pending_find = null;
-					publish_find_result(gobj, term, graph_find_nodes(gobj, term));
-				}
+			create_links(gobj);
+			graph_draw(gobj).then(() => {
+				graph_layout(gobj).then(() => {
+					if (priv.edit_mode) graph_add_plugin(gobj, "history");
+					else graph_remove_plugin(gobj, "history");
+					if (priv._pending_focus_topic !== null) {
+						let ft = priv._pending_focus_topic;
+						priv._pending_focus_topic = null;
+						graph_focus_topic(gobj, ft);
+					}
+					if (priv._pending_find !== null) {
+						let term = priv._pending_find;
+						priv._pending_find = null;
+						publish_find_result(gobj, term, graph_find_nodes(gobj, term));
+					}
+				});
 			});
 		});
-	});
+	}
 	return 0;
 }
 /************************************************************

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yuneta/gobj-ui",
-  "version": "7.4.4",
+  "version": "7.4.5",
   "type": "module",
   "main": "dist/gobj-ui.cjs.js",
   "module": "dist/gobj-ui.es.js",

package/src/c_g6_nodes_tree.js

@@ -4524,8 +4524,6 @@ function refresh_minimap(gobj)
         return;         /*  already up  */
     }
 
-    log_warning(`${gobj_short_name(gobj)}: minimap: adding for ${n} nodes ` +
-                `(threshold ${threshold})`);
     graph_add_plugin(
         gobj,
         "minimap",
@@ -4558,18 +4556,6 @@ function refresh_minimap(gobj)
             }
         }
     );
-
-    /*  The plugin builds its canvas on its FIRST render, and it renders off
-     *  the graph's draw events — which have all already happened by the time
-     *  the node count is known and this runs. Without a draw of its own it
-     *  sits there having never painted, with no container in the DOM at all.  */
-    try {
-        graph.draw();
-    } catch(e) {
-        log_error(`${gobj_short_name(gobj)}: cannot draw the minimap: ${e}`);
-    }
-    log_warning(`${gobj_short_name(gobj)}: minimap: plugin is ` +
-                `${graph_get_plugin(gobj, "minimap")? "up" : "MISSING"}`);
 }
 
 /************************************************************
@@ -5768,6 +5754,19 @@ function ac_load_data(gobj, event, kw, src)
     }
 
     if(do_links && priv.graph) {
+        /*  Decided BEFORE the drawing, not after it.
+         *
+         *  The plugin builds its canvas on its FIRST render, and it renders
+         *  off the graph's draw events — so a minimap added after the last
+         *  draw is instantiated, bound, and never painted: no container in
+         *  the DOM at all, and nothing anywhere says so. Added here it rides
+         *  the two draws and the layout that follow. The node count it needs
+         *  is already known: the records were turned into nodes above.  */
+        try {
+            refresh_minimap(gobj);
+        } catch(e) {
+            log_error(`${gobj_short_name(gobj)}: refresh_minimap failed: ${e}`);
+        }
         graph_draw(gobj).then(() => { // draw nodes, else the link fails
             create_links(gobj);
             graph_draw(gobj).then(() => {
@@ -5777,18 +5776,6 @@ function ac_load_data(gobj, event, kw, src)
                     } else {
                         graph_remove_plugin(gobj, "history");
                     }
-                    /*  The graph knows its size now: a minimap appears only
-                     *  when there is something to be lost in.
-                     *
-                     *  Guarded because this runs inside a .then(): an
-                     *  exception here becomes an unhandled rejection, which
-                     *  is not a console error and not a pageerror — it is
-                     *  nothing at all, and the feature just fails to appear. */
-                    try {
-                        refresh_minimap(gobj);
-                    } catch(e) {
-                        log_error(`${gobj_short_name(gobj)}: refresh_minimap failed: ${e}`);
-                    }
                     /*  Nodes exist and are positioned now: apply a focus
                      *  requested before the data was ready (deep link). */
                     if(priv._pending_focus_topic !== null) {