@odoo/o-spreadsheet 19.3.16 → 19.3.18

package/dist/o_spreadsheet.cjs

@@ -2,9 +2,9 @@
 /**
  * This file is generated by o-spreadsheet build tools. Do not edit it.
  * @see https://github.com/odoo/o-spreadsheet
- * @version 19.3.16
- * @date 2026-08-12T09:15:31.627Z
- * @hash 464be42
+ * @version 19.3.18
+ * @date 2026-08-29T09:16:28.272Z
+ * @hash 1f4b1ae
  */
 
 Object.defineProperty(exports, Symbol.toStringTag, { value: 'Module' });
@@ -4070,6 +4070,18 @@ function toString(data) {
 		default: return "";
 	}
 }
+/**
+* Only converts the decimal separator, ignore number format such as % or dates
+*/
+function toLocaleString(data, locale) {
+	const value = toValue(data);
+	switch (typeof value) {
+		case "string": return value;
+		case "number": return value.toString().replace(".", locale.decimalSeparator);
+		case "boolean": return value ? "TRUE" : "FALSE";
+		default: return "";
+	}
+}
 /** Normalize string by setting it to lowercase and replacing accent letters with plain letters */
 const normalizeString = memoize(function normalizeString(str) {
 	return str.toLowerCase().normalize("NFD").replace(/[\u0300-\u036f]/g, "");
@@ -10474,6 +10486,7 @@ function getRadarChartDatasets(definition, args) {
 			hidden,
 			borderColor,
 			backgroundColor: borderColor,
+			pointBackgroundColor: borderColor,
 			pointRadius: definition.hideDataMarkers ? 0 : 3
 		};
 		if (fill) {
@@ -29088,12 +29101,13 @@ var FilterMenu = class extends _odoo_owl.Component {
 	}
 	sortFilterZone(sortDirection) {
 		const filterPosition = this.props.filterPosition;
-		const tableZone = this.table?.range.zone;
+		const table = this.table;
+		const tableZone = table?.range.zone;
 		if (!filterPosition || !tableZone || tableZone.top === tableZone.bottom) return;
 		const sheetId = this.env.model.getters.getActiveSheetId();
 		const contentZone = {
 			...tableZone,
-			top: tableZone.top + 1
+			top: tableZone.top + table.config.numberOfHeaders
 		};
 		const sortAnchor = {
 			col: filterPosition.col,
@@ -49826,6 +49840,7 @@ var Squisher = class {
 	baseFormulaWasTransformed = false;
 	rangeToStringOptions = { doNotSimplifyRange: true };
 	baseNumber = void 0;
+	baseNumberFormat = void 0;
 	constructor(getters) {
 		this.getters = getters;
 	}
@@ -49860,6 +49875,11 @@ var Squisher = class {
 			this.baseFormulaWasTransformed = false;
 		}
 	}
+	/** Reset the base number literal chain, so the next literal number cannot be squished as an offset of it */
+	resetBaseNumber() {
+		this.baseNumber = void 0;
+		this.baseNumberFormat = void 0;
+	}
 	/**
 	* Takes a cell and squish their formulas against the previous one (in the previous call).
 	* We should call this method for each cell in the sheet, in order from top to bottom, then from left to right (all cells of a columns, for each columns left to right).
@@ -49898,8 +49918,9 @@ var Squisher = class {
 		if (typeof cell.parsedValue === "number" && cell.parsedValue % 1 === 0) {
 			this.resetBaseFormula();
 			const numberValue = cell.parsedValue;
-			if (this.baseNumber === void 0) {
+			if (this.baseNumber === void 0 || cell.format !== this.baseNumberFormat) {
 				this.baseNumber = numberValue;
+				this.baseNumberFormat = cell.format;
 				return cell.content;
 			}
 			const numberOffset = numberValue - this.baseNumber;
@@ -49910,7 +49931,7 @@ var Squisher = class {
 			}
 		}
 		this.resetBaseFormula();
-		this.baseNumber = void 0;
+		this.resetBaseNumber();
 		return cell.content;
 	}
 	squishCommand(command) {
@@ -49919,6 +49940,8 @@ var Squisher = class {
 			const squished = this.squish(cell, command.sheetId);
 			return typeof squished === "string" ? command.content : squished;
 		}
+		this.resetBaseFormula();
+		this.resetBaseNumber();
 		return command.content;
 	}
 	/**
@@ -50072,8 +50095,10 @@ var Unsquisher = class {
 				continue;
 			}
 			const current = command.content;
-			if (current === void 0 || current === null || current === "") strategy = "NOT_A_FORMULA";
-			else strategy = this.chooseStrategy(current, strategy, command.sheetId, getters);
+			if (current === void 0 || current === null || current === "") {
+				this.rebase();
+				strategy = "NOT_A_FORMULA";
+			} else strategy = this.chooseStrategy(current, strategy, command.sheetId, getters);
 			let targetPositions;
 			if (command.type === "SQUISHED_UPDATE_CELL" && "targetRange" in command) {
 				const [start, end] = command.targetRange.split(":");
@@ -50118,8 +50143,11 @@ var Unsquisher = class {
 		let strategy;
 		for (const { key } of keys) {
 			const current = squished[key];
-			if (current === void 0 || current === null || current === "") continue;
-			else strategy = this.chooseStrategy(current, strategy, sheetId, getters);
+			if (current === void 0 || current === null || current === "") {
+				this.rebase();
+				strategy = void 0;
+				continue;
+			} else strategy = this.chooseStrategy(current, strategy, sheetId, getters);
 			const parts = key.split(":");
 			const targetPositions = parts.length === 1 ? expandXc(key) : expandRange(parts[0], parts[1]);
 			yield* this.applyStrategy(strategy, targetPositions, current, sheetId, getters);
@@ -50165,6 +50193,9 @@ var Unsquisher = class {
 				case "FIRST_OFFSET":
 					strategy = "COMBINE_OFFSET";
 					break;
+				case "COMBINE_OFFSET":
+				case "OFFSET_NUMBER": break;
+				default: throw new Error(`Cannot unsquish an offset without a preceding base cell: ${strategy}`);
 			}
 		}
 		return strategy;
@@ -60947,6 +60978,7 @@ var Session = class extends EventBus {
 	transportService;
 	serverRevisionId;
 	commandSquisher;
+	shouldVerifySquish;
 	/**
 	* Positions of the others client.
 	*/
@@ -60981,13 +61013,15 @@ var Session = class extends EventBus {
 	* between all connected clients
 	* @param serverRevisionId
 	* @param commandSquisher used to squish and unsquish commands to reduce the size of messages sent to the server
+	* @param shouldVerifySquish either the commands should be unsquished and verified before being sent to the server
 	*/
-	constructor(revisions, transportService, serverRevisionId = DEFAULT_REVISION_ID, commandSquisher) {
+	constructor(revisions, transportService, serverRevisionId = DEFAULT_REVISION_ID, commandSquisher, shouldVerifySquish = true) {
 		super();
 		this.revisions = revisions;
 		this.transportService = transportService;
 		this.serverRevisionId = serverRevisionId;
 		this.commandSquisher = commandSquisher;
+		this.shouldVerifySquish = shouldVerifySquish;
 	}
 	canApplyOptimisticUpdate() {
 		return !this.waitingUndoRedoAck;
@@ -61243,10 +61277,20 @@ var Session = class extends EventBus {
 				this.revisions.rebase(revision.id);
 				revision = this.revisions.get(message.nextRevisionId);
 			}
-			message = {
+			const squishedCommands = this.commandSquisher.squish(revision.commands);
+			if (!squishedCommands.some((x) => x.type === "SQUISHED_UPDATE_CELL")) message = {
 				...message,
 				commands: revision.commands
 			};
+			else if (this.shouldVerifySquish && !deepEquals(revision.commands, this.commandSquisher.unsquish(squishedCommands))) message = {
+				...message,
+				commands: revision.commands,
+				squishedFailed: true
+			};
+			else message = {
+				...message,
+				commands: squishedCommands
+			};
 		}
 		if (this.isReplayingInitialRevisions) throw new Error(`Trying to send a new revision while replaying initial revision. This can lead to endless dispatches every time the spreadsheet is open.
       ${JSON.stringify(message)}`);
@@ -77399,7 +77443,7 @@ const ARRAYTOTEXT = {
 		else if (_format === 0) {
 			const rowSeparator = this.locale.decimalSeparator === "," ? "/" : ",";
 			return transposeMatrix(_array).flatMap((row) => row.map((value) => {
-				return isEvaluationError(value.value) ? value.value : toString(value);
+				return isEvaluationError(value.value) ? value.value : toLocaleString(value, this.locale);
 			})).join(rowSeparator);
 		} else return new EvaluationError(_t("Format must be 0 or 1"));
 	},
@@ -83160,7 +83204,7 @@ const CONCAT = {
 	description: _t("Concatenation of two values."),
 	args: [arg("value1 (string)", _t("The value to which value2 will be appended.")), arg("value2 (string)", _t("The value to append to value1."))],
 	compute: function(value1, value2) {
-		return toString(value1) + toString(value2);
+		return toLocaleString(value1, this.locale) + toLocaleString(value2, this.locale);
 	},
 	isExported: true
 };
@@ -84025,7 +84069,7 @@ const CLEAN = {
 	description: _t("Remove non-printable characters from a piece of text."),
 	args: [arg("text (string)", _t("The text whose non-printable characters are to be removed."))],
 	compute: function(text) {
-		const _text = toString(text);
+		const _text = toLocaleString(text, this.locale);
 		let cleanedStr = "";
 		for (const char of _text) if (char && char.charCodeAt(0) > 31) cleanedStr += char;
 		return cleanedStr;
@@ -84036,7 +84080,7 @@ const CONCATENATE = {
 	description: _t("Appends strings to one another."),
 	args: [arg("string (string, range<string>, repeating)", _t("String to append in sequence."))],
 	compute: function(...datas) {
-		return reduceAny(datas, (acc, a) => acc + toString(a), "");
+		return reduceAny(datas, (acc, a) => acc + toLocaleString(a, this.locale), "");
 	},
 	isExported: true
 };
@@ -84044,7 +84088,7 @@ const EXACT = {
 	description: _t("Tests whether two strings are identical."),
 	args: [arg("string1 (string)", _t("The first string to compare.")), arg("string2 (string)", _t("The second string to compare."))],
 	compute: function(string1, string2) {
-		return toString(string1) === toString(string2);
+		return toLocaleString(string1, this.locale) === toLocaleString(string2, this.locale);
 	},
 	isExported: true
 };
@@ -84056,8 +84100,8 @@ const FIND = {
 		arg(`starting_at (number, default=${DEFAULT_STARTING_AT})`, _t("The character within text_to_search at which to start the search."))
 	],
 	compute: function(searchFor, textToSearch, startingAt = { value: DEFAULT_STARTING_AT }) {
-		const _searchFor = toString(searchFor);
-		const _textToSearch = toString(textToSearch);
+		const _searchFor = toLocaleString(searchFor, this.locale);
+		const _textToSearch = toLocaleString(textToSearch, this.locale);
 		const _startingAt = toNumber(startingAt, this.locale);
 		if (_textToSearch === "") return new EvaluationError(_t("The text_to_search must be non-empty."));
 		if (_startingAt < 1) return new EvaluationError(_t("The starting_at (%s) must be greater than or equal to 1.", _startingAt));
@@ -84094,8 +84138,8 @@ const JOIN = {
 	description: _t("Concatenates elements of arrays with delimiter."),
 	args: [arg("delimiter (string)", _t("The character or string to place between each concatenated value.")), arg("value_or_array (string, range<string>, repeating)", _t("Value to be appended using delimiter."))],
 	compute: function(delimiter, ...valuesOrArrays) {
-		const _delimiter = toString(delimiter);
-		return reduceAny(valuesOrArrays, (acc, a) => (acc ? acc + _delimiter : "") + toString(a), "");
+		const _delimiter = toLocaleString(delimiter, this.locale);
+		return reduceAny(valuesOrArrays, (acc, a) => (acc ? acc + _delimiter : "") + toLocaleString(a, this.locale), "");
 	}
 };
 const LEFT = {
@@ -84104,7 +84148,7 @@ const LEFT = {
 	compute: function(text, ...args) {
 		const _numberOfCharacters = args.length ? toNumber(args[0], this.locale) : 1;
 		if (_numberOfCharacters < 0) return new EvaluationError(_t("The number_of_characters (%s) must be positive or null.", _numberOfCharacters));
-		return toString(text).substring(0, _numberOfCharacters);
+		return toLocaleString(text, this.locale).substring(0, _numberOfCharacters);
 	},
 	isExported: true
 };
@@ -84112,7 +84156,7 @@ const LEN = {
 	description: _t("Length of a string."),
 	args: [arg("text (string)", _t("The string whose length will be returned."))],
 	compute: function(text) {
-		return toString(text).length;
+		return toLocaleString(text, this.locale).length;
 	},
 	isExported: true
 };
@@ -84120,7 +84164,7 @@ const LOWER = {
 	description: _t("Converts a specified string to lowercase."),
 	args: [arg("text (string)", _t("The string to convert to lowercase."))],
 	compute: function(text) {
-		return toString(text).toLowerCase();
+		return toLocaleString(text, this.locale).toLowerCase();
 	},
 	isExported: true
 };
@@ -84132,7 +84176,7 @@ const MID = {
 		arg("extract_length (number)", _t("The length of the segment to extract."))
 	],
 	compute: function(text, starting_at, extract_length) {
-		const _text = toString(text);
+		const _text = toLocaleString(text, this.locale);
 		const _starting_at = toNumber(starting_at, this.locale);
 		const _extract_length = toNumber(extract_length, this.locale);
 		if (_starting_at < 1) return new EvaluationError(_t("The starting_at argument (%s) must be positive greater than one.", _starting_at.toString()));
@@ -84145,7 +84189,7 @@ const PROPER = {
 	description: _t("Capitalizes each word in a specified string."),
 	args: [arg("text_to_capitalize (string)", _t("The text which will be returned with the first letter of each word in uppercase and all other letters in lowercase."))],
 	compute: function(text) {
-		return toString(text).replace(wordRegex, (word) => {
+		return toLocaleString(text, this.locale).replace(wordRegex, (word) => {
 			return word.charAt(0).toUpperCase() + word.slice(1).toLowerCase();
 		});
 	},
@@ -84293,9 +84337,9 @@ const REPLACE = {
 	compute: function(text, position, length, newText) {
 		const _position = toNumber(position, this.locale);
 		if (_position < 1) return new EvaluationError(_t("The position (%s) must be greater than or equal to 1.", _position));
-		const _text = toString(text);
+		const _text = toLocaleString(text, this.locale);
 		const _length = toNumber(length, this.locale);
-		const _newText = toString(newText);
+		const _newText = toLocaleString(newText, this.locale);
 		return _text.substring(0, _position - 1) + _newText + _text.substring(_position - 1 + _length);
 	},
 	isExported: true
@@ -84306,7 +84350,7 @@ const RIGHT = {
 	compute: function(text, ...args) {
 		const _numberOfCharacters = args.length ? toNumber(args[0], this.locale) : 1;
 		if (_numberOfCharacters < 0) return new EvaluationError(_t("The number_of_characters (%s) must be positive or null.", _numberOfCharacters));
-		const _text = toString(text);
+		const _text = toLocaleString(text, this.locale);
 		const stringLength = _text.length;
 		return _text.substring(stringLength - _numberOfCharacters, stringLength);
 	},
@@ -84320,8 +84364,8 @@ const SEARCH = {
 		arg(`starting_at (number, default=${DEFAULT_STARTING_AT})`, _t("The character within text_to_search at which to start the search."))
 	],
 	compute: function(searchFor, textToSearch, startingAt = { value: DEFAULT_STARTING_AT }) {
-		const _searchFor = toString(searchFor).toLowerCase();
-		const _textToSearch = toString(textToSearch).toLowerCase();
+		const _searchFor = toLocaleString(searchFor, this.locale).toLowerCase();
+		const _textToSearch = toLocaleString(textToSearch, this.locale).toLowerCase();
 		const _startingAt = toNumber(startingAt, this.locale);
 		if (_textToSearch === "") return {
 			value: CellErrorType.GenericError,
@@ -84351,8 +84395,8 @@ const SPLIT = {
 		arg(`remove_empty_text (boolean, default=${SPLIT_DEFAULT_REMOVE_EMPTY_TEXT})`, _t("Whether or not to remove empty text messages from the split results. The default behavior is to treat         consecutive delimiters as one (if TRUE). If FALSE, empty cells values are added between consecutive delimiters."))
 	],
 	compute: function(text, delimiter, splitByEach = { value: SPLIT_DEFAULT_SPLIT_BY_EACH }, removeEmptyText = { value: SPLIT_DEFAULT_REMOVE_EMPTY_TEXT }) {
-		const _text = toString(text);
-		const _delimiter = escapeRegExp(toString(delimiter));
+		const _text = toLocaleString(text, this.locale);
+		const _delimiter = escapeRegExp(toLocaleString(delimiter, this.locale));
 		const _splitByEach = toBoolean(splitByEach);
 		const _removeEmptyText = toBoolean(removeEmptyText);
 		if (_delimiter.length <= 0) return new EvaluationError(_t("The delimiter (%s) must be not be empty.", _delimiter));
@@ -84374,10 +84418,10 @@ const SUBSTITUTE = {
 	compute: function(textToSearch, searchFor, replaceWith, occurrenceNumber) {
 		const _occurrenceNumber = toNumber(occurrenceNumber, this.locale);
 		if (_occurrenceNumber < 0) return new EvaluationError(_t("The occurrenceNumber (%s) must be positive or null.", _occurrenceNumber));
-		const _textToSearch = toString(textToSearch);
-		const _searchFor = toString(searchFor);
+		const _textToSearch = toLocaleString(textToSearch, this.locale);
+		const _searchFor = toLocaleString(searchFor, this.locale);
 		if (_searchFor === "") return _textToSearch;
-		const _replaceWith = toString(replaceWith);
+		const _replaceWith = toLocaleString(replaceWith, this.locale);
 		const reg = new RegExp(escapeRegExp(_searchFor), "g");
 		if (_occurrenceNumber === 0) return _textToSearch.replace(reg, _replaceWith);
 		let n = 0;
@@ -84400,10 +84444,10 @@ const TEXTJOIN = {
 		arg("texts (string, range<string>, repeating)", _t("Text item to join."))
 	],
 	compute: function(delimiter, ignoreEmpty = { value: TEXTJOIN_DEFAULT_IGNORE_EMPTY }, ...textsOrArrays) {
-		const _delimiter = toString(delimiter);
+		const _delimiter = toLocaleString(delimiter, this.locale);
 		const _ignoreEmpty = toBoolean(ignoreEmpty);
 		let n = 0;
-		return reduceAny(textsOrArrays, (acc, a) => !(_ignoreEmpty && toString(a) === "") ? (n++ ? acc + _delimiter : "") + toString(a) : acc, "");
+		return reduceAny(textsOrArrays, (acc, a) => !(_ignoreEmpty && toLocaleString(a, this.locale) === "") ? (n++ ? acc + _delimiter : "") + toLocaleString(a, this.locale) : acc, "");
 	},
 	isExported: true
 };
@@ -84456,7 +84500,7 @@ const TRIM = {
 	description: _t("Removes space characters."),
 	args: [arg("text (string)", _t("The text or reference to a cell containing text to be trimmed."))],
 	compute: function(text) {
-		return trimContent(toString(text));
+		return trimContent(toLocaleString(text, this.locale));
 	},
 	isExported: true
 };
@@ -84464,7 +84508,7 @@ const UPPER = {
 	description: _t("Converts a specified string to uppercase."),
 	args: [arg("text (string)", _t("The string to convert to uppercase."))],
 	compute: function(text) {
-		return toString(text).toUpperCase();
+		return toLocaleString(text, this.locale).toUpperCase();
 	},
 	isExported: true
 };
@@ -84561,7 +84605,7 @@ const HYPERLINK = {
 	args: [arg("url (string)", _t("The full URL of the link enclosed in quotation marks.")), arg("link_label (string, optional)", _t("The text to display in the cell, enclosed in quotation marks."))],
 	compute: function(url, linkLabel) {
 		const processedUrl = toString(url).trim();
-		const processedLabel = toString(linkLabel) || processedUrl;
+		const processedLabel = toLocaleString(linkLabel, this.locale) || processedUrl;
 		if (processedUrl === "") return processedLabel;
 		return markdownLink(processedLabel, processedUrl);
 	},
@@ -84958,6 +85002,6 @@ exports.stores = stores;
 exports.tokenColors = tokenColors;
 exports.tokenize = tokenize;
 
-__info__.version = "19.3.16";
-__info__.date = "2026-08-12T09:15:31.627Z";
-__info__.hash = "464be42";
+__info__.version = "19.3.18";
+__info__.date = "2026-08-29T09:16:28.272Z";
+__info__.hash = "1f4b1ae";

package/dist/o_spreadsheet.css

@@ -2,9 +2,9 @@
 /*
  * This file is generated by o-spreadsheet build tools. Do not edit it.
  * @see https://github.com/odoo/o-spreadsheet
- * @version 19.3.16
- * @date 2026-08-12T09:15:33.329Z
- * @hash 464be42
+ * @version 19.3.18
+ * @date 2026-08-29T09:16:30.020Z
+ * @hash 1f4b1ae
  */
 :root {
   --os-gray-100: light-dark(#f9fafb, #1b1d26);
@@ -300,6 +300,7 @@
   .os-input {
     border-width: 0 0 1px 0;
     border-color: transparent;
+    border-style: solid;
     outline: none;
     text-overflow: ellipsis;
     color: var(--os-text-body);
@@ -630,6 +631,7 @@
     width: 100%;
     outline: none;
     border-color: var(--os-border-color);
+    border-style: solid;
     color: var(--os-gray-900);
 
     &::placeholder {
@@ -1478,6 +1480,12 @@
         background-color: var(--os-action-color);
         border-color: var(--os-action-color) !important;
       }
+
+      &:focus {
+        outline: none;
+        box-shadow: 0 0 0 0.25rem rgba(113, 75, 103, 0.25);
+        border-color: var(--os-action-color);
+      }
     }
   }
 }