@odoo/o-spreadsheet 19.4.6 → 19.4.8

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.4.6
- * @date 2026-08-10T12:33:32.728Z
- * @hash 5c667fe
+ * @version 19.4.8
+ * @date 2026-08-19T09:25:13.754Z
+ * @hash 9dd828e
  */
 
 Object.defineProperty(exports, Symbol.toStringTag, { value: 'Module' });
@@ -4026,6 +4026,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, "");
@@ -5958,6 +5970,9 @@ var FunctionRegistry = class extends Registry {
 	}
 };
 const functionRegistry = new FunctionRegistry();
+/** Formulas using one of these functions are never squished: they are always exported
+* as a full formula string, and they don't become the base formula of the next cells. */
+const NonSquishableFunctionRegistry = new Registry();
 
 //#endregion
 //#region src/helpers/references.ts
@@ -15476,18 +15491,21 @@ const figureRegistry = new Registry();
 figureRegistry.add("chart", {
 	Component: ChartFigure,
 	SidePanelComponent: "ChartPanel",
-	menuBuilder: getChartMenuActions
+	menuBuilder: getChartMenuActions,
+	isThemeDependant: true
 });
 figureRegistry.add("image", {
 	Component: ImageFigure,
 	keepRatio: true,
 	minFigSize: 20,
 	borderWidth: 0,
-	menuBuilder: getImageMenuActions
+	menuBuilder: getImageMenuActions,
+	isThemeDependant: false
 });
 figureRegistry.add("carousel", {
 	Component: CarouselFigure,
-	menuBuilder: getCarouselMenuActions
+	menuBuilder: getCarouselMenuActions,
+	isThemeDependant: true
 });
 
 //#endregion
@@ -15516,6 +15534,9 @@ var FigureComponent = class extends Component {
 	get isSelected() {
 		return this.env.model.getters.getSelectedFigureIds().includes(this.props.figureUI.id);
 	}
+	get isThemeDependant() {
+		return figureRegistry.get(this.props.figureUI.tag).isThemeDependant;
+	}
 	get figureRegistry() {
 		return figureRegistry;
 	}
@@ -51150,7 +51171,8 @@ var Squisher = class {
 	* The result of this method is:
 	* - if the cell is not a formula, returns the content as is and resets the base formula
 	* - if the cell is a formula:
-	*   - if there is no previous formula, or the normalized formula is different from the previous one, resets the base formula to this one and returns the full formula string
+	*   - if there is no previous formula, or the normalized formula is different from the previous one or it contains blacklisted functions (see NonSquishableFunctionRegistry),
+	*        resets the base formula to this one and returns the full formula string
 	*   - else, compares the literal values and range dependencies to the previous formula, and for each parameter:
 	*     - for numbers: returns a relative change (+N or -N) if possible, else the full number or "=" if unchanged
 	*     - for strings: returns the full string if changed, else "="
@@ -51158,6 +51180,10 @@ var Squisher = class {
 	* */
 	squish(cell, forSheetId) {
 		if (cell.isFormula) {
+			if (NonSquishableFunctionRegistry.getAll().some((functionName) => cell.compiledFormula.usesSymbol(functionName))) {
+				this.resetBaseFormula();
+				return cell.compiledFormula.toFormulaString(this.getters, this.rangeToStringOptions);
+			}
 			let numbers = [];
 			let strings = [];
 			let references = [];
@@ -62602,7 +62628,7 @@ var Session = class extends EventBus {
 			}
 			message = {
 				...message,
-				commands: this.commandSquisher.squish(revision.commands)
+				commands: revision.commands
 			};
 		}
 		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.
@@ -78018,12 +78044,13 @@ var FilterMenu = class extends 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,
@@ -80280,7 +80307,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"));
 	},
@@ -86046,7 +86073,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
 };
@@ -86929,7 +86956,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;
@@ -86940,7 +86967,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
 };
@@ -86948,7 +86975,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
 };
@@ -86960,8 +86987,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));
@@ -86998,8 +87025,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 = {
@@ -87008,7 +87035,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
 };
@@ -87016,7 +87043,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
 };
@@ -87024,7 +87051,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
 };
@@ -87036,7 +87063,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()));
@@ -87049,7 +87076,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();
 		});
 	},
@@ -87199,8 +87226,8 @@ const REPLACE = {
 		if (_position < 1) return new EvaluationError(_t("The position (%s) must be greater than or equal to 1.", _position));
 		const _length = toNumber(length, this.locale);
 		if (_length < 0) return new EvaluationError(_t("The length (%s) must be positive or zero.", _length));
-		const _text = toString(text);
-		const _newText = toString(newText);
+		const _text = toLocaleString(text, this.locale);
+		const _newText = toLocaleString(newText, this.locale);
 		return _text.substring(0, _position - 1) + _newText + _text.substring(_position - 1 + _length);
 	},
 	isExported: true
@@ -87211,7 +87238,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 zero.", _numberOfCharacters));
-		const _text = toString(text);
+		const _text = toLocaleString(text, this.locale);
 		const stringLength = _text.length;
 		return _text.substring(stringLength - _numberOfCharacters, stringLength);
 	},
@@ -87225,8 +87252,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,
@@ -87256,8 +87283,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));
@@ -87279,10 +87306,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;
@@ -87305,10 +87332,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
 };
@@ -87361,7 +87388,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
 };
@@ -87369,7 +87396,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
 };
@@ -87466,7 +87493,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);
 	},
@@ -87612,7 +87639,8 @@ const registries = {
 	migrationStepRegistry,
 	chartJsExtensionRegistry,
 	onIterationEndEvaluationRegistry,
-	specificRangeTransformRegistry
+	specificRangeTransformRegistry,
+	NonSquishableFunctionRegistry
 };
 const helpers = {
 	arg,
@@ -87914,6 +87942,6 @@ exports.stores = stores;
 exports.tokenColors = tokenColors;
 exports.tokenize = tokenize;
 
-__info__.version = "19.4.6";
-__info__.date = "2026-08-10T12:33:32.728Z";
-__info__.hash = "5c667fe";
+__info__.version = "19.4.8";
+__info__.date = "2026-08-19T09:25:13.754Z";
+__info__.hash = "9dd828e";

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.4.6
- * @date 2026-08-10T12:33:34.157Z
- * @hash 5c667fe
+ * @version 19.4.8
+ * @date 2026-08-19T09:25:15.535Z
+ * @hash 9dd828e
  */
 :root {
   --os-gray-100: light-dark(#f9fafb, #1b1d26);
@@ -2565,6 +2565,22 @@
   }
 }
 
+/* Originates from src/components/figures/figure_image/figure_image.css */
+.o-spreadsheet {
+  .o-figure-menu-item {
+    cursor: pointer;
+    color: var(--os-gray-900) !important;
+    height: 20px;
+    width: 20px;
+    &.active,
+    &:hover,
+    &:target {
+      cursor: pointer;
+      background-color: var(--os-background-gray-color-hover);
+    }
+  }
+}
+
 /* Originates from src/components/figures/figure_container/figure_container.css */
 .o-spreadsheet {
   .o-figure-snap-line {