@yuneta/gobj-ui 7.7.1 → 7.8.1

package/README.md

@@ -491,6 +491,39 @@ The buttons of that toolbar never shrink. When the row runs out of room, the
 url is what gives way, cut with an ellipsis, and the whole value stays in the
 `title` and the `aria-label`.
 
+### Editing a topic table in place
+
+A writable scalar is editable in the table, in edition mode
+(`with_inline_edit`, default on). Changing one field used to mean opening the
+record form, changing it, saving and closing.
+
+**Which cells, and why not the rest.** The schema decides first: only a column
+flagged `writable`, and never the pkey — renaming what a record is KEYED by is
+not a field edit. Then the type: a hook holds children and an fkey IS a link,
+so both are edited by linking; a dict or a list is a document the form has an
+editor for; a date cell shows a formatted string over an epoch, so typing into
+it would write the string. Those stay with the form, one click away on the
+same row. `boolean` gets a tick, `enum` the list of its own values, numbers a
+number editor.
+
+**The write is a partial update with no `autolink`, and that is the whole
+safety of it.** `treedb_update_node()` merges (`json_object_update`), so the
+fields it does not carry are left alone; `autolink` is the option that wipes a
+node's links and rebuilds them from the fkeys the record carries, and on a
+partial record it reads that as "no parents", detaches the node and answers
+**success**. So a cell edit travels as its own event, `EV_UPDATE_FIELD`, and
+not as `EV_UPDATE_RECORD` — that one does send autolink, and may, because the
+form hands it the whole record with its fkeys in it. See
+`schema_write_options.js` for the rule and why each word of it is there.
+
+`editable` is a **function** on the column, not a flag: edition mode is
+toggled on a table that is already built, so the answer has to be asked for at
+the moment of the click.
+
+A refused write puts the topic back to what the treedb has. Leaving the typed
+value on screen is tolerable for a form, which stays open on the values it
+failed with; a cell edited in place would just look saved.
+
 ### Reading a topic table: filters, columns, CSV
 
 `C_YUI_TREEDB_TOPIC_WITH_FORM` had one global search box over the loaded rows.

package/dist/gobj-ui.cjs.js

@@ -14416,6 +14416,66 @@ function yui_shell_show_route_map(shell, opts) {
 	return modal_ref.modal;
 }
 //#endregion
+//#region src/nodes_answer.js
+/***********************************************************************
+*          nodes_answer.js
+*
+*      What the `nodes` command answered, whatever shape it came in.
+*
+*      A treedb lives in memory on the backend, so walking it is not
+*      what costs: serializing every node, pushing it through a
+*      websocket and parsing it here is. So `nodes` learned to answer a
+*      PAGE, with the contract `list-keys` of C_TRANGER already used:
+*
+*          no `limit`  ->  the plain list it has always answered
+*          a `limit`   ->  {total_rows, pages, data}
+*
+*      Both shapes are alive at once and will be for a long time: the
+*      SPA talks to backends the operator configures, and an older one
+*      answers the plain list whatever this app asks for. So the shape
+*      is read, never assumed — a view that indexed `data` as an array
+*      would render an object's keys as rows the day a backend
+*      upgraded, which is the kind of break that looks like a bug in
+*      the app.
+*
+*      Pure and tested, because this is exactly the sort of thing that
+*      cannot be exercised until the other side moves.
+*
+*          Copyright (c) 2026, ArtGins.
+*          All Rights Reserved.
+***********************************************************************/
+/***************************************************************
+*  nodes_answer(data) -> {rows, total, pages, paged}
+*
+*  `rows` is always an array, so a caller never has to ask which
+*  shape it got. `paged` says whether the backend answered a page,
+*  which is the one thing a caller may legitimately care about (it is
+*  what tells a pager it has something to page).
+***************************************************************/
+function nodes_answer(data) {
+	if (Array.isArray(data)) return {
+		rows: data,
+		total: data.length,
+		pages: 1,
+		paged: false
+	};
+	if (data && typeof data === "object" && Array.isArray(data.data)) {
+		let rows = data.data;
+		return {
+			rows,
+			total: typeof data.total_rows === "number" ? data.total_rows : rows.length,
+			pages: typeof data.pages === "number" && data.pages > 0 ? data.pages : 1,
+			paged: true
+		};
+	}
+	return {
+		rows: [],
+		total: 0,
+		pages: 1,
+		paged: false
+	};
+}
+//#endregion
 //#region src/c_yui_treedb_topics.js
 /***********************************************************************
 *          c_yui_treedb_topics.js
@@ -15609,7 +15669,13 @@ function ac_mt_command_answer$2(gobj, event, kw, src) {
 	}
 	if (result < 0) {
 		if (command === "descs") show_load_error$1(gobj, (0, i18next.t)(comment));
-		else yui_shell_show_error(yui_shell_of(gobj), comment, { t: i18next.t });
+		else {
+			yui_shell_show_error(yui_shell_of(gobj), comment, { t: i18next.t });
+			if (command === "update-node" || command === "create-node") {
+				let failed_topic = (0, _yuneta_gobj_js.kw_get_str)(gobj, kw_command, "topic_name", "", 0);
+				if (failed_topic) treedb_nodes$1(gobj, (0, _yuneta_gobj_js.gobj_read_str_attr)(gobj, "treedb_name"), failed_topic, { list_dict: true });
+			}
+		}
 		return 0;
 	}
 	switch (command) {
@@ -15623,7 +15689,7 @@ function ac_mt_command_answer$2(gobj, event, kw, src) {
 			let topic_name = (0, _yuneta_gobj_js.kw_get_str)(gobj, kw_command, "topic_name", "", _yuneta_gobj_js.kw_flag_t.KW_REQUIRED);
 			if (result >= 0) {
 				let gobj_topic_form = (0, _yuneta_gobj_js.gobj_find_child)(gobj, { __gobj_name__: `${(0, _yuneta_gobj_js.gobj_name)(gobj)}?${topic_name}` });
-				(0, _yuneta_gobj_js.gobj_send_event)(gobj_topic_form, "EV_LOAD_NODES", data, gobj);
+				(0, _yuneta_gobj_js.gobj_send_event)(gobj_topic_form, "EV_LOAD_NODES", nodes_answer(data).rows, gobj);
 			}
 			break;
 		case "create-node":
@@ -15849,6 +15915,35 @@ function ac_create_record(gobj, event, kw, src) {
 	});
 }
 /********************************************
+*  Event from formtable: ONE field of one record, edited in place.
+*
+*  Written as a PARTIAL update and, deliberately, with NO `autolink`.
+*  `treedb_update_node()` merges (`json_object_update`), so the fields
+*  it does not carry are left alone; and `autolink` is what wipes a
+*  node's links to rebuild them from the fkeys the record carries — on
+*  a partial record that reads as "no parents" and DETACHES the node,
+*  answering success. A cell edit changes a scalar and must not be able
+*  to touch a link at all.
+*
+*  This is why it is its own event and not EV_UPDATE_RECORD: that one
+*  DOES send autolink, and may, because the form hands it the whole
+*  record with its fkeys in it.
+********************************************/
+function ac_update_field(gobj, event, kw, src) {
+	if (refuse_if_readonly$3(gobj, event)) return -1;
+	let topic_name = kw.topic_name || (0, _yuneta_gobj_js.gobj_read_attr)(src, "topic_name");
+	let desc = ((0, _yuneta_gobj_js.gobj_read_attr)(gobj, "descs") || {})[topic_name];
+	let pkey = desc && desc.pkey || "id";
+	if (kw.id === void 0 || kw.id === null || !kw.field) {
+		(0, _yuneta_gobj_js.log_error)(`${(0, _yuneta_gobj_js.gobj_short_name)(gobj)}: a field write with no id or no field`);
+		return -1;
+	}
+	let record = {};
+	record[pkey] = kw.id;
+	record[kw.field] = kw.value;
+	return treedb_update_node$1(gobj, (0, _yuneta_gobj_js.gobj_read_str_attr)(gobj, "treedb_name"), topic_name, record, { list_dict: true });
+}
+/********************************************
 *  Event from formtable
 *  kw: {
 *      topic_name,
@@ -15970,6 +16065,11 @@ function create_gclass$6(gclass_name) {
 			ac_update_record,
 			null
 		],
+		[
+			"EV_UPDATE_FIELD",
+			ac_update_field,
+			null
+		],
 		[
 			"EV_DELETE_RECORD",
 			ac_delete_record,
@@ -16043,6 +16143,7 @@ function create_gclass$6(gclass_name) {
 		["EV_TREEDB_NODE_DELETED", _yuneta_gobj_js.event_flag_t.EVF_PUBLIC_EVENT],
 		["EV_CREATE_RECORD", 0],
 		["EV_UPDATE_RECORD", 0],
+		["EV_UPDATE_FIELD", 0],
 		["EV_DELETE_RECORD", 0],
 		["EV_RECORD_WRITTEN", _yuneta_gobj_js.event_flag_t.EVF_OUTPUT_EVENT | _yuneta_gobj_js.event_flag_t.EVF_NO_WARN_SUBS],
 		["EV_REFRESH_TOPIC", 0],
@@ -16256,6 +16357,7 @@ var attrs_table$5 = [
 	(0, _yuneta_gobj_js.SDATA)(_yuneta_gobj_js.data_type_t.DTP_BOOLEAN, "with_columns_button", 0, true, "Button toolbar COLUMNS (choose which columns the table shows)"),
 	(0, _yuneta_gobj_js.SDATA)(_yuneta_gobj_js.data_type_t.DTP_BOOLEAN, "with_export_button", 0, true, "Button toolbar EXPORT (download as CSV what the table holds)"),
 	(0, _yuneta_gobj_js.SDATA)(_yuneta_gobj_js.data_type_t.DTP_BOOLEAN, "with_header_filters", 0, true, "Per-column filter box in the table header, on the columns a text/number match means something (not hooks, fkeys or json)"),
+	(0, _yuneta_gobj_js.SDATA)(_yuneta_gobj_js.data_type_t.DTP_BOOLEAN, "with_inline_edit", 0, true, "Edit a writable scalar cell in place while in edition mode (the record form keeps the rest)"),
 	(0, _yuneta_gobj_js.SDATA)(_yuneta_gobj_js.data_type_t.DTP_BOOLEAN, "with_in_row_edit_icons", 0, true, "Add a last column with internal EDIT/DELETE icon"),
 	(0, _yuneta_gobj_js.SDATA)(_yuneta_gobj_js.data_type_t.DTP_BOOLEAN, "editable", 0, false, "Edit state"),
 	(0, _yuneta_gobj_js.SDATA)(_yuneta_gobj_js.data_type_t.DTP_BOOLEAN, "with_checkbox", 0, true, "Auxiliary first column to select rows"),
@@ -16765,6 +16867,7 @@ function create_tabulator(gobj) {
 	let table_id = (0, _yuneta_gobj_js.gobj_read_str_attr)(gobj, "table_id");
 	let desc = (0, _yuneta_gobj_js.gobj_read_attr)(gobj, "desc");
 	let with_header_filters = (0, _yuneta_gobj_js.gobj_read_bool_attr)(gobj, "with_header_filters");
+	let with_inline_edit = (0, _yuneta_gobj_js.gobj_read_bool_attr)(gobj, "with_inline_edit");
 	let columns = [];
 	let with_checkbox = (0, _yuneta_gobj_js.gobj_read_bool_attr)(gobj, "with_checkbox");
 	let with_radio = (0, _yuneta_gobj_js.gobj_read_bool_attr)(gobj, "with_radio");
@@ -16808,6 +16911,42 @@ function create_tabulator(gobj) {
 			col_id: cell.getField()
 		}, gobj);
 	}
+	const INLINE_EDITABLE = [
+		"string",
+		"integer",
+		"real",
+		"email",
+		"url",
+		"tel",
+		"id",
+		"hex",
+		"currency",
+		"percent",
+		"uuid"
+	];
+	function inline_editor_for(colDef, col, field_desc) {
+		if (!field_desc.is_writable) return;
+		if (col.id === (desc.pkey || "id")) return;
+		switch (field_desc.type) {
+			case "boolean":
+				colDef.editor = "tickCross";
+				break;
+			case "enum":
+				colDef.editor = "list";
+				colDef.editorParams = { values: field_desc.enum_list || [] };
+				break;
+			case "integer":
+			case "real":
+				colDef.editor = "number";
+				break;
+			default:
+				if (!INLINE_EDITABLE.includes(field_desc.type)) return;
+				colDef.editor = "input";
+		}
+		colDef.editable = function() {
+			return (0, _yuneta_gobj_js.gobj_read_bool_attr)(gobj, "editable") && !(0, _yuneta_gobj_js.gobj_read_bool_attr)(gobj, "readonly");
+		};
+	}
 	const FILTERABLE = [
 		"string",
 		"integer",
@@ -16930,6 +17069,7 @@ function create_tabulator(gobj) {
 			formatter: colFormatter
 		};
 		if (with_header_filters) apply_header_filter(colDef, field_desc);
+		if (with_inline_edit) inline_editor_for(colDef, col, field_desc);
 		if (vertAlign) colDef.vertAlign = vertAlign;
 		if (formatterParams) colDef.formatterParams = formatterParams;
 		if (cellClick) colDef.cellClick = cellClick;
@@ -17016,6 +17156,21 @@ function create_tabulator(gobj) {
 	});
 	tabulator.on("dataProcessed", update_rowcount);
 	tabulator.on("dataChanged", update_rowcount);
+	tabulator.on("cellEdited", function(cell) {
+		let pkey = ((0, _yuneta_gobj_js.gobj_read_attr)(gobj, "desc") || {}).pkey || "id";
+		let data = cell.getRow().getData();
+		let id = data ? data[pkey] : void 0;
+		if (id === void 0 || id === null) {
+			(0, _yuneta_gobj_js.log_error)(`${(0, _yuneta_gobj_js.gobj_short_name)(gobj)}: edited a row with no '${pkey}'`);
+			cell.restoreOldValue();
+			return;
+		}
+		(0, _yuneta_gobj_js.gobj_send_event)(gobj, "EV_CELL_EDITED", {
+			id,
+			field: cell.getField(),
+			value: cell.getValue()
+		}, gobj);
+	});
 	tabulator.on("dataFiltered", function(filters, rows) {
 		update_rowcount(Array.isArray(rows) ? rows.length : void 0);
 	});
@@ -18136,6 +18291,20 @@ function ac_show_cell_json(gobj, event, kw, src) {
 	return 0;
 }
 /************************************************************
+*  One cell of one record changed. Publish it UP as a field write;
+*  the host owns the treedb command and its options.
+************************************************************/
+function ac_cell_edited(gobj, event, kw, src) {
+	if (refuse_if_readonly$2(gobj, event)) return -1;
+	(0, _yuneta_gobj_js.gobj_publish_event)(gobj, "EV_UPDATE_FIELD", {
+		topic_name: (0, _yuneta_gobj_js.gobj_read_str_attr)(gobj, "topic_name"),
+		id: kw.id,
+		field: kw.field,
+		value: kw.value
+	});
+	return 0;
+}
+/************************************************************
 *  Show the schema (desc) of this topic
 ************************************************************/
 function ac_show_schema(gobj, event, kw, src) {
@@ -18385,6 +18554,11 @@ function create_gclass$5(gclass_name) {
 			ac_show_schema,
 			null
 		],
+		[
+			"EV_CELL_EDITED",
+			ac_cell_edited,
+			null
+		],
 		[
 			"EV_SEARCH",
 			ac_search,
@@ -18435,6 +18609,8 @@ function create_gclass$5(gclass_name) {
 		["EV_CHANGE_LOCALE", 0],
 		["EV_REFRESH", 0],
 		["EV_SHOW_SCHEMA", 0],
+		["EV_CELL_EDITED", 0],
+		["EV_UPDATE_FIELD", _yuneta_gobj_js.event_flag_t.EVF_OUTPUT_EVENT],
 		["EV_SEARCH", 0],
 		["EV_OPEN_COLUMNS", 0],
 		["EV_TOGGLE_COLUMN", 0],

package/dist/gobj-ui.es.js

@@ -14411,6 +14411,66 @@ function yui_shell_show_route_map(shell, opts) {
 	return modal_ref.modal;
 }
 //#endregion
+//#region src/nodes_answer.js
+/***********************************************************************
+*          nodes_answer.js
+*
+*      What the `nodes` command answered, whatever shape it came in.
+*
+*      A treedb lives in memory on the backend, so walking it is not
+*      what costs: serializing every node, pushing it through a
+*      websocket and parsing it here is. So `nodes` learned to answer a
+*      PAGE, with the contract `list-keys` of C_TRANGER already used:
+*
+*          no `limit`  ->  the plain list it has always answered
+*          a `limit`   ->  {total_rows, pages, data}
+*
+*      Both shapes are alive at once and will be for a long time: the
+*      SPA talks to backends the operator configures, and an older one
+*      answers the plain list whatever this app asks for. So the shape
+*      is read, never assumed — a view that indexed `data` as an array
+*      would render an object's keys as rows the day a backend
+*      upgraded, which is the kind of break that looks like a bug in
+*      the app.
+*
+*      Pure and tested, because this is exactly the sort of thing that
+*      cannot be exercised until the other side moves.
+*
+*          Copyright (c) 2026, ArtGins.
+*          All Rights Reserved.
+***********************************************************************/
+/***************************************************************
+*  nodes_answer(data) -> {rows, total, pages, paged}
+*
+*  `rows` is always an array, so a caller never has to ask which
+*  shape it got. `paged` says whether the backend answered a page,
+*  which is the one thing a caller may legitimately care about (it is
+*  what tells a pager it has something to page).
+***************************************************************/
+function nodes_answer(data) {
+	if (Array.isArray(data)) return {
+		rows: data,
+		total: data.length,
+		pages: 1,
+		paged: false
+	};
+	if (data && typeof data === "object" && Array.isArray(data.data)) {
+		let rows = data.data;
+		return {
+			rows,
+			total: typeof data.total_rows === "number" ? data.total_rows : rows.length,
+			pages: typeof data.pages === "number" && data.pages > 0 ? data.pages : 1,
+			paged: true
+		};
+	}
+	return {
+		rows: [],
+		total: 0,
+		pages: 1,
+		paged: false
+	};
+}
+//#endregion
 //#region src/c_yui_treedb_topics.js
 /***********************************************************************
 *          c_yui_treedb_topics.js
@@ -15604,7 +15664,13 @@ function ac_mt_command_answer$2(gobj, event, kw, src) {
 	}
 	if (result < 0) {
 		if (command === "descs") show_load_error$1(gobj, t(comment));
-		else yui_shell_show_error(yui_shell_of(gobj), comment, { t });
+		else {
+			yui_shell_show_error(yui_shell_of(gobj), comment, { t });
+			if (command === "update-node" || command === "create-node") {
+				let failed_topic = kw_get_str(gobj, kw_command, "topic_name", "", 0);
+				if (failed_topic) treedb_nodes$1(gobj, gobj_read_str_attr(gobj, "treedb_name"), failed_topic, { list_dict: true });
+			}
+		}
 		return 0;
 	}
 	switch (command) {
@@ -15618,7 +15684,7 @@ function ac_mt_command_answer$2(gobj, event, kw, src) {
 			let topic_name = kw_get_str(gobj, kw_command, "topic_name", "", kw_flag_t.KW_REQUIRED);
 			if (result >= 0) {
 				let gobj_topic_form = gobj_find_child(gobj, { __gobj_name__: `${gobj_name(gobj)}?${topic_name}` });
-				gobj_send_event(gobj_topic_form, "EV_LOAD_NODES", data, gobj);
+				gobj_send_event(gobj_topic_form, "EV_LOAD_NODES", nodes_answer(data).rows, gobj);
 			}
 			break;
 		case "create-node":
@@ -15844,6 +15910,35 @@ function ac_create_record(gobj, event, kw, src) {
 	});
 }
 /********************************************
+*  Event from formtable: ONE field of one record, edited in place.
+*
+*  Written as a PARTIAL update and, deliberately, with NO `autolink`.
+*  `treedb_update_node()` merges (`json_object_update`), so the fields
+*  it does not carry are left alone; and `autolink` is what wipes a
+*  node's links to rebuild them from the fkeys the record carries — on
+*  a partial record that reads as "no parents" and DETACHES the node,
+*  answering success. A cell edit changes a scalar and must not be able
+*  to touch a link at all.
+*
+*  This is why it is its own event and not EV_UPDATE_RECORD: that one
+*  DOES send autolink, and may, because the form hands it the whole
+*  record with its fkeys in it.
+********************************************/
+function ac_update_field(gobj, event, kw, src) {
+	if (refuse_if_readonly$3(gobj, event)) return -1;
+	let topic_name = kw.topic_name || gobj_read_attr(src, "topic_name");
+	let desc = (gobj_read_attr(gobj, "descs") || {})[topic_name];
+	let pkey = desc && desc.pkey || "id";
+	if (kw.id === void 0 || kw.id === null || !kw.field) {
+		log_error(`${gobj_short_name(gobj)}: a field write with no id or no field`);
+		return -1;
+	}
+	let record = {};
+	record[pkey] = kw.id;
+	record[kw.field] = kw.value;
+	return treedb_update_node$1(gobj, gobj_read_str_attr(gobj, "treedb_name"), topic_name, record, { list_dict: true });
+}
+/********************************************
 *  Event from formtable
 *  kw: {
 *      topic_name,
@@ -15965,6 +16060,11 @@ function create_gclass$6(gclass_name) {
 			ac_update_record,
 			null
 		],
+		[
+			"EV_UPDATE_FIELD",
+			ac_update_field,
+			null
+		],
 		[
 			"EV_DELETE_RECORD",
 			ac_delete_record,
@@ -16038,6 +16138,7 @@ function create_gclass$6(gclass_name) {
 		["EV_TREEDB_NODE_DELETED", event_flag_t.EVF_PUBLIC_EVENT],
 		["EV_CREATE_RECORD", 0],
 		["EV_UPDATE_RECORD", 0],
+		["EV_UPDATE_FIELD", 0],
 		["EV_DELETE_RECORD", 0],
 		["EV_RECORD_WRITTEN", event_flag_t.EVF_OUTPUT_EVENT | event_flag_t.EVF_NO_WARN_SUBS],
 		["EV_REFRESH_TOPIC", 0],
@@ -16251,6 +16352,7 @@ var attrs_table$5 = [
 	SDATA(data_type_t.DTP_BOOLEAN, "with_columns_button", 0, true, "Button toolbar COLUMNS (choose which columns the table shows)"),
 	SDATA(data_type_t.DTP_BOOLEAN, "with_export_button", 0, true, "Button toolbar EXPORT (download as CSV what the table holds)"),
 	SDATA(data_type_t.DTP_BOOLEAN, "with_header_filters", 0, true, "Per-column filter box in the table header, on the columns a text/number match means something (not hooks, fkeys or json)"),
+	SDATA(data_type_t.DTP_BOOLEAN, "with_inline_edit", 0, true, "Edit a writable scalar cell in place while in edition mode (the record form keeps the rest)"),
 	SDATA(data_type_t.DTP_BOOLEAN, "with_in_row_edit_icons", 0, true, "Add a last column with internal EDIT/DELETE icon"),
 	SDATA(data_type_t.DTP_BOOLEAN, "editable", 0, false, "Edit state"),
 	SDATA(data_type_t.DTP_BOOLEAN, "with_checkbox", 0, true, "Auxiliary first column to select rows"),
@@ -16760,6 +16862,7 @@ function create_tabulator(gobj) {
 	let table_id = gobj_read_str_attr(gobj, "table_id");
 	let desc = gobj_read_attr(gobj, "desc");
 	let with_header_filters = gobj_read_bool_attr(gobj, "with_header_filters");
+	let with_inline_edit = gobj_read_bool_attr(gobj, "with_inline_edit");
 	let columns = [];
 	let with_checkbox = gobj_read_bool_attr(gobj, "with_checkbox");
 	let with_radio = gobj_read_bool_attr(gobj, "with_radio");
@@ -16803,6 +16906,42 @@ function create_tabulator(gobj) {
 			col_id: cell.getField()
 		}, gobj);
 	}
+	const INLINE_EDITABLE = [
+		"string",
+		"integer",
+		"real",
+		"email",
+		"url",
+		"tel",
+		"id",
+		"hex",
+		"currency",
+		"percent",
+		"uuid"
+	];
+	function inline_editor_for(colDef, col, field_desc) {
+		if (!field_desc.is_writable) return;
+		if (col.id === (desc.pkey || "id")) return;
+		switch (field_desc.type) {
+			case "boolean":
+				colDef.editor = "tickCross";
+				break;
+			case "enum":
+				colDef.editor = "list";
+				colDef.editorParams = { values: field_desc.enum_list || [] };
+				break;
+			case "integer":
+			case "real":
+				colDef.editor = "number";
+				break;
+			default:
+				if (!INLINE_EDITABLE.includes(field_desc.type)) return;
+				colDef.editor = "input";
+		}
+		colDef.editable = function() {
+			return gobj_read_bool_attr(gobj, "editable") && !gobj_read_bool_attr(gobj, "readonly");
+		};
+	}
 	const FILTERABLE = [
 		"string",
 		"integer",
@@ -16925,6 +17064,7 @@ function create_tabulator(gobj) {
 			formatter: colFormatter
 		};
 		if (with_header_filters) apply_header_filter(colDef, field_desc);
+		if (with_inline_edit) inline_editor_for(colDef, col, field_desc);
 		if (vertAlign) colDef.vertAlign = vertAlign;
 		if (formatterParams) colDef.formatterParams = formatterParams;
 		if (cellClick) colDef.cellClick = cellClick;
@@ -17011,6 +17151,21 @@ function create_tabulator(gobj) {
 	});
 	tabulator.on("dataProcessed", update_rowcount);
 	tabulator.on("dataChanged", update_rowcount);
+	tabulator.on("cellEdited", function(cell) {
+		let pkey = (gobj_read_attr(gobj, "desc") || {}).pkey || "id";
+		let data = cell.getRow().getData();
+		let id = data ? data[pkey] : void 0;
+		if (id === void 0 || id === null) {
+			log_error(`${gobj_short_name(gobj)}: edited a row with no '${pkey}'`);
+			cell.restoreOldValue();
+			return;
+		}
+		gobj_send_event(gobj, "EV_CELL_EDITED", {
+			id,
+			field: cell.getField(),
+			value: cell.getValue()
+		}, gobj);
+	});
 	tabulator.on("dataFiltered", function(filters, rows) {
 		update_rowcount(Array.isArray(rows) ? rows.length : void 0);
 	});
@@ -18131,6 +18286,20 @@ function ac_show_cell_json(gobj, event, kw, src) {
 	return 0;
 }
 /************************************************************
+*  One cell of one record changed. Publish it UP as a field write;
+*  the host owns the treedb command and its options.
+************************************************************/
+function ac_cell_edited(gobj, event, kw, src) {
+	if (refuse_if_readonly$2(gobj, event)) return -1;
+	gobj_publish_event(gobj, "EV_UPDATE_FIELD", {
+		topic_name: gobj_read_str_attr(gobj, "topic_name"),
+		id: kw.id,
+		field: kw.field,
+		value: kw.value
+	});
+	return 0;
+}
+/************************************************************
 *  Show the schema (desc) of this topic
 ************************************************************/
 function ac_show_schema(gobj, event, kw, src) {
@@ -18380,6 +18549,11 @@ function create_gclass$5(gclass_name) {
 			ac_show_schema,
 			null
 		],
+		[
+			"EV_CELL_EDITED",
+			ac_cell_edited,
+			null
+		],
 		[
 			"EV_SEARCH",
 			ac_search,
@@ -18430,6 +18604,8 @@ function create_gclass$5(gclass_name) {
 		["EV_CHANGE_LOCALE", 0],
 		["EV_REFRESH", 0],
 		["EV_SHOW_SCHEMA", 0],
+		["EV_CELL_EDITED", 0],
+		["EV_UPDATE_FIELD", event_flag_t.EVF_OUTPUT_EVENT],
 		["EV_SEARCH", 0],
 		["EV_OPEN_COLUMNS", 0],
 		["EV_TOGGLE_COLUMN", 0],

package/package.json

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

package/src/c_yui_treedb_topic_with_form.js

@@ -111,6 +111,7 @@ SDATA(data_type_t.DTP_BOOLEAN,  "with_schema_button",        0,  true,   "Button
 SDATA(data_type_t.DTP_BOOLEAN,  "with_columns_button",       0,  true,   "Button toolbar COLUMNS (choose which columns the table shows)"),
 SDATA(data_type_t.DTP_BOOLEAN,  "with_export_button",        0,  true,   "Button toolbar EXPORT (download as CSV what the table holds)"),
 SDATA(data_type_t.DTP_BOOLEAN,  "with_header_filters",       0,  true,   "Per-column filter box in the table header, on the columns a text/number match means something (not hooks, fkeys or json)"),
+SDATA(data_type_t.DTP_BOOLEAN,  "with_inline_edit",          0,  true,   "Edit a writable scalar cell in place while in edition mode (the record form keeps the rest)"),
 SDATA(data_type_t.DTP_BOOLEAN,  "with_in_row_edit_icons",    0,  true,   "Add a last column with internal EDIT/DELETE icon"),
 
 SDATA(data_type_t.DTP_BOOLEAN,  "editable",             0,  false,  "Edit state"),
@@ -730,6 +731,7 @@ function create_tabulator(gobj)
     let table_id = gobj_read_str_attr(gobj, "table_id");
     let desc = gobj_read_attr(gobj, "desc");
     let with_header_filters = gobj_read_bool_attr(gobj, "with_header_filters");
+    let with_inline_edit = gobj_read_bool_attr(gobj, "with_inline_edit");
 
     let columns = [];
 
@@ -819,6 +821,62 @@ function create_tabulator(gobj)
      *  value is stringified first because a fkey can arrive as a ref string,
      *  a list of them or a dict.
      */
+    /*  A cell you can type into, and the ones you cannot.
+     *
+     *  Editing in place is for a SCALAR the table can round-trip. A hook
+     *  holds children and an fkey IS a link — both are edited by linking,
+     *  not by typing — a dict or a list is a document (the form has an
+     *  editor for it), and a date cell shows a formatted string over an
+     *  epoch, so typing into it would write the string. Those stay with
+     *  the form, which is one click away on the same row.
+     *
+     *  The schema decides the rest: only a column flagged `writable` is
+     *  offered, and never the pkey — renaming what a record is KEYED by
+     *  is not a field edit.
+     */
+    const INLINE_EDITABLE = [
+        "string", "integer", "real",
+        "email", "url", "tel", "id", "hex", "currency", "percent", "uuid"
+    ];
+
+    function inline_editor_for(colDef, col, field_desc)
+    {
+        if(!field_desc.is_writable) {
+            return;
+        }
+        if(col.id === (desc.pkey || "id")) {
+            return;
+        }
+
+        switch(field_desc.type) {
+            case "boolean":
+                colDef.editor = "tickCross";
+                break;
+            case "enum":
+                colDef.editor = "list";
+                colDef.editorParams = {values: field_desc.enum_list || []};
+                break;
+            case "integer":
+            case "real":
+                colDef.editor = "number";
+                break;
+            default:
+                if(!INLINE_EDITABLE.includes(field_desc.type)) {
+                    return;
+                }
+                colDef.editor = "input";
+                break;
+        }
+
+        /*  A FUNCTION, not a flag: edition mode is toggled on a table that
+         *  is already built (ac_edition_mode only shows and hides columns),
+         *  so the answer has to be asked for at the moment of the click. */
+        colDef.editable = function() {
+            return gobj_read_bool_attr(gobj, "editable") &&
+                   !gobj_read_bool_attr(gobj, "readonly");
+        };
+    }
+
     const FILTERABLE = [
         "string", "integer", "real", "rowid", "uuid", "qualified",
         "email", "url", "tel", "id", "hex", "currency", "percent"
@@ -974,6 +1032,9 @@ function create_tabulator(gobj)
         if(with_header_filters) {
             apply_header_filter(colDef, field_desc);
         }
+        if(with_inline_edit) {
+            inline_editor_for(colDef, col, field_desc);
+        }
         if(vertAlign) {
             colDef.vertAlign = vertAlign;
         }
@@ -1139,6 +1200,33 @@ function create_tabulator(gobj)
      *  box; a filter per column made it a claim you read on every keystroke. */
     tabulator.on("dataProcessed", update_rowcount);
     tabulator.on("dataChanged", update_rowcount);
+    /*  A cell edited in place. Announced as ONE FIELD of one record — not
+     *  as the row — and that is the whole safety of it: the host writes it
+     *  with a partial update and no `autolink`, so nothing but that field
+     *  moves. `autolink` wipes a node's links and rebuilds them from the
+     *  fkeys the record carries, so sending a row that the table shaped for
+     *  DISPLAY would unlink the node and answer success.  */
+    tabulator.on("cellEdited", function(cell) {
+        let pkey = (gobj_read_attr(gobj, "desc") || {}).pkey || "id";
+        let data = cell.getRow().getData();
+        let id = data ? data[pkey] : undefined;
+        if(id === undefined || id === null) {
+            log_error(`${gobj_short_name(gobj)}: edited a row with no '${pkey}'`);
+            cell.restoreOldValue();
+            return;
+        }
+        gobj_send_event(
+            gobj,
+            "EV_CELL_EDITED",
+            {
+                id:    id,
+                field: cell.getField(),
+                value: cell.getValue()
+            },
+            gobj
+        );
+    });
+
     tabulator.on("dataFiltered", function(filters, rows) {
         update_rowcount(Array.isArray(rows)? rows.length : undefined);
     });
@@ -2845,6 +2933,28 @@ function ac_show_cell_json(gobj, event, kw, src)
     return 0;
 }
 
+/************************************************************
+ *  One cell of one record changed. Publish it UP as a field write;
+ *  the host owns the treedb command and its options.
+ ************************************************************/
+function ac_cell_edited(gobj, event, kw, src)
+{
+    if(refuse_if_readonly(gobj, event)) {
+        return -1;      /*  Error already logged  */
+    }
+    gobj_publish_event(
+        gobj,
+        "EV_UPDATE_FIELD",
+        {
+            topic_name: gobj_read_str_attr(gobj, "topic_name"),
+            id:         kw.id,
+            field:      kw.field,
+            value:      kw.value
+        }
+    );
+    return 0;
+}
+
 /************************************************************
  *  Show the schema (desc) of this topic
  ************************************************************/
@@ -3076,6 +3186,7 @@ function create_gclass(gclass_name)
             ["EV_CHANGE_LOCALE",        ac_change_locale,      null],
             ["EV_REFRESH",              ac_refresh,            null],
             ["EV_SHOW_SCHEMA",          ac_show_schema,        null],
+            ["EV_CELL_EDITED",          ac_cell_edited,        null],
             ["EV_SEARCH",               ac_search,             null],
             ["EV_OPEN_COLUMNS",         ac_open_columns,       null],
             ["EV_TOGGLE_COLUMN",        ac_toggle_column,      null],
@@ -3108,6 +3219,8 @@ function create_gclass(gclass_name)
         ["EV_CHANGE_LOCALE",        0],
         ["EV_REFRESH",              0],
         ["EV_SHOW_SCHEMA",          0],
+        ["EV_CELL_EDITED",          0],
+        ["EV_UPDATE_FIELD",         event_flag_t.EVF_OUTPUT_EVENT],
         ["EV_SEARCH",               0],
         ["EV_OPEN_COLUMNS",         0],
         ["EV_TOGGLE_COLUMN",        0],

package/src/c_yui_treedb_topics.js

@@ -58,6 +58,7 @@ import "./c_yui_treedb_topics.css";
 
 import {yui_shell_show_error, yui_shell_show_modal, yui_shell_popup_layer} from "./shell_modals.js";
 import {yui_shell_of, yui_shell_set_sub_routes} from "./c_yui_shell.js";
+import {nodes_answer} from "./nodes_answer.js";
 
 import {t} from "i18next";
 
@@ -1740,6 +1741,18 @@ function ac_mt_command_answer(gobj, event, kw, src)
             show_load_error(gobj, t(comment));
         } else {
             yui_shell_show_error(yui_shell_of(gobj), comment, {t: t});
+            /*  A refused write leaves the TABLE showing what the operator
+             *  typed and the store holding what it had. That is tolerable
+             *  for a form (it stays open on the values it failed with) and
+             *  is not for a cell edited in place, which looks saved. Put the
+             *  topic back to what the treedb actually has. */
+            if(command === "update-node" || command === "create-node") {
+                let failed_topic = kw_get_str(gobj, kw_command, "topic_name", "", 0);
+                if(failed_topic) {
+                    treedb_nodes(gobj, gobj_read_str_attr(gobj, "treedb_name"),
+                                 failed_topic, {list_dict: true});
+                }
+            }
         }
         return 0;
     }
@@ -1758,10 +1771,18 @@ function ac_mt_command_answer(gobj, event, kw, src)
                 let gobj_topic_form = gobj_find_child(gobj, {
                     __gobj_name__: `${gobj_name(gobj)}?${topic_name}`
                 });
+                /*  `nodes` answers a plain list, or — when asked for a page —
+                 *  the {total_rows, pages, data} envelope. Both shapes are
+                 *  alive at once and will be for a long time: this SPA talks
+                 *  to backends the operator configures, and an older one
+                 *  answers the plain list whatever is asked of it. Reading
+                 *  the shape rather than assuming it is what keeps the table
+                 *  from rendering an object's keys as rows the day a backend
+                 *  upgrades. */
                 gobj_send_event(
                     gobj_topic_form,
                     "EV_LOAD_NODES",
-                    data,
+                    nodes_answer(data).rows,
                     gobj
                 );
             }
@@ -2126,6 +2147,49 @@ function ac_create_record(gobj, event, kw, src)
     );
 }
 
+/********************************************
+ *  Event from formtable: ONE field of one record, edited in place.
+ *
+ *  Written as a PARTIAL update and, deliberately, with NO `autolink`.
+ *  `treedb_update_node()` merges (`json_object_update`), so the fields
+ *  it does not carry are left alone; and `autolink` is what wipes a
+ *  node's links to rebuild them from the fkeys the record carries — on
+ *  a partial record that reads as "no parents" and DETACHES the node,
+ *  answering success. A cell edit changes a scalar and must not be able
+ *  to touch a link at all.
+ *
+ *  This is why it is its own event and not EV_UPDATE_RECORD: that one
+ *  DOES send autolink, and may, because the form hands it the whole
+ *  record with its fkeys in it.
+ ********************************************/
+function ac_update_field(gobj, event, kw, src)
+{
+    if(refuse_if_readonly(gobj, event)) {
+        return -1;      /*  Error already logged  */
+    }
+    let topic_name = kw.topic_name || gobj_read_attr(src, "topic_name");
+    let descs = gobj_read_attr(gobj, "descs") || {};
+    let desc = descs[topic_name];
+    let pkey = (desc && desc.pkey) || "id";
+
+    if(kw.id === undefined || kw.id === null || !kw.field) {
+        log_error(`${gobj_short_name(gobj)}: a field write with no id or no field`);
+        return -1;
+    }
+
+    let record = {};
+    record[pkey] = kw.id;
+    record[kw.field] = kw.value;
+
+    return treedb_update_node(
+        gobj,
+        gobj_read_str_attr(gobj, "treedb_name"),
+        topic_name,
+        record,
+        {list_dict: true}
+    );
+}
+
 /********************************************
  *  Event from formtable
  *  kw: {
@@ -2292,6 +2356,7 @@ function create_gclass(gclass_name)
             ["EV_TREEDB_NODE_DELETED",  ac_treedb_node_deleted,     null],
             ["EV_CREATE_RECORD",        ac_create_record,           null],
             ["EV_UPDATE_RECORD",        ac_update_record,           null],
+            ["EV_UPDATE_FIELD",         ac_update_field,            null],
             ["EV_DELETE_RECORD",        ac_delete_record,           null],
             ["EV_REFRESH_TOPIC",        ac_refresh_topic,           null],
             ["EV_OPEN_JSON",            ac_open_json,               null],
@@ -2318,6 +2383,7 @@ function create_gclass(gclass_name)
         ["EV_TREEDB_NODE_DELETED",  event_flag_t.EVF_PUBLIC_EVENT],
         ["EV_CREATE_RECORD",        0],
         ["EV_UPDATE_RECORD",        0],
+        ["EV_UPDATE_FIELD",         0],
         ["EV_DELETE_RECORD",        0],
         ["EV_RECORD_WRITTEN",       event_flag_t.EVF_OUTPUT_EVENT|
                                     event_flag_t.EVF_NO_WARN_SUBS],

package/src/nodes_answer.js

@@ -0,0 +1,72 @@
+/***********************************************************************
+ *          nodes_answer.js
+ *
+ *      What the `nodes` command answered, whatever shape it came in.
+ *
+ *      A treedb lives in memory on the backend, so walking it is not
+ *      what costs: serializing every node, pushing it through a
+ *      websocket and parsing it here is. So `nodes` learned to answer a
+ *      PAGE, with the contract `list-keys` of C_TRANGER already used:
+ *
+ *          no `limit`  ->  the plain list it has always answered
+ *          a `limit`   ->  {total_rows, pages, data}
+ *
+ *      Both shapes are alive at once and will be for a long time: the
+ *      SPA talks to backends the operator configures, and an older one
+ *      answers the plain list whatever this app asks for. So the shape
+ *      is read, never assumed — a view that indexed `data` as an array
+ *      would render an object's keys as rows the day a backend
+ *      upgraded, which is the kind of break that looks like a bug in
+ *      the app.
+ *
+ *      Pure and tested, because this is exactly the sort of thing that
+ *      cannot be exercised until the other side moves.
+ *
+ *          Copyright (c) 2026, ArtGins.
+ *          All Rights Reserved.
+ ***********************************************************************/
+
+/***************************************************************
+ *  nodes_answer(data) -> {rows, total, pages, paged}
+ *
+ *  `rows` is always an array, so a caller never has to ask which
+ *  shape it got. `paged` says whether the backend answered a page,
+ *  which is the one thing a caller may legitimately care about (it is
+ *  what tells a pager it has something to page).
+ ***************************************************************/
+function nodes_answer(data)
+{
+    if(Array.isArray(data)) {
+        return {
+            rows:  data,
+            total: data.length,
+            pages: 1,
+            paged: false
+        };
+    }
+
+    if(data && typeof data === "object" && Array.isArray(data.data)) {
+        let rows = data.data;
+        let total = (typeof data.total_rows === "number")? data.total_rows : rows.length;
+        let pages = (typeof data.pages === "number" && data.pages > 0)? data.pages : 1;
+        return {
+            rows:  rows,
+            total: total,
+            pages: pages,
+            paged: true
+        };
+    }
+
+    /*  Anything else — null, an error body, a shape nobody has seen — is
+     *  no rows. The caller renders an empty table, which is honest, and
+     *  the command answer's own `result` is what says something failed. */
+    return {
+        rows:  [],
+        total: 0,
+        pages: 1,
+        paged: false
+    };
+}
+
+
+export {nodes_answer};

package/src/nodes_answer.test.js

@@ -0,0 +1,50 @@
+import {describe, it, expect} from "vitest";
+import {nodes_answer} from "./nodes_answer.js";
+
+describe("nodes_answer", () => {
+    it("reads the plain list an unpaged backend answers", () => {
+        let a = nodes_answer([{id: "a"}, {id: "b"}]);
+        expect(a.rows.length).toBe(2);
+        expect(a.total).toBe(2);
+        expect(a.pages).toBe(1);
+        expect(a.paged).toBe(false);
+    });
+
+    it("reads the envelope a paged backend answers", () => {
+        let a = nodes_answer({total_rows: 120, pages: 3, data: [{id: "a"}]});
+        expect(a.rows.length).toBe(1);
+        expect(a.total).toBe(120);
+        expect(a.pages).toBe(3);
+        expect(a.paged).toBe(true);
+    });
+
+    it("an empty page still reports the true total", () => {
+        let a = nodes_answer({total_rows: 120, pages: 3, data: []});
+        expect(a.rows).toEqual([]);
+        expect(a.total).toBe(120);
+        expect(a.paged).toBe(true);
+    });
+
+    it("an empty list is not a page", () => {
+        let a = nodes_answer([]);
+        expect(a.rows).toEqual([]);
+        expect(a.total).toBe(0);
+        expect(a.paged).toBe(false);
+    });
+
+    it("falls back to the row count when the envelope omits its totals", () => {
+        let a = nodes_answer({data: [{id: "a"}, {id: "b"}]});
+        expect(a.total).toBe(2);
+        expect(a.pages).toBe(1);
+        expect(a.paged).toBe(true);
+    });
+
+    it("gives no rows for anything it does not recognise", () => {
+        for(let bad of [null, undefined, 0, "x", {}, {data: "nope"}]) {
+            let a = nodes_answer(bad);
+            expect(a.rows).toEqual([]);
+            expect(a.total).toBe(0);
+            expect(a.paged).toBe(false);
+        }
+    });
+});