@yuneta/gobj-ui 7.25.15 → 7.25.16

package/README.md

@@ -1809,6 +1809,21 @@ fails:
 gobj_send_event(engine, "EV_GRAPHS_WRITE_REFUSED", {topic: record.topic}, gobj);
 ```
 
+**A refused write stays OWED until a Save writes it** (since `7.25.16`). In
+`7.25.15` the Save lit by the refusal went dark again at the next repaint of
+the history buttons -- they decide the Save from `history.canUndo()`, and they
+run on every history change, every change of mode and every redraw of the
+theme -- and a refusal heard in reading was never shown. The engine now keeps
+the topic as owed: the Save stays lit while one is, in edition and on entering
+it; the Save that writes the topic again pays it, and a reload of the data
+forgets it (the view holds what the backend holds).
+
+```js
+gobj_send_event(engine, "EV_GRAPHS_WRITE_REFUSED", {topic: "users"}, host);  // Save lit
+// ... a node moved and moved back (nothing to undo): Save still lit
+gobj_send_event(engine, "EV_SAVE_GRAPH", {}, engine);  // writes users; Save dark
+```
+
 **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
@@ -2113,7 +2128,7 @@ columns**, each in its declared `order`, with the storage composed underneath
 the qualified id, the fkey to the parent, the place among the siblings, and the
 versions that publish the change.
 
-**An edit here is a draft** (7.23.196, with the SDK's M36 design). A write moves
+**An edit here is a draft** (7.23.196, with SDK 7.25.0). A write moves
 no version and reaches no treedb. The HOST publishes it with `C_TREEDB`'s
 `save-schema`, which raises the `topic_version` of each topic that changed
 (the one that regenerates `topic_cols.json`) and the `schema_version` (the one
@@ -2206,6 +2221,7 @@ else.
   master the tranger, so it refuses every write), `base_route`, `$container`.
 - In: `EV_SHOW` (`{subpath}` — the tail it owns is `<treedb>[/<topic>]` or
   `<treedb>/diagram`), `EV_HIDE`, `EV_TRANSPORT_STATE`, `EV_REFRESH`,
+  `EV_DRAFTS` (`{drafts: {<treedb>: [<topic>, …]}}`, see above),
   `EV_LANGUAGE_CHANGED`, `EV_MT_COMMAND_ANSWER`.
 - A **drop** of the transport — `EV_TRANSPORT_STATE {connected: false}`, or a
   failed answer that arrives while the transport is out of `ST_SESSION` — ends
@@ -2230,6 +2246,19 @@ else.
   gobj_send_event(editor, "EV_SHOW", {subpath: "db"}, host);   // waits
   // ... the three `nodes` answers land -> ST_TOPICS of `db`
   ```
+
+  **`EV_REFRESH` waits too** (7.25.16). Heard in `ST_SAVING` -- a host that
+  sends it after a Save while the writes are still in flight -- it is logged
+  and run when the writes END, forgetting the drafts as a Refresh does. It
+  ran at once: the load replaced `ST_SAVING`, and what was still queued (the
+  topic of a topic delete, after its columns) was never sent, with no word.
+
+  ```js
+  gobj_send_event(editor, "EV_CONFIRMED", {what: "topic", topic: "users",
+      model_gen: n}, editor);                                  // ST_SAVING, 2 deletes
+  gobj_send_event(editor, "EV_REFRESH", {}, host);             // logged, waits
+  // ... both deletes answered -> ST_LOADING -> the model is read again
+  ```
 - **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
@@ -2315,6 +2344,28 @@ else.
   // ... the load lands: the same click now opens the form
   ```
 
+  **The Save of a form open when the reload is owed keeps what was typed**
+  (7.25.16). It runs the owed reload and says
+  `the schemas shown may be out of date: they are read again, and the form opens again on them with your changes`.
+  The load that lands closes the form and opens it again on the schemas it
+  read, with the fields the operator CHANGED put back on top -- only those,
+  so a field left as it was shows what the store holds now and the next Save
+  does not write the old value over a newer one. A new column or topic comes
+  back whole (nothing of it is in the store). Before, the form was closed and
+  what was typed was lost. It is not opened again when what it edited is not
+  on the schemas read (a column deleted meanwhile, another treedb, another
+  screen): that is said with
+  `the schemas were read again and what the form was editing is not there any more`.
+  A form the operator closed while the load ran stays closed. Both texts are
+  i18n keys the consumer's locales carry.
+
+  ```js
+  // reload owed; column form of db.users.id open, header typed "Identifier"
+  // Save -> refused, the model is read again
+  // ... the load lands: the form opens again, header "Identifier", the rest
+  //     of the fields as the store holds them now; Save writes them
+  ```
+
   A write answered with no record reloads the model and
   says what it did not send after it
   (`the treedb did not describe a write back: the writes after it were not sent`).
@@ -3280,9 +3331,13 @@ a modal
 `TOAST*`. Every button of that chrome -- each ✕, a dialog's back arrow, a
 confirmation's answers (`CONFIRM_BTN`) -- carries a translatable `title` and
 `aria-label` (7.25.10). An answer is named by its own label, so pass i18n
-KEYS as `yes_label` / `confirm_label` / … (`{yes_label: "install"}`): the
-defaults (`"Yes"`, `"Delete"`) are not lower-case and no validated locale
-can hold them.
+KEYS as `yes_label` / `confirm_label` / … (`{yes_label: "install"}`). The
+defaults are keys too since 7.25.16 -- `"ok"`, `"yes"`, `"no"`, `"delete"`
+and `"cancel"` (they were `"OK"`, `"Yes"`, … , which no validated locale can
+hold) -- so an app that mounts these dialogs defines those five keys, and
+its `validate-locales` asks for them. `yui_install_ask_once()` asks
+`"install this app"` (was `"Install this app?"`), answered `"install"` /
+`"not now"`.
 
 Those names identify the *kind* of block, not the *instance*: every window in
 the app is a `C_YUI_WINDOW`, every popup is a `MODAL`. To target **one**

package/dist/gobj-ui.cjs.js

@@ -3437,8 +3437,28 @@ function build_dialog(shell, message, buttons, opts) {
 		if ($primary) $primary.focus();
 	});
 }
+/***************************************************************
+*  The labels a confirmation shows when the caller names none.
+*
+*  They are i18n KEYS, lower-case like every key: a button
+*  translates its label with the app's t() -- its text, its title
+*  and its aria-label. Until 7.25.16 they were "OK", "Yes", "No",
+*  "Delete" and "Cancel", keys that no locale carries, so a dialog
+*  that took the defaults read in English in every language.
+*
+*  Written as `i18n:` entries so the validate-locales script of a
+*  consumer finds them when it scans this module: an app that
+*  mounts these dialogs must define the five keys.
+***************************************************************/
+var DEFAULT_LABEL = Object.freeze({
+	ok: Object.freeze({ i18n: "ok" }),
+	yes: Object.freeze({ i18n: "yes" }),
+	no: Object.freeze({ i18n: "no" }),
+	delete: Object.freeze({ i18n: "delete" }),
+	cancel: Object.freeze({ i18n: "cancel" })
+});
 function yui_shell_confirm_ok(shell, message, opts) {
-	let label = opts && opts.ok_label || "OK";
+	let label = opts && opts.ok_label || DEFAULT_LABEL.ok.i18n;
 	if (!opts || !opts.type) opts = Object.assign({}, opts, { type: "success" });
 	return build_dialog(shell, message, [{
 		label,
@@ -3447,8 +3467,8 @@ function yui_shell_confirm_ok(shell, message, opts) {
 	}], opts).then(() => void 0);
 }
 function yui_shell_confirm_yesno(shell, message, opts) {
-	let yes_label = opts && opts.yes_label || "Yes";
-	let no_label = opts && opts.no_label || "No";
+	let yes_label = opts && opts.yes_label || DEFAULT_LABEL.yes.i18n;
+	let no_label = opts && opts.no_label || DEFAULT_LABEL.no.i18n;
 	return build_dialog(shell, message, [{
 		label: yes_label,
 		value: "yes",
@@ -3473,8 +3493,8 @@ function yui_shell_confirm_yesno(shell, message, opts) {
 *  pressed.
 ***************************************************************/
 function yui_shell_confirm_danger(shell, message, opts) {
-	let confirm_label = opts && opts.confirm_label || "Delete";
-	let cancel_label = opts && opts.cancel_label || "Cancel";
+	let confirm_label = opts && opts.confirm_label || DEFAULT_LABEL.delete.i18n;
+	let cancel_label = opts && opts.cancel_label || DEFAULT_LABEL.cancel.i18n;
 	if (!opts || !opts.type) opts = Object.assign({}, opts, { type: "danger" });
 	return build_dialog(shell, message, [{
 		label: confirm_label,
@@ -3486,9 +3506,9 @@ function yui_shell_confirm_danger(shell, message, opts) {
 	}], opts).then((v) => v === "confirm");
 }
 function yui_shell_confirm_yesnocancel(shell, message, opts) {
-	let yes_label = opts && opts.yes_label || "Yes";
-	let no_label = opts && opts.no_label || "No";
-	let cancel_label = opts && opts.cancel_label || "Cancel";
+	let yes_label = opts && opts.yes_label || DEFAULT_LABEL.yes.i18n;
+	let no_label = opts && opts.no_label || DEFAULT_LABEL.no.i18n;
+	let cancel_label = opts && opts.cancel_label || DEFAULT_LABEL.cancel.i18n;
 	return build_dialog(shell, message, [
 		{
 			label: yes_label,
@@ -59232,6 +59252,7 @@ var PRIVATE_DATA$4 = {
 	__graphs__: [],
 	_graph_properties: {},
 	_saved_graph_properties: {},
+	_graphs_writes_owed: {},
 	yet_showed: false,
 	edit_mode: false,
 	operation_mode: null,
@@ -60927,6 +60948,7 @@ function save_topic_graph_properties(gobj, topic_name) {
 	if (!topic_arrangement_changed(properties, priv._saved_graph_properties[topic_name])) return;
 	properties.__origin__ = (0, _yuneta_gobj_js.gobj_read_str_attr)((0, _yuneta_gobj_js.gobj_yuno)(), "node_uuid");
 	priv._saved_graph_properties[topic_name] = (0, _yuneta_gobj_js.json_deep_copy)(properties);
+	delete priv._graphs_writes_owed[topic_name];
 	let kw_update = {
 		treedb_name: priv.treedb_name,
 		topic_name: "__graphs__",
@@ -61008,8 +61030,13 @@ function update_history_buttons(gobj) {
 			} else {
 				disableElements($container, ".EV_HISTORY_UNDO");
 				set_active_state($container, ".EV_HISTORY_UNDO", false);
-				disableElements($container, ".EV_SAVE_GRAPH");
-				set_submit_state($container, ".EV_SAVE_GRAPH", false);
+				if (graphs_write_is_owed(gobj)) {
+					enableElements($container, ".EV_SAVE_GRAPH");
+					set_submit_state($container, ".EV_SAVE_GRAPH", true);
+				} else {
+					disableElements($container, ".EV_SAVE_GRAPH");
+					set_submit_state($container, ".EV_SAVE_GRAPH", false);
+				}
 			}
 		} else {
 			disableElements($container, ".EV_HISTORY_REDO");
@@ -61966,6 +61993,15 @@ function run_preview_undo(gobj, key) {
 		(0, _yuneta_gobj_js.log_error)(`${(0, _yuneta_gobj_js.gobj_short_name)(gobj)}: cannot undo the preview: ${e}`);
 	}
 }
+/************************************************************
+*  Is a `__graphs__` write owed? The host said one did not land
+*  (EV_GRAPHS_WRITE_REFUSED), and no Save has written that topic
+*  since. The Save button stays lit while it is: the history
+*  knows nothing of it, so `canUndo()` alone would put it out.
+************************************************************/
+function graphs_write_is_owed(gobj) {
+	return Object.keys(gobj.priv._graphs_writes_owed).length > 0;
+}
 function mark_graph_dirty(gobj) {
 	let $container = (0, _yuneta_gobj_js.gobj_read_attr)(gobj, "$container");
 	enableElements($container, ".EV_SAVE_GRAPH");
@@ -65706,6 +65742,7 @@ function ac_clear_data(gobj, event, kw, src) {
 	deselect_node(gobj);
 	hide_create_popover(gobj);
 	(0, _yuneta_gobj_js.gobj_write_attr)(gobj, "records", {});
+	priv._graphs_writes_owed = {};
 	priv._nodes_total = 0;
 	priv._nodes_placed = 0;
 	priv._fold_model = null;
@@ -65798,11 +65835,9 @@ function ac_save_graph(gobj, event, kw, src) {
 		disableElements($container, ".EV_SAVE_GRAPH");
 		set_submit_state($container, ".EV_SAVE_GRAPH", false);
 		let history = graph_get_plugin(gobj, "history");
-		if (history) {
-			history.clear();
-			update_history_buttons(gobj);
-		}
+		if (history) history.clear();
 		save_geometry(gobj);
+		update_history_buttons(gobj);
 	}
 	return 0;
 }
@@ -65825,6 +65860,7 @@ function ac_graphs_write_refused(gobj, event, kw, src) {
 	}
 	(0, _yuneta_gobj_js.log_warning)(`${(0, _yuneta_gobj_js.gobj_short_name)(gobj)}: the arrangement of '${topic_name}' was not saved: the next Save writes it again`);
 	forget_refused_graphs_write(priv._saved_graph_properties, topic_name);
+	priv._graphs_writes_owed[topic_name] = true;
 	if (priv.edit_mode) mark_graph_dirty(gobj);
 	return 0;
 }
@@ -72334,6 +72370,8 @@ var PRIVATE_DATA$1 = {
 	dropped_round: 0,
 	dropped_write: 0,
 	reload_after_write: false,
+	refresh_after_write: false,
+	reopen_form: null,
 	pending_seg: null,
 	import_plan: null,
 	import_pane: null,
@@ -72539,14 +72577,61 @@ function owes_reload(gobj) {
 	let state = (0, _yuneta_gobj_js.gobj_current_state)(gobj);
 	return gobj.priv.reload_on_open && transport_in_session(gobj) && state !== "ST_LOADING" && state !== "ST_SAVING";
 }
-function reload_first(gobj, event) {
+function reload_first(gobj, event, reopen) {
+	let priv = gobj.priv;
 	if (!owes_reload(gobj)) return false;
-	gobj.priv.reload_on_open = false;
+	priv.reload_on_open = false;
+	if (reopen && priv.dialog) {
+		priv.reopen_form = Object.assign((0, _yuneta_gobj_js.json_deep_copy)(reopen), {
+			dialog: priv.dialog,
+			treedb_id: priv.treedb_id
+		});
+		(0, _yuneta_gobj_js.log_warning)(`${(0, _yuneta_gobj_js.gobj_short_name)(gobj)}: ${event} refused, the schemas shown may be out of date: they are read again first, and the form opens again on them with its changes`);
+		yui_shell_show_error(yui_shell_of(gobj), "the schemas shown may be out of date: they are read again, and the form opens again on them with your changes", { t: i18next.t });
+		request_model(gobj, true);
+		return true;
+	}
 	(0, _yuneta_gobj_js.log_warning)(`${(0, _yuneta_gobj_js.gobj_short_name)(gobj)}: ${event} refused, the schemas shown may be out of date: they are read again first`);
 	yui_shell_show_error(yui_shell_of(gobj), "the schemas shown may be out of date: they are read again, try again when they are in", { t: i18next.t });
 	request_model(gobj, true);
 	return true;
 }
+/***************************************************************
+*  The form reload_first() took down, opened again on the model
+*  the load just made, with the operator's changes on top.
+*
+*  Only where it can mean the same thing: on the treedb it was
+*  opened on, on the screen whose Save it has, and on a record
+*  that is still there. Otherwise it is not opened, and that is
+*  said -- a form opened on another treedb, or on nothing, would
+*  save somewhere the operator never meant.
+***************************************************************/
+function reopen_form(gobj, r) {
+	let priv = gobj.priv;
+	let state = (0, _yuneta_gobj_js.gobj_current_state)(gobj);
+	let treedb = current_treedb(gobj);
+	let topic = null;
+	let col = null;
+	let fits = false;
+	if (treedb && r.treedb_id === priv.treedb_id) {
+		if (r.form === "column" && state === "ST_COLUMNS") {
+			topic = find_topic(priv.model, priv.treedb_id, r.topic);
+			col = topic && !r.creating ? find_col(priv.model, priv.treedb_id, r.topic, r.col) : null;
+			fits = !!topic && (r.creating || !!col);
+		} else if (r.form === "topic" && state === "ST_TOPICS") {
+			topic = r.creating ? null : find_topic(priv.model, priv.treedb_id, r.topic);
+			fits = r.creating || !!topic;
+		}
+	}
+	if (!fits) {
+		(0, _yuneta_gobj_js.log_warning)(`${(0, _yuneta_gobj_js.gobj_short_name)(gobj)}: the ${r.form} form was not opened again: what it edited is not on the schemas the reload read (${r.treedb_id} ${r.topic}${r.col ? "." + r.col : ""}, now ${state})`);
+		yui_shell_show_error(yui_shell_of(gobj), "the schemas were read again and what the form was editing is not there any more", { t: i18next.t });
+		return -1;
+	}
+	if (r.form === "column") open_column_form(gobj, topic, col, null, r.changed);
+	else open_topic_form(gobj, treedb, topic, r.changed);
+	return 0;
+}
 function reload_after_move(gobj) {
 	if (!owes_reload(gobj)) return 0;
 	gobj.priv.reload_on_open = false;
@@ -72692,7 +72777,7 @@ function build_model(gobj) {
 *  measurement of "changed since" can be measured against.
 *
 *  `written` is forgotten only by an EV_REFRESH (the view's own
-*  Refresh button, or a host that sends one). Since M36 a write moves no
+*  Refresh button, or a host that sends one). Since 7.23.196 a write moves no
 *  version, so `written` is the draft chip of what this session
 *  wrote and no longer half of a version measurement: a reload of
 *  the reconnect, or of a write that turned out done, forgot the
@@ -73970,6 +74055,55 @@ function read_form($root) {
 	return out;
 }
 /***************************************************************
+*  A column form as its Save sends it: the fields and the flags.
+***************************************************************/
+function column_form_state($form) {
+	let values = read_form($form);
+	values.flag = read_flags($form);
+	return values;
+}
+/***************************************************************
+*  The fields the operator CHANGED: those whose value is not the
+*  one the form was opened with.
+***************************************************************/
+function form_changes(initial, values) {
+	let changed = {};
+	for (let name of Object.keys(values)) if (JSON.stringify(values[name]) !== JSON.stringify(initial[name])) changed[name] = values[name];
+	return changed;
+}
+/***************************************************************
+*  Put changes back into a form just built (reopen_form()). The
+*  flags are drawn again for the type the changes leave, holding
+*  the flags they name, so none of them is hidden by the type.
+***************************************************************/
+function apply_form_changes(gobj, $form, changes) {
+	for (let name of Object.keys(changes)) {
+		if (name === "flag") continue;
+		let $control = $form.querySelector(`[data-name="${name}"]`);
+		if (!$control) {
+			(0, _yuneta_gobj_js.log_warning)(`${(0, _yuneta_gobj_js.gobj_short_name)(gobj)}: a change of '${name}' has no field in the form opened again: not put back`);
+			continue;
+		}
+		if ($control.type === "checkbox") $control.checked = !!changes[name];
+		else $control.value = changes[name];
+	}
+	let $type = $form.querySelector("[data-name=\"type\"]");
+	if ($type && $form.querySelector(".SCHEMA_FLAGS")) {
+		let flags = Array.isArray(changes.flag) ? changes.flag : read_flags($form);
+		redraw_flags(gobj, $form, $type.value, flags);
+	}
+}
+/***************************************************************
+*  Draw the flags of a column form again, for `type`, with
+*  `current` checked.
+***************************************************************/
+function redraw_flags(gobj, $form, type, current) {
+	let $holder = $form.querySelector(".SCHEMA_FLAGS");
+	let $new = (0, _yuneta_gobj_js.createElement2)(flags_field(gobj, type, current));
+	$holder.parentNode.replaceChild($new, $holder);
+	(0, _yuneta_gobj_js.refresh_language)($new, i18next.t);
+}
+/***************************************************************
 *  The flags of a column, as checkboxes that say what they do.
 *
 *  Grouped the way they act, greyed when they are meaningless on
@@ -74043,7 +74177,7 @@ function read_flags($root) {
 *  Offering a field that silently does something else is worse
 *  than not offering it.
 ***************************************************************/
-function open_column_form(gobj, topic, col, prefill) {
+function open_column_form(gobj, topic, col, prefill, changes) {
 	let priv = gobj.priv;
 	let record = prefill || col && col.record || {};
 	let creating = !col;
@@ -74147,11 +74281,7 @@ function open_column_form(gobj, topic, col, prefill) {
 	]);
 	let $type = $form.querySelector("[data-name=\"type\"]");
 	$type.addEventListener("change", () => {
-		let $holder = $form.querySelector(".SCHEMA_FLAGS");
-		let current = read_flags($form);
-		let $new = (0, _yuneta_gobj_js.createElement2)(flags_field(gobj, $type.value, current));
-		$holder.parentNode.replaceChild($new, $holder);
-		(0, _yuneta_gobj_js.refresh_language)($new, i18next.t);
+		redraw_flags(gobj, $form, $type.value, read_flags($form));
 	});
 	$form.addEventListener("change", (evt) => {
 		let $box = evt.target.closest(".SCHEMA_FLAG_BOX");
@@ -74163,15 +74293,17 @@ function open_column_form(gobj, topic, col, prefill) {
 		evt.stopPropagation();
 		close_dialog(gobj);
 	});
+	let initial = column_form_state($form);
+	if (changes) apply_form_changes(gobj, $form, changes);
 	$form.querySelector(".SCHEMA_COL_FORM_SAVE").addEventListener("click", (evt) => {
 		evt.stopPropagation();
-		let values = read_form($form);
-		values.flag = read_flags($form);
+		let values = column_form_state($form);
 		(0, _yuneta_gobj_js.gobj_send_event)(gobj, "EV_SAVE_COLUMN", {
 			topic: topic.name,
 			col: creating ? "" : col.name,
 			creating,
 			values,
+			changed: creating ? values : form_changes(initial, values),
 			model_gen
 		}, gobj);
 	});
@@ -74191,7 +74323,7 @@ function stringify_field(value) {
 *  actually has: a pkey naming no column is a topic that does
 *  not open, and it is the first thing the validator looks for.
 ***************************************************************/
-function open_topic_form(gobj, treedb, topic) {
+function open_topic_form(gobj, treedb, topic, changes) {
 	let record = topic && topic.record || {};
 	let creating = !topic;
 	let col_names = topic ? topic.cols.map((c) => c.name) : [];
@@ -74286,12 +74418,16 @@ function open_topic_form(gobj, treedb, topic) {
 		evt.stopPropagation();
 		close_dialog(gobj);
 	});
+	let initial = read_form($form);
+	if (changes) apply_form_changes(gobj, $form, changes);
 	$form.querySelector(".SCHEMA_TOPIC_FORM_SAVE").addEventListener("click", (evt) => {
 		evt.stopPropagation();
+		let values = read_form($form);
 		(0, _yuneta_gobj_js.gobj_send_event)(gobj, "EV_SAVE_TOPIC", {
 			topic: creating ? "" : topic.name,
 			creating,
-			values: read_form($form),
+			values,
+			changed: creating ? values : form_changes(initial, values),
 			model_gen
 		}, gobj);
 	});
@@ -75061,20 +75197,28 @@ function transport_dropped(gobj) {
 function end_load(gobj) {
 	let priv = gobj.priv;
 	let seg = priv.pending_seg;
+	let reopen = priv.reopen_form;
 	priv.pending_seg = null;
+	priv.reopen_form = null;
 	if (priv.dialog && priv.dialog_gen !== priv.model_gen) {
-		(0, _yuneta_gobj_js.log_warning)(`${(0, _yuneta_gobj_js.gobj_short_name)(gobj)}: a dialog was open on the schemas the load replaced: closed`);
+		if (!reopen || reopen.dialog !== priv.dialog) {
+			reopen = null;
+			(0, _yuneta_gobj_js.log_warning)(`${(0, _yuneta_gobj_js.gobj_short_name)(gobj)}: a dialog was open on the schemas the load replaced: closed`);
+			yui_shell_show_error(yui_shell_of(gobj), "the schemas were read again: open the dialog again", { t: i18next.t });
+		}
 		close_dialog(gobj);
-		yui_shell_show_error(yui_shell_of(gobj), "the schemas were read again: open the dialog again", { t: i18next.t });
-	}
+	} else reopen = null;
+	let ret = 0;
 	if (priv.model) {
-		if (seg !== null) return apply_seg(gobj, seg);
-		return go(gobj, priv.treedb_id, priv.topic_name, priv.diagram, false);
+		if (seg !== null) ret = apply_seg(gobj, seg);
+		else ret = go(gobj, priv.treedb_id, priv.topic_name, priv.diagram, false);
+	} else {
+		if (seg !== null) adopt_seg(gobj, seg);
+		(0, _yuneta_gobj_js.gobj_change_state)(gobj, "ST_IDLE");
+		render(gobj);
 	}
-	if (seg !== null) adopt_seg(gobj, seg);
-	(0, _yuneta_gobj_js.gobj_change_state)(gobj, "ST_IDLE");
-	render(gobj);
-	return 0;
+	if (reopen) reopen_form(gobj, reopen);
+	return ret;
 }
 /***************************************************************
 *  The queue is done, or gave up. Either way the model is what
@@ -75099,7 +75243,11 @@ function end_writes(gobj, error) {
 		render(gobj);
 		publish_check(gobj);
 	}
-	reload_if_owed(gobj);
+	if (priv.refresh_after_write) {
+		priv.refresh_after_write = false;
+		priv.reload_after_write = false;
+		request_model(gobj, false);
+	} else reload_if_owed(gobj);
 	return error ? -1 : 0;
 }
 /***************************************************************
@@ -75380,7 +75528,9 @@ function ac_mt_command_answer(gobj, event, kw, src) {
 				priv.save_return = "";
 				priv.save_wrote = false;
 				(0, _yuneta_gobj_js.gobj_publish_event)(gobj, "EV_RECORD_WRITTEN", { treedb_name: (0, _yuneta_gobj_js.gobj_read_str_attr)(gobj, "treedb_name") });
-				request_model(gobj, true);
+				let refresh = priv.refresh_after_write;
+				priv.refresh_after_write = false;
+				request_model(gobj, !refresh);
 				return 0;
 			}
 		}
@@ -75527,6 +75677,21 @@ function ac_refresh$1(gobj, event, kw, src) {
 	return request_model(gobj, false);
 }
 /***************************************************************
+*  EV_REFRESH while writes are in flight: the view's own Refresh
+*  is busy then, so it is a host's (after a Save, say).
+*
+*  Run when the writes END (end_writes()), not now. Run now, the
+*  load replaced ST_SAVING and what was still queued was never
+*  sent, with no word: nothing but the end of what is in flight
+*  moves this view out of ST_LOADING or ST_SAVING.
+***************************************************************/
+function ac_refresh_after_the_writes(gobj, event, kw, src) {
+	let priv = gobj.priv;
+	(0, _yuneta_gobj_js.log_warning)(`${(0, _yuneta_gobj_js.gobj_short_name)(gobj)}: ${event} while writes are in flight: the schemas are read again when the writes end`);
+	priv.refresh_after_write = true;
+	return 0;
+}
+/***************************************************************
 *  What a DIALOG sends while the model is being read again: the
 *  Save of a form left open when the reload started, a
 *  confirmation, an import. The loading screen took the rest of
@@ -75701,7 +75866,13 @@ function ac_save_column(gobj, event, kw, src) {
 	let topic = find_topic(priv.model, priv.treedb_id, kw.topic);
 	let col = kw.creating ? null : find_col(priv.model, priv.treedb_id, kw.topic, kw.col);
 	if (refuse_if_readonly(gobj, event)) return -1;
-	if (reload_first(gobj, event)) return -1;
+	if (reload_first(gobj, event, {
+		form: "column",
+		topic: kw.topic,
+		col: kw.col,
+		creating: !!kw.creating,
+		changed: kw.changed || {}
+	})) return -1;
 	if (built_on_a_replaced_model(gobj, event, kw)) return -1;
 	if (!topic || !kw.creating && !col) {
 		(0, _yuneta_gobj_js.log_error)(`${(0, _yuneta_gobj_js.gobj_short_name)(gobj)}: ${event} names no column: ${kw.topic}.${kw.col}`);
@@ -75829,7 +76000,13 @@ function ac_save_topic(gobj, event, kw, src) {
 	let treedb = current_treedb(gobj);
 	let topic = kw.creating ? null : find_topic(priv.model, priv.treedb_id, kw.topic);
 	if (refuse_if_readonly(gobj, event)) return -1;
-	if (reload_first(gobj, event)) return -1;
+	if (reload_first(gobj, event, {
+		form: "topic",
+		topic: kw.topic,
+		col: "",
+		creating: !!kw.creating,
+		changed: kw.changed || {}
+	})) return -1;
 	if (built_on_a_replaced_model(gobj, event, kw)) return -1;
 	if (!treedb || !kw.creating && !topic) {
 		(0, _yuneta_gobj_js.log_error)(`${(0, _yuneta_gobj_js.gobj_short_name)(gobj)}: ${event} names no topic: ${kw.topic}`);
@@ -76292,7 +76469,11 @@ function create_gclass$1(gclass_name) {
 				null
 			]
 		])],
-		["ST_SAVING", COMMON.slice()]
+		["ST_SAVING", COMMON.filter((a) => a[0] !== "EV_REFRESH").concat([[
+			"EV_REFRESH",
+			ac_refresh_after_the_writes,
+			null
+		]])]
 	];
 	const event_types = [
 		["EV_MT_COMMAND_ANSWER", _yuneta_gobj_js.event_flag_t.EVF_PUBLIC_EVENT],