@yuneta/gobj-ui 7.25.16 → 7.25.17

package/README.md

@@ -1824,6 +1824,20 @@ gobj_send_event(engine, "EV_GRAPHS_WRITE_REFUSED", {topic: "users"}, host);  //
 gobj_send_event(engine, "EV_SAVE_GRAPH", {}, engine);  // writes users; Save dark
 ```
 
+**The refusal reaches the engine through the real transports** (since
+`7.25.17`). The answer of a command carries back only the request's
+`__md_command__`, as its `command_stack` frame -- `C_IEVENT_CLI` and gui_agent's
+`C_AGENT_TREEDB_LINK` both do, and neither echoes the request's `record`. The
+host read the topic from that `record`, found none, logged *"a refused
+__graphs__ write names no topic"* and never told the engine: the 7.25.15 fix
+worked only in a test whose fake transport echoed the whole request. The host
+now puts the topic in the echo it asks for:
+
+```js
+kw.__md_command__ = {topic_name: "__graphs__", graph_topic: "users"};
+// the answer: __md_iev__.command_stack[0].kw === that echo, and nothing else
+```
+
 **Every saved look has a way back, in the context menu** (since `7.23.134`,
 replacing `reset sizes` / `reset topic sizes`). In edition, the node, the port
 and the edge menus each offer three resets — this one, its kind (`reset topic
@@ -2259,6 +2273,39 @@ else.
   gobj_send_event(editor, "EV_REFRESH", {}, host);             // logged, waits
   // ... both deletes answered -> ST_LOADING -> the model is read again
   ```
+- **A write in flight takes the body out of the keyboard's reach too**
+  (7.25.17). `ST_SAVING` keeps the screen under it and marks the body busy.
+  The busy class only stopped the pointer (`pointer-events: none`); the
+  shell's focus trap put the focus back on the row control that had it, and
+  Enter or Delete there sent `EV_DELETE_COLUMN` or `EV_SELECT_TOPIC`, which
+  `ST_SAVING` does not declare (*"Event NOT DEFINED"*). The busy body is now
+  also `inert`. `ST_LOADING` needs nothing: its body is emptied.
+
+  ```js
+  // in ST_SAVING
+  $container.querySelector(".SCHEMA_BODY").hasAttribute("inert");   // true
+  ```
+- **A write marks the topic of its RECORD, not the topic on screen**
+  (7.25.17). What "its version still has to move" is asked about is the set of
+  topics this session wrote (the draft chip, the export warning,
+  `validate_schema()`'s `written_topics`). A column write marked the topic on
+  screen, so an import from the topics screen (no topic on it) marked nothing,
+  and an import that added `roles` from the screen of `users` marked `users`.
+  The mark is read from the record the store answered (a new topic has its id
+  only there), then from the record queued (a column names its topic in its
+  fkey), and only then from the screen.
+
+  ```js
+  // import adding topic `roles`, run from db/users, every write answered
+  editor.priv.written;   // {"db.roles": true}   (was {"db.users": true})
+  ```
+- **The reason a load failed is a key** (7.25.17). The notice's detail was
+  the text of `t()` taken when the load failed (`cannot reach the treedb`,
+  `cannot load the schemas`) or the backend's words, shown as plain text: it
+  did not change language, and with no comment it repeated the notice's own
+  title. It is now kept as a key, written with `data-i18n` (the backend's
+  words translate to themselves when no locale has them), and left out when
+  it is the title.
 - **The loading screen is drawn** (7.25.8): the body is cleared, the drawing
   destroyed and every toolbar button disabled, as in `ST_SAVING`, because
   `ST_LOADING` declares none of a screen's actions. A **dialog** left open is

package/dist/gobj-ui.cjs.js

@@ -68378,6 +68378,7 @@ function treedb_update_node(gobj, treedb_name, topic_name, record, options) {
 		options: options || {}
 	};
 	kw.__md_command__ = { topic_name };
+	if (topic_name === "__graphs__") kw.__md_command__.graph_topic = graph_topic_of(record);
 	let ret = (0, _yuneta_gobj_js.gobj_command)(priv.gobj_remote_yuno, command, kw, gobj);
 	if (ret) {
 		(0, _yuneta_gobj_js.log_error)(ret);
@@ -68656,7 +68657,7 @@ function ac_mt_command_answer$1(gobj, event, kw, src) {
 		case "delete-node":
 		case "link-nodes":
 		case "unlink-nodes":
-			if (result < 0 && command === "update-node") graphs_write_refused(gobj, (0, _yuneta_gobj_js.kw_get_str)(gobj, kw_command, "topic_name", "", 0), (0, _yuneta_gobj_js.kw_get_dict)(gobj, kw_command, "record", {}, 0));
+			if (result < 0 && command === "update-node") graphs_write_refused(gobj, (0, _yuneta_gobj_js.kw_get_str)(gobj, kw_command, "topic_name", "", 0), (0, _yuneta_gobj_js.kw_get_str)(gobj, kw_command, "graph_topic", "", 0));
 			if (result >= 0 && (0, _yuneta_gobj_js.kw_get_str)(gobj, kw_command, "topic_name", "", 0) !== "__graphs__") (0, _yuneta_gobj_js.gobj_publish_event)(gobj, "EV_RECORD_WRITTEN", {
 				treedb_name: (0, _yuneta_gobj_js.kw_get_str)(gobj, kw_command, "treedb_name", "", 0),
 				topic_name: (0, _yuneta_gobj_js.kw_get_str)(gobj, kw_command, "topic_name", "", 0),
@@ -68905,29 +68906,35 @@ function ac_update_node(gobj, event, kw, src) {
 	let record = kw.record;
 	let options = kw.options || {};
 	if (refuse_if_readonly$1(gobj, event)) {
-		graphs_write_refused(gobj, topic_name, record);
+		graphs_write_refused(gobj, topic_name, graph_topic_of(record));
 		return -1;
 	}
 	if (topic_name === "__graphs__" && !is_connected(gobj)) {
 		(0, _yuneta_gobj_js.log_warning)(`${(0, _yuneta_gobj_js.gobj_short_name)(gobj)}: no session, the arrangement was not sent`);
-		graphs_write_refused(gobj, topic_name, record);
+		graphs_write_refused(gobj, topic_name, graph_topic_of(record));
 		return -1;
 	}
 	let ret = treedb_update_node(gobj, treedb_name, topic_name, record, options);
-	if (ret < 0) graphs_write_refused(gobj, topic_name, record);
+	if (ret < 0) graphs_write_refused(gobj, topic_name, graph_topic_of(record));
 	return ret;
 }
 /************************************************************
+*  The topic a `__graphs__` record arranges.
+************************************************************/
+function graph_topic_of(record) {
+	if (!record || typeof record !== "object") return "";
+	return record.topic || record.id || "";
+}
+/************************************************************
 *  A write of `__graphs__` did not land. The engine
 *  (C_G6_NODES_TREE) took it for granted when it left -- nothing
 *  answers it when it lands -- so it is told, and its next Save
 *  writes the topic again. Other topics are the operator's data:
 *  the answer's error says what happened to them.
 ************************************************************/
-function graphs_write_refused(gobj, topic_name, record) {
+function graphs_write_refused(gobj, topic_name, topic) {
 	let priv = gobj.priv;
 	if (topic_name !== "__graphs__") return;
-	let topic = record && typeof record === "object" ? record.topic || record.id : "";
 	if (!topic) {
 		(0, _yuneta_gobj_js.log_error)(`${(0, _yuneta_gobj_js.gobj_short_name)(gobj)}: a refused __graphs__ write names no topic`);
 		return;
@@ -72486,11 +72493,18 @@ function el_attrs(attrs) {
 	return attrs;
 }
 /***************************************************************
-*  The body stops answering while a write is in flight.
+*  The body stops answering while a write is in flight. For the
+*  keyboard too: `pointer-events` stops the mouse alone, and the
+*  shell's focus trap puts the focus back on the row control that
+*  had it, where Enter or Delete sent an action ST_SAVING does
+*  not declare. `inert` takes the whole body out of reach.
 ***************************************************************/
 function set_busy(gobj, busy) {
 	let $body = $of(gobj, ".SCHEMA_BODY");
-	if ($body) $body.classList.toggle("SCHEMA_BUSY", !!busy);
+	if ($body) {
+		$body.classList.toggle("SCHEMA_BUSY", !!busy);
+		$body.toggleAttribute("inert", !!busy);
+	}
 }
 /***************************************************************
 *  Shorthands for the three panes of the frame.
@@ -72706,7 +72720,7 @@ function request_model(gobj, keep_written) {
 		priv.records_before = null;
 		priv.load_round++;
 		priv.reload_on_open = true;
-		priv.load_error = (0, i18next.t)("cannot reach the treedb");
+		priv.load_error = "cannot reach the treedb";
 		if (priv.model) yui_shell_show_error(yui_shell_of(gobj), "cannot reach the treedb", { t: i18next.t });
 		end_load(gobj);
 		return -1;
@@ -72917,10 +72931,13 @@ function show_notice(gobj, key, detail) {
 		},
 		(0, i18next.t)(key)
 	]));
-	if (detail) $notice.appendChild((0, _yuneta_gobj_js.createElement2)([
+	if (detail && detail !== key) $notice.appendChild((0, _yuneta_gobj_js.createElement2)([
 		"span",
-		{ class: "SCHEMA_NOTICE_DETAIL yui-text-quiet ml-2" },
-		`${detail}`
+		{
+			class: "SCHEMA_NOTICE_DETAIL yui-text-quiet ml-2",
+			i18n: `${detail}`
+		},
+		(0, i18next.t)(`${detail}`)
 	]));
 }
 /***************************************************************
@@ -75475,7 +75492,7 @@ function ac_mt_command_answer(gobj, event, kw, src) {
 			return 0;
 		}
 		priv.pending--;
-		if (result < 0) priv.load_error = comment || (0, i18next.t)("cannot load the schemas");
+		if (result < 0) priv.load_error = comment || "cannot load the schemas";
 		else if (priv.records && Array.isArray(priv.records[topic_name])) priv.records[topic_name] = Array.isArray(data) ? data : [];
 		if (priv.pending > 0) return 0;
 		if (priv.load_error) {
@@ -75511,9 +75528,10 @@ function ac_mt_command_answer(gobj, event, kw, src) {
 		}
 		if (result < 0) return end_writes(gobj, comment || (0, i18next.t)("the treedb refused the write"));
 		priv.save_wrote = true;
+		let record = null;
 		if (command === "delete-node") forget_record(gobj, topic_name, (0, _yuneta_gobj_js.kw_get_str)(gobj, md, "record_id", "", 0));
 		else {
-			let record = (0, _yuneta_gobj_js.is_object)(data) ? data : Array.isArray(data) && (0, _yuneta_gobj_js.is_object)(data[0]) ? data[0] : null;
+			record = (0, _yuneta_gobj_js.is_object)(data) ? data : Array.isArray(data) && (0, _yuneta_gobj_js.is_object)(data[0]) ? data[0] : null;
 			if (record) patch_record(gobj, topic_name, record);
 			else {
 				let not_sent = priv.save_queue.length - 1;
@@ -75521,7 +75539,7 @@ function ac_mt_command_answer(gobj, event, kw, src) {
 				if (not_sent > 0) yui_shell_show_error(yui_shell_of(gobj), "the treedb did not describe a write back: the writes after it were not sent", { t: i18next.t });
 				if (topic_name === T_TOPICS || topic_name === T_COLS) {
 					let write = priv.save_queue[0];
-					mark_written(gobj, topic_name, write ? write.record : null);
+					mark_written(gobj, topic_name, null, write ? write.record : null);
 				}
 				priv.reload_after_write = false;
 				priv.save_queue = [];
@@ -75536,7 +75554,7 @@ function ac_mt_command_answer(gobj, event, kw, src) {
 		}
 		if (topic_name === T_TOPICS || topic_name === T_COLS) {
 			let write = priv.save_queue[0];
-			mark_written(gobj, topic_name, write ? write.record : null);
+			mark_written(gobj, topic_name, record, write ? write.record : null);
 		}
 		priv.save_queue.shift();
 		return run_next_write(gobj);
@@ -75593,25 +75611,34 @@ function late_write_done(gobj, command, topic_name, data) {
 ***************************************************************/
 function mark_written_record(gobj, topic_name, record) {
 	let priv = gobj.priv;
-	if (!record) return;
+	let marked = 0;
+	if (!record) return 0;
 	if (topic_name === T_TOPICS && typeof record.id === "string" && record.id) {
 		priv.written[record.id] = true;
-		return;
+		return 1;
+	}
+	if (topic_name !== T_COLS) return 0;
+	for (let ref of parse_fkey_ref(record.topics)) {
+		priv.written[ref.id] = true;
+		marked++;
 	}
-	if (topic_name !== T_COLS) return;
-	for (let ref of parse_fkey_ref(record.topics)) priv.written[ref.id] = true;
+	return marked;
 }
 /***************************************************************
 *  Remember which topic this write belongs to, which is what
 *  "its version still has to move" is asked about.
+*
+*  Asked of the RECORD first: the one the store answered (a new
+*  topic has its id only there), then the one queued (a column
+*  names its topic in its fkey). The screen is the last resort,
+*  and a poor one: an import writes topics the operator is not
+*  on, and from the topics screen there is no topic on it at all.
 ***************************************************************/
-function mark_written(gobj, topic_name, record) {
+function mark_written(gobj, topic_name, answered, queued) {
 	let priv = gobj.priv;
+	if (mark_written_record(gobj, topic_name, answered) > 0) return;
+	if (mark_written_record(gobj, topic_name, queued) > 0) return;
 	let topic = current_topic(gobj);
-	if (topic_name === T_TOPICS && record && record.id) {
-		priv.written[record.id] = true;
-		return;
-	}
 	if (topic) priv.written[topic.id] = true;
 }
 /***************************************************************

package/dist/gobj-ui.es.js

@@ -68373,6 +68373,7 @@ function treedb_update_node(gobj, treedb_name, topic_name, record, options) {
 		options: options || {}
 	};
 	kw.__md_command__ = { topic_name };
+	if (topic_name === "__graphs__") kw.__md_command__.graph_topic = graph_topic_of(record);
 	let ret = gobj_command(priv.gobj_remote_yuno, command, kw, gobj);
 	if (ret) {
 		log_error(ret);
@@ -68651,7 +68652,7 @@ function ac_mt_command_answer$1(gobj, event, kw, src) {
 		case "delete-node":
 		case "link-nodes":
 		case "unlink-nodes":
-			if (result < 0 && command === "update-node") graphs_write_refused(gobj, kw_get_str(gobj, kw_command, "topic_name", "", 0), kw_get_dict(gobj, kw_command, "record", {}, 0));
+			if (result < 0 && command === "update-node") graphs_write_refused(gobj, kw_get_str(gobj, kw_command, "topic_name", "", 0), kw_get_str(gobj, kw_command, "graph_topic", "", 0));
 			if (result >= 0 && kw_get_str(gobj, kw_command, "topic_name", "", 0) !== "__graphs__") gobj_publish_event(gobj, "EV_RECORD_WRITTEN", {
 				treedb_name: kw_get_str(gobj, kw_command, "treedb_name", "", 0),
 				topic_name: kw_get_str(gobj, kw_command, "topic_name", "", 0),
@@ -68900,29 +68901,35 @@ function ac_update_node(gobj, event, kw, src) {
 	let record = kw.record;
 	let options = kw.options || {};
 	if (refuse_if_readonly$1(gobj, event)) {
-		graphs_write_refused(gobj, topic_name, record);
+		graphs_write_refused(gobj, topic_name, graph_topic_of(record));
 		return -1;
 	}
 	if (topic_name === "__graphs__" && !is_connected(gobj)) {
 		log_warning(`${gobj_short_name(gobj)}: no session, the arrangement was not sent`);
-		graphs_write_refused(gobj, topic_name, record);
+		graphs_write_refused(gobj, topic_name, graph_topic_of(record));
 		return -1;
 	}
 	let ret = treedb_update_node(gobj, treedb_name, topic_name, record, options);
-	if (ret < 0) graphs_write_refused(gobj, topic_name, record);
+	if (ret < 0) graphs_write_refused(gobj, topic_name, graph_topic_of(record));
 	return ret;
 }
 /************************************************************
+*  The topic a `__graphs__` record arranges.
+************************************************************/
+function graph_topic_of(record) {
+	if (!record || typeof record !== "object") return "";
+	return record.topic || record.id || "";
+}
+/************************************************************
 *  A write of `__graphs__` did not land. The engine
 *  (C_G6_NODES_TREE) took it for granted when it left -- nothing
 *  answers it when it lands -- so it is told, and its next Save
 *  writes the topic again. Other topics are the operator's data:
 *  the answer's error says what happened to them.
 ************************************************************/
-function graphs_write_refused(gobj, topic_name, record) {
+function graphs_write_refused(gobj, topic_name, topic) {
 	let priv = gobj.priv;
 	if (topic_name !== "__graphs__") return;
-	let topic = record && typeof record === "object" ? record.topic || record.id : "";
 	if (!topic) {
 		log_error(`${gobj_short_name(gobj)}: a refused __graphs__ write names no topic`);
 		return;
@@ -72481,11 +72488,18 @@ function el_attrs(attrs) {
 	return attrs;
 }
 /***************************************************************
-*  The body stops answering while a write is in flight.
+*  The body stops answering while a write is in flight. For the
+*  keyboard too: `pointer-events` stops the mouse alone, and the
+*  shell's focus trap puts the focus back on the row control that
+*  had it, where Enter or Delete sent an action ST_SAVING does
+*  not declare. `inert` takes the whole body out of reach.
 ***************************************************************/
 function set_busy(gobj, busy) {
 	let $body = $of(gobj, ".SCHEMA_BODY");
-	if ($body) $body.classList.toggle("SCHEMA_BUSY", !!busy);
+	if ($body) {
+		$body.classList.toggle("SCHEMA_BUSY", !!busy);
+		$body.toggleAttribute("inert", !!busy);
+	}
 }
 /***************************************************************
 *  Shorthands for the three panes of the frame.
@@ -72701,7 +72715,7 @@ function request_model(gobj, keep_written) {
 		priv.records_before = null;
 		priv.load_round++;
 		priv.reload_on_open = true;
-		priv.load_error = t("cannot reach the treedb");
+		priv.load_error = "cannot reach the treedb";
 		if (priv.model) yui_shell_show_error(yui_shell_of(gobj), "cannot reach the treedb", { t });
 		end_load(gobj);
 		return -1;
@@ -72912,10 +72926,13 @@ function show_notice(gobj, key, detail) {
 		},
 		t(key)
 	]));
-	if (detail) $notice.appendChild(createElement2([
+	if (detail && detail !== key) $notice.appendChild(createElement2([
 		"span",
-		{ class: "SCHEMA_NOTICE_DETAIL yui-text-quiet ml-2" },
-		`${detail}`
+		{
+			class: "SCHEMA_NOTICE_DETAIL yui-text-quiet ml-2",
+			i18n: `${detail}`
+		},
+		t(`${detail}`)
 	]));
 }
 /***************************************************************
@@ -75470,7 +75487,7 @@ function ac_mt_command_answer(gobj, event, kw, src) {
 			return 0;
 		}
 		priv.pending--;
-		if (result < 0) priv.load_error = comment || t("cannot load the schemas");
+		if (result < 0) priv.load_error = comment || "cannot load the schemas";
 		else if (priv.records && Array.isArray(priv.records[topic_name])) priv.records[topic_name] = Array.isArray(data) ? data : [];
 		if (priv.pending > 0) return 0;
 		if (priv.load_error) {
@@ -75506,9 +75523,10 @@ function ac_mt_command_answer(gobj, event, kw, src) {
 		}
 		if (result < 0) return end_writes(gobj, comment || t("the treedb refused the write"));
 		priv.save_wrote = true;
+		let record = null;
 		if (command === "delete-node") forget_record(gobj, topic_name, kw_get_str(gobj, md, "record_id", "", 0));
 		else {
-			let record = is_object(data) ? data : Array.isArray(data) && is_object(data[0]) ? data[0] : null;
+			record = is_object(data) ? data : Array.isArray(data) && is_object(data[0]) ? data[0] : null;
 			if (record) patch_record(gobj, topic_name, record);
 			else {
 				let not_sent = priv.save_queue.length - 1;
@@ -75516,7 +75534,7 @@ function ac_mt_command_answer(gobj, event, kw, src) {
 				if (not_sent > 0) yui_shell_show_error(yui_shell_of(gobj), "the treedb did not describe a write back: the writes after it were not sent", { t });
 				if (topic_name === T_TOPICS || topic_name === T_COLS) {
 					let write = priv.save_queue[0];
-					mark_written(gobj, topic_name, write ? write.record : null);
+					mark_written(gobj, topic_name, null, write ? write.record : null);
 				}
 				priv.reload_after_write = false;
 				priv.save_queue = [];
@@ -75531,7 +75549,7 @@ function ac_mt_command_answer(gobj, event, kw, src) {
 		}
 		if (topic_name === T_TOPICS || topic_name === T_COLS) {
 			let write = priv.save_queue[0];
-			mark_written(gobj, topic_name, write ? write.record : null);
+			mark_written(gobj, topic_name, record, write ? write.record : null);
 		}
 		priv.save_queue.shift();
 		return run_next_write(gobj);
@@ -75588,25 +75606,34 @@ function late_write_done(gobj, command, topic_name, data) {
 ***************************************************************/
 function mark_written_record(gobj, topic_name, record) {
 	let priv = gobj.priv;
-	if (!record) return;
+	let marked = 0;
+	if (!record) return 0;
 	if (topic_name === T_TOPICS && typeof record.id === "string" && record.id) {
 		priv.written[record.id] = true;
-		return;
+		return 1;
+	}
+	if (topic_name !== T_COLS) return 0;
+	for (let ref of parse_fkey_ref(record.topics)) {
+		priv.written[ref.id] = true;
+		marked++;
 	}
-	if (topic_name !== T_COLS) return;
-	for (let ref of parse_fkey_ref(record.topics)) priv.written[ref.id] = true;
+	return marked;
 }
 /***************************************************************
 *  Remember which topic this write belongs to, which is what
 *  "its version still has to move" is asked about.
+*
+*  Asked of the RECORD first: the one the store answered (a new
+*  topic has its id only there), then the one queued (a column
+*  names its topic in its fkey). The screen is the last resort,
+*  and a poor one: an import writes topics the operator is not
+*  on, and from the topics screen there is no topic on it at all.
 ***************************************************************/
-function mark_written(gobj, topic_name, record) {
+function mark_written(gobj, topic_name, answered, queued) {
 	let priv = gobj.priv;
+	if (mark_written_record(gobj, topic_name, answered) > 0) return;
+	if (mark_written_record(gobj, topic_name, queued) > 0) return;
 	let topic = current_topic(gobj);
-	if (topic_name === T_TOPICS && record && record.id) {
-		priv.written[record.id] = true;
-		return;
-	}
 	if (topic) priv.written[topic.id] = true;
 }
 /***************************************************************

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yuneta/gobj-ui",
-  "version": "7.25.16",
+  "version": "7.25.17",
   "type": "module",
   "main": "dist/gobj-ui.cjs.js",
   "module": "dist/gobj-ui.es.js",
@@ -30,7 +30,7 @@
   "devDependencies": {
     "@rollup/plugin-terser": "^1.0.0",
     "@vitest/coverage-v8": "^4.1.10",
-    "@yuneta/gobj-js": "^7.25.2",
+    "@yuneta/gobj-js": "^7.25.3",
     "vite": "^8.2.1",
     "vitest": "^4.1.10"
   },

package/src/c_yui_schema_editor.js

@@ -403,7 +403,11 @@ function el_attrs(attrs)
 }
 
 /***************************************************************
- *  The body stops answering while a write is in flight.
+ *  The body stops answering while a write is in flight. For the
+ *  keyboard too: `pointer-events` stops the mouse alone, and the
+ *  shell's focus trap puts the focus back on the row control that
+ *  had it, where Enter or Delete sent an action ST_SAVING does
+ *  not declare. `inert` takes the whole body out of reach.
  ***************************************************************/
 function set_busy(gobj, busy)
 {
@@ -411,6 +415,7 @@ function set_busy(gobj, busy)
 
     if($body) {
         $body.classList.toggle("SCHEMA_BUSY", !!busy);
+        $body.toggleAttribute("inert", !!busy);
     }
 }
 
@@ -707,7 +712,7 @@ function request_model(gobj, keep_written)
         priv.records_before = null;
         priv.load_round++;
         priv.reload_on_open = true;
-        priv.load_error = t("cannot reach the treedb");
+        priv.load_error = "cannot reach the treedb";   /*  a key: the notice translates it  */
         if(priv.model) {
             yui_shell_show_error(yui_shell_of(gobj), "cannot reach the treedb", {t: t});
         }
@@ -993,9 +998,14 @@ function show_notice(gobj, key, detail)
     $notice.appendChild(createElement2(
         ["span", {class: "SCHEMA_NOTICE_TEXT", i18n: key}, t(key)]
     ));
-    if(detail) {
+    /*  The detail is a KEY too (ours, or the backend's words, which
+     *  translate to themselves when no locale has them), so a change
+     *  of language changes it with the rest. One that repeats the
+     *  title says nothing.  */
+    if(detail && detail !== key) {
         $notice.appendChild(createElement2(
-            ["span", {class: "SCHEMA_NOTICE_DETAIL yui-text-quiet ml-2"}, `${detail}`]
+            ["span", {class: "SCHEMA_NOTICE_DETAIL yui-text-quiet ml-2", i18n: `${detail}`},
+                t(`${detail}`)]
         ));
     }
 }
@@ -3350,7 +3360,7 @@ function ac_mt_command_answer(gobj, event, kw, src)
         }
         priv.pending--;
         if(result < 0) {
-            priv.load_error = comment || t("cannot load the schemas");
+            priv.load_error = comment || "cannot load the schemas";   /*  a key  */
         } else if(priv.records && Array.isArray(priv.records[topic_name])) {
             priv.records[topic_name] = Array.isArray(data) ? data : [];
         }
@@ -3427,10 +3437,11 @@ function ac_mt_command_answer(gobj, event, kw, src)
             return end_writes(gobj, comment || t("the treedb refused the write"));
         }
         priv.save_wrote = true;
+        let record = null;
         if(command === "delete-node") {
             forget_record(gobj, topic_name, kw_get_str(gobj, md, "record_id", "", 0));
         } else {
-            let record = is_object(data) ? data
+            record = is_object(data) ? data
                 : (Array.isArray(data) && is_object(data[0]) ? data[0] : null);
             if(record) {
                 patch_record(gobj, topic_name, record);
@@ -3453,7 +3464,7 @@ function ac_mt_command_answer(gobj, event, kw, src)
                 }
                 if(topic_name === T_TOPICS || topic_name === T_COLS) {
                     let write = priv.save_queue[0];
-                    mark_written(gobj, topic_name, write ? write.record : null);
+                    mark_written(gobj, topic_name, null, write ? write.record : null);
                 }
                 priv.reload_after_write = false;
                 priv.save_queue = [];
@@ -3471,11 +3482,12 @@ function ac_mt_command_answer(gobj, event, kw, src)
             }
         }
         if(topic_name === T_TOPICS || topic_name === T_COLS) {
-            /*  Which record it was is in the QUEUE, not in the answer: an
-             *  adapter echoes only what it was asked to echo, and the
-             *  record is not small enough to be worth echoing.  */
+            /*  Which record it was: the one the store answered, else the
+             *  one in the QUEUE -- an adapter echoes only what it was
+             *  asked to echo, and the record is not small enough to be
+             *  worth echoing.  */
             let write = priv.save_queue[0];
-            mark_written(gobj, topic_name, write ? write.record : null);
+            mark_written(gobj, topic_name, record, write ? write.record : null);
         }
         priv.save_queue.shift();
         return run_next_write(gobj);
@@ -3544,35 +3556,46 @@ function late_write_done(gobj, command, topic_name, data)
 function mark_written_record(gobj, topic_name, record)
 {
     let priv = gobj.priv;
+    let marked = 0;
 
     if(!record) {
-        return;
+        return 0;
     }
     if(topic_name === T_TOPICS && typeof record.id === "string" && record.id) {
         priv.written[record.id] = true;
-        return;
+        return 1;
     }
     if(topic_name !== T_COLS) {
-        return;
+        return 0;
     }
     for(let ref of parse_fkey_ref(record.topics)) {
         priv.written[ref.id] = true;
+        marked++;
     }
+    return marked;
 }
 
 /***************************************************************
  *  Remember which topic this write belongs to, which is what
  *  "its version still has to move" is asked about.
+ *
+ *  Asked of the RECORD first: the one the store answered (a new
+ *  topic has its id only there), then the one queued (a column
+ *  names its topic in its fkey). The screen is the last resort,
+ *  and a poor one: an import writes topics the operator is not
+ *  on, and from the topics screen there is no topic on it at all.
  ***************************************************************/
-function mark_written(gobj, topic_name, record)
+function mark_written(gobj, topic_name, answered, queued)
 {
     let priv = gobj.priv;
-    let topic = current_topic(gobj);
 
-    if(topic_name === T_TOPICS && record && record.id) {
-        priv.written[record.id] = true;
+    if(mark_written_record(gobj, topic_name, answered) > 0) {
         return;
     }
+    if(mark_written_record(gobj, topic_name, queued) > 0) {
+        return;
+    }
+    let topic = current_topic(gobj);
     if(topic) {
         priv.written[topic.id] = true;
     }

package/src/c_yui_treedb_graph.js

@@ -1520,6 +1520,12 @@ function treedb_update_node(gobj, treedb_name, topic_name, record, options)
     kw.__md_command__ = { // Data to be returned
         topic_name: topic_name,
     };
+    /*  The answer carries this echo and nothing else of the request: a
+     *  refused `__graphs__` write is told to the engine by the topic it
+     *  arranges, so that topic has to travel here.  */
+    if(topic_name === "__graphs__") {
+        kw.__md_command__.graph_topic = graph_topic_of(record);
+    }
 
     let ret = gobj_command(priv.gobj_remote_yuno,
         command,
@@ -2118,7 +2124,7 @@ function ac_mt_command_answer(gobj, event, kw, src)
             if(result < 0 && command === "update-node") {
                 graphs_write_refused(gobj,
                     kw_get_str(gobj, kw_command, "topic_name", "", 0),
-                    kw_get_dict(gobj, kw_command, "record", {}, 0));
+                    kw_get_str(gobj, kw_command, "graph_topic", "", 0));
             }
             if(result >= 0 &&
                 kw_get_str(gobj, kw_command, "topic_name", "", 0) !== "__graphs__") {
@@ -2486,7 +2492,7 @@ function ac_update_node(gobj, event, kw, src)
     let options = kw.options || {};
 
     if(refuse_if_readonly(gobj, event)) {
-        graphs_write_refused(gobj, topic_name, record);
+        graphs_write_refused(gobj, topic_name, graph_topic_of(record));
         return -1;
     }
 
@@ -2496,7 +2502,7 @@ function ac_update_node(gobj, event, kw, src)
     if(topic_name === "__graphs__" && !is_connected(gobj)) {
         log_warning(`${gobj_short_name(gobj)}: no session, the arrangement ` +
             `was not sent`);
-        graphs_write_refused(gobj, topic_name, record);
+        graphs_write_refused(gobj, topic_name, graph_topic_of(record));
         return -1;
     }
 
@@ -2508,11 +2514,22 @@ function ac_update_node(gobj, event, kw, src)
         options
     );
     if(ret < 0) {
-        graphs_write_refused(gobj, topic_name, record);
+        graphs_write_refused(gobj, topic_name, graph_topic_of(record));
     }
     return ret;
 }
 
+/************************************************************
+ *  The topic a `__graphs__` record arranges.
+ ************************************************************/
+function graph_topic_of(record)
+{
+    if(!record || typeof record !== "object") {
+        return "";
+    }
+    return record.topic || record.id || "";
+}
+
 /************************************************************
  *  A write of `__graphs__` did not land. The engine
  *  (C_G6_NODES_TREE) took it for granted when it left -- nothing
@@ -2520,14 +2537,13 @@ function ac_update_node(gobj, event, kw, src)
  *  writes the topic again. Other topics are the operator's data:
  *  the answer's error says what happened to them.
  ************************************************************/
-function graphs_write_refused(gobj, topic_name, record)
+function graphs_write_refused(gobj, topic_name, topic)
 {
     let priv = gobj.priv;
 
     if(topic_name !== "__graphs__") {
         return;
     }
-    let topic = (record && typeof record === "object") ? (record.topic || record.id) : "";
     if(!topic) {
         log_error(`${gobj_short_name(gobj)}: a refused __graphs__ write names no topic`);
         return;

package/src/schema_editor_reload.wiring.test.js

@@ -1501,3 +1501,164 @@ describe("EV_REFRESH while a write is in flight", () => {
         expect(not_defined()).toEqual([]);
     });
 });
+
+describe("the body while a write is in flight", () => {
+
+    /*  pointer-events stops the mouse and nothing else: the shell's
+     *  focus trap puts the focus back on the row control that had it,
+     *  and Delete or Enter there sent an action ST_SAVING does not
+     *  declare. `inert` takes the body out of the keyboard's reach too.  */
+    test("is inert, and answers again when the write ends", () => {
+        const {editor, remote, host} = build("r16k1", "db/users");
+        const $body = $in(editor, ".SCHEMA_BODY");
+        expect($body.hasAttribute("inert")).toBe(false);
+
+        const write = start_a_write(editor, host, "name");
+        expect($body.hasAttribute("inert")).toBe(true);
+
+        answer(editor, remote, write, 0, write.kw.record);
+        expect(gobj_current_state(editor)).toBe("ST_COLUMNS");
+        expect($in(editor, ".SCHEMA_BODY").hasAttribute("inert")).toBe(false);
+        expect(errors()).toEqual([]);
+    });
+
+    test("a drop that ends the write gives the body back too", () => {
+        const {editor, remote, host} = build("r16k2", "db/users");
+        start_a_write(editor, host, "name");
+        drop(editor, remote, host);
+        expect($in(editor, ".SCHEMA_BODY").hasAttribute("inert")).toBe(false);
+    });
+});
+
+describe("the writes of an import mark the topic they write", () => {
+
+    /*  An import that adds a topic, `roles`.  */
+    const IMPORT = JSON.stringify({id: "db", topics: [
+        {id: "users", pkey: "id", cols: [
+            {id: "id", type: "string", flag: ["persistent", "required"]}
+        ]},
+        {id: "roles", pkey: "id", cols: [
+            {id: "id", type: "string", flag: ["persistent", "required"]}
+        ]}
+    ]});
+
+    /*  The record as the store answers it: with the id it gave a new
+     *  one, and its fkeys in list_dict.  */
+    function as_the_store_answers(write)
+    {
+        const record = JSON.parse(JSON.stringify(write.kw.record));
+        const to_dict = (refs) => refs.map((ref) => {
+            const [topic_name, id, hook_name] = ref.split("^");
+            return {topic_name, id, hook_name};
+        });
+        if(write.kw.topic_name === "topics") {
+            record.id = record.id || `db.${record.value}`;
+            record.treedbs = to_dict(record.treedbs || []);
+        }
+        if(write.kw.topic_name === "cols") {
+            record.id = record.id || `${record.topics[0].split("^")[1]}.${record.value}`;
+            record.topics = to_dict(record.topics || []);
+        }
+        return record;
+    }
+
+    function run_the_import(editor, host)
+    {
+        gobj_send_event(editor, "EV_IMPORT", {}, host);
+        gobj_send_event(editor, "EV_PREVIEW_IMPORT", {prune: false, text: IMPORT,
+            model_gen: editor.priv.model_gen}, editor);
+        expect(editor.priv.import_plan.writes.length).toBeGreaterThan(0);
+        gobj_send_event(editor, "EV_APPLY_IMPORT", {model_gen: editor.priv.model_gen}, editor);
+        expect(gobj_current_state(editor)).toBe("ST_SAVING");
+    }
+
+    function answer_every_write(editor, remote)
+    {
+        for(let i = 0; i < 10; i++) {
+            const [write] = take("update-node");
+            if(!write) {
+                break;
+            }
+            answer(editor, remote, write, 0, as_the_store_answers(write));
+        }
+    }
+
+    test("from another topic's screen: the topic it wrote, not the one on screen", () => {
+        const {editor, remote, host} = build("r16m1", "db/users");
+        run_the_import(editor, host);
+        answer_every_write(editor, remote);
+        expect(gobj_current_state(editor)).toBe("ST_COLUMNS");
+        expect(editor.priv.written).toEqual({"db.roles": true});
+        expect(errors()).toEqual([]);
+    });
+
+    test("from the topics screen, where no topic is on screen", () => {
+        const {editor, remote, host} = build("r16m2", "db");
+        run_the_import(editor, host);
+        answer_every_write(editor, remote);
+        expect(gobj_current_state(editor)).toBe("ST_TOPICS");
+        expect(editor.priv.written).toEqual({"db.roles": true});
+        expect(errors()).toEqual([]);
+    });
+
+    test("a column answered with no record: the record it queued says the topic", () => {
+        const {editor, remote, host} = build("r16m3", "db/users");
+        run_the_import(editor, host);
+        const [topic] = take("update-node");
+        expect(topic.kw.topic_name).toBe("topics");
+        answer(editor, remote, topic, 0, as_the_store_answers(topic));
+        const [col] = take("update-node");
+        expect(col.kw.topic_name).toBe("cols");
+        answer(editor, remote, col, 0, null);
+        expect(editor.priv.written).toEqual({"db.roles": true});
+    });
+});
+
+describe("the reason a load failed", () => {
+
+    test("is a key the notice translates, not a string frozen in one language", () => {
+        const remote = gobj_create_service("r16e1_remote", "C_TEST_REMOTE", {}, yuno);
+        gobj_change_state(remote, "ST_SESSION");
+        const host = gobj_create("r16e1_host", "C_TEST_EDITOR_HOST", {}, yuno);
+        const editor = gobj_create("r16e1_editor", "C_YUI_SCHEMA_EDITOR", {
+            gobj_remote_yuno: remote, treedb_name: "treedb_system_schema"
+        }, host);
+        gobj_start(editor);
+        for(const c of take("nodes")) {
+            answer(editor, remote, c, -1, null, "");
+        }
+        expect(gobj_current_state(editor)).toBe("ST_IDLE");
+        expect(editor.priv.load_error).toBe("cannot load the schemas");
+        /*  The notice already says it: no detail repeating the title.  */
+        expect($in(editor, ".SCHEMA_NOTICE_DETAIL")).toBe(null);
+    });
+
+    test("the backend's own words travel as its key", () => {
+        const remote = gobj_create_service("r16e2_remote", "C_TEST_REMOTE", {}, yuno);
+        gobj_change_state(remote, "ST_SESSION");
+        const host = gobj_create("r16e2_host", "C_TEST_EDITOR_HOST", {}, yuno);
+        const editor = gobj_create("r16e2_editor", "C_YUI_SCHEMA_EDITOR", {
+            gobj_remote_yuno: remote, treedb_name: "treedb_system_schema"
+        }, host);
+        gobj_start(editor);
+        for(const c of take("nodes")) {
+            answer(editor, remote, c, -1, null, "treedb not found");
+        }
+        const $detail = $in(editor, ".SCHEMA_NOTICE_DETAIL");
+        expect($detail.getAttribute("data-i18n")).toBe("treedb not found");
+    });
+
+    test("a load that cannot leave keeps its key", () => {
+        const remote = gobj_create_service("r16e3_remote", "C_TEST_REMOTE", {}, yuno);
+        const host = gobj_create("r16e3_host", "C_TEST_EDITOR_HOST", {}, yuno);
+        const editor = gobj_create("r16e3_editor", "C_YUI_SCHEMA_EDITOR", {
+            gobj_remote_yuno: remote, treedb_name: "treedb_system_schema"
+        }, host);
+        gobj_start(editor);
+        gobj_change_state(remote, "ST_DISCONNECTED");
+        gobj_send_event(editor, "EV_REFRESH", {}, host);
+        expect(editor.priv.load_error).toBe("cannot reach the treedb");
+        const $detail = $in(editor, ".SCHEMA_NOTICE_DETAIL");
+        expect($detail.getAttribute("data-i18n")).toBe("cannot reach the treedb");
+    });
+});

package/src/treedb_graph_writes.wiring.test.js

@@ -149,13 +149,19 @@ function save_arrangement(engine, topic)
     });
 }
 
+/*  The answer as the real transports build it: the command frame
+ *  carries ONLY the request's `__md_command__` (C_IEVENT_CLI's
+ *  mt_command, gui_agent's C_AGENT_TREEDB_LINK), never the request's
+ *  own kw. Echoing the whole kw here once hid that the refusal read
+ *  its topic from a `record` no real answer carries.  */
 function answer(host, remote, request, result, comment)
 {
+    const md_command = request.kw.__md_command__ || {};
     gobj_send_event(host, "EV_MT_COMMAND_ANSWER", {
         result: result,
         comment: comment || "",
         data: null,
-        __md_iev__: {command_stack: [{command: request.command, kw: request.kw}]}
+        __md_iev__: {command_stack: [{command: request.command, kw: md_command}]}
     }, remote);
 }
 
@@ -172,6 +178,7 @@ describe("a __graphs__ write that does not land is said to the engine", () => {
         expect(commands.length).toBe(1);
         answer(host, remote, commands[0], -1, "not authorized");
         expect(refused()).toEqual(["yunos"]);
+        expect(logged.filter((l) => /names no topic/.test(l.msg))).toEqual([]);
     });
 
     test("landed: nothing is said", () => {