@odoo/o-spreadsheet 17.2.0 → 17.2.2

package/dist/o-spreadsheet.esm.js

@@ -3,9 +3,9 @@
 /**
  * This file is generated by o-spreadsheet build tools. Do not edit it.
  * @see https://github.com/odoo/o-spreadsheet
- * @version 17.2.0
- * @date 2024-03-20T08:12:44.398Z
- * @hash 1869c24
+ * @version 17.2.2
+ * @date 2024-04-05T14:01:19.824Z
+ * @hash c15836d
  */
 
 import { reactive, useEnv, useSubEnv, useState, onWillUnmount, markRaw, toRaw, Component, useRef, onMounted, useEffect, onPatched, useComponent, onWillPatch, onWillUpdateProps, useExternalListener, onWillStart, xml, useChildSubEnv } from '@odoo/owl';
@@ -29,7 +29,6 @@ const DEFAULT_COLOR_SCALE_MIDPOINT_COLOR = 0xb6d7a8;
 const LINK_COLOR = "#017E84";
 const FILTERS_COLOR = "#188038";
 const BACKGROUND_HEADER_FILTER_COLOR = "#E6F4EA";
-const BACKGROUND_HEADER_SELECTED_FILTER_COLOR = "#CEEAD6";
 const SEPARATOR_COLOR = "#E0E2E4";
 const ICONS_COLOR = "#4A4F59";
 const HEADER_GROUPING_BACKGROUND_COLOR = "#F5F5F5";
@@ -465,7 +464,7 @@ function getItemId(item, itemsDic) {
     }
     // Generate new Id if the item didn't exist in the dictionary
     const ids = Object.keys(itemsDic);
-    const maxId = ids.length === 0 ? 0 : Math.max(...ids.map((id) => parseInt(id, 10)));
+    const maxId = ids.length === 0 ? 0 : largeMax(ids.map((id) => parseInt(id, 10)));
     itemsDic[maxId + 1] = item;
     return maxId + 1;
 }
@@ -685,6 +684,34 @@ function getSearchRegex(searchStr, searchOptions) {
     }
     return RegExp(searchValue, flags);
 }
+/**
+ * Alternative to Math.max that works with large arrays.
+ * Typically useful for arrays bigger than 100k elements.
+ */
+function largeMax(array) {
+    let len = array.length;
+    if (len < 100000)
+        return Math.max(...array);
+    let max = -Infinity;
+    while (len--) {
+        max = array[len] > max ? array[len] : max;
+    }
+    return max;
+}
+/**
+ * Alternative to Math.min that works with large arrays.
+ * Typically useful for arrays bigger than 100k elements.
+ */
+function largeMin(array) {
+    let len = array.length;
+    if (len < 100000)
+        return Math.min(...array);
+    let min = +Infinity;
+    while (len--) {
+        min = array[len] < min ? array[len] : min;
+    }
+    return min;
+}
 
 const RBA_REGEX = /rgba?\(|\s+|\)/gi;
 const HEX_MATCH = /^#([A-F\d]{2}){3,4}$/;
@@ -4544,8 +4571,9 @@ function computeTextLinesHeight(textLineHeight, numberOfLines = 1) {
  * Get the default height of the cell given its style.
  */
 function getDefaultCellHeight(ctx, cell, colSize) {
-    if (!cell || !cell.content)
+    if (!cell || (!cell.isFormula && !cell.content)) {
         return DEFAULT_CELL_HEIGHT;
+    }
     const maxWidth = cell.style?.wrapping === "wrap" ? colSize - 2 * MIN_CELL_TEXT_MARGIN : undefined;
     const numberOfLines = cell.isFormula
         ? 1
@@ -8003,6 +8031,9 @@ class DependencyContainer {
     instantiate(Store, ...args) {
         return this.factory.build(Store, ...args);
     }
+    resetStores() {
+        this.dependencies.clear();
+    }
 }
 class StoreFactory {
     get;
@@ -9936,11 +9967,11 @@ autoCompleteProviders.add("dataValidation", {
         }
         else {
             const range = this.getters.getRangeFromSheetXC(position.sheetId, rule.criterion.values[0]);
-            values = this.getters
+            values = Array.from(new Set(this.getters
                 .getRangeValues(range)
                 .filter(isNotNull)
                 .map((value) => value.toString())
-                .filter((val) => val !== "");
+                .filter((val) => val !== "")));
         }
         return values.map((value) => ({ text: value }));
     },
@@ -19387,10 +19418,10 @@ function aggregateDataForLabels(labels, datasets) {
         }
     }
     return {
-        labels: Object.keys(labelMap),
+        labels: Array.from(labelSet),
         dataSetsValues: datasets.map((dataset, indexOfDataset) => ({
             ...dataset,
-            data: Object.values(labelMap).map((dataOfLabel) => dataOfLabel[indexOfDataset]),
+            data: Array.from(labelSet).map((label) => labelMap[label][indexOfDataset]),
         })),
     };
 }
@@ -20122,7 +20153,7 @@ function getBestTimeUnitForScale(labels, format, locale) {
         return undefined;
     }
     const labelsTimestamps = labelDates.map((date) => date.getTime());
-    const period = Math.max(...labelsTimestamps) - Math.min(...labelsTimestamps);
+    const period = largeMax(labelsTimestamps) - largeMin(labelsTimestamps);
     const minUnit = getFormatMinDisplayUnit(format);
     if (UNIT_LENGTH.second >= UNIT_LENGTH[minUnit] && Milliseconds.inSeconds(period) < 180) {
         return "second";
@@ -20610,7 +20641,7 @@ function getPieConfiguration(chart, labels, localeFormat) {
 }
 function getPieColors(colors, dataSetsValues) {
     const pieColors = [];
-    const maxLength = Math.max(...dataSetsValues.map((ds) => ds.data.length));
+    const maxLength = largeMax(dataSetsValues.map((ds) => ds.data.length));
     for (let i = 0; i <= maxLength; i++) {
         pieColors.push(colors.next());
     }
@@ -23415,8 +23446,8 @@ const DELETE_CONTENT_ROWS_NAME = (env) => {
     let last;
     const activesRows = env.model.getters.getActiveRows();
     if (activesRows.size !== 0) {
-        first = Math.min(...activesRows);
-        last = Math.max(...activesRows);
+        first = largeMin([...activesRows]);
+        last = largeMax([...activesRows]);
     }
     else {
         const zone = env.model.getters.getSelectedZones()[0];
@@ -23444,8 +23475,8 @@ const DELETE_CONTENT_COLUMNS_NAME = (env) => {
     let last;
     const activeCols = env.model.getters.getActiveCols();
     if (activeCols.size !== 0) {
-        first = Math.min(...activeCols);
-        last = Math.max(...activeCols);
+        first = largeMin([...activeCols]);
+        last = largeMax([...activeCols]);
     }
     else {
         const zone = env.model.getters.getSelectedZones()[0];
@@ -23473,8 +23504,8 @@ const REMOVE_ROWS_NAME = (env) => {
     let last;
     const activesRows = env.model.getters.getActiveRows();
     if (activesRows.size !== 0) {
-        first = Math.min(...activesRows);
-        last = Math.max(...activesRows);
+        first = largeMin([...activesRows]);
+        last = largeMax([...activesRows]);
     }
     else {
         const zone = env.model.getters.getSelectedZones()[0];
@@ -23515,8 +23546,8 @@ const REMOVE_COLUMNS_NAME = (env) => {
     let last;
     const activeCols = env.model.getters.getActiveCols();
     if (activeCols.size !== 0) {
-        first = Math.min(...activeCols);
-        last = Math.max(...activeCols);
+        first = largeMin([...activeCols]);
+        last = largeMax([...activeCols]);
     }
     else {
         const zone = env.model.getters.getSelectedZones()[0];
@@ -23557,7 +23588,7 @@ const INSERT_ROWS_BEFORE_ACTION = (env) => {
     let row;
     let quantity;
     if (activeRows.size) {
-        row = Math.min(...activeRows);
+        row = largeMin([...activeRows]);
         quantity = activeRows.size;
     }
     else {
@@ -23578,7 +23609,7 @@ const INSERT_ROWS_AFTER_ACTION = (env) => {
     let row;
     let quantity;
     if (activeRows.size) {
-        row = Math.max(...activeRows);
+        row = largeMax([...activeRows]);
         quantity = activeRows.size;
     }
     else {
@@ -23599,7 +23630,7 @@ const INSERT_COLUMNS_BEFORE_ACTION = (env) => {
     let column;
     let quantity;
     if (activeCols.size) {
-        column = Math.min(...activeCols);
+        column = largeMin([...activeCols]);
         quantity = activeCols.size;
     }
     else {
@@ -23620,7 +23651,7 @@ const INSERT_COLUMNS_AFTER_ACTION = (env) => {
     let column;
     let quantity;
     if (activeCols.size) {
-        column = Math.max(...activeCols);
+        column = largeMax([...activeCols]);
         quantity = activeCols.size;
     }
     else {
@@ -24213,14 +24244,19 @@ const categorieFunctionAll = {
     children: [allFunctionListMenuBuilder],
 };
 function allFunctionListMenuBuilder() {
-    const fnNames = functionRegistry.getKeys();
+    const fnNames = functionRegistry.getKeys().filter((key) => !functionRegistry.get(key).hidden);
     return createFormulaFunctions(fnNames);
 }
 const categoriesFunctionListMenuBuilder = () => {
     const functions = functionRegistry.content;
-    const categories = [...new Set(functionRegistry.getAll().map((fn) => fn.category))].filter(isDefined$1);
+    const categories = [
+        ...new Set(functionRegistry
+            .getAll()
+            .filter((fn) => !fn.hidden)
+            .map((fn) => fn.category)),
+    ].filter(isDefined$1);
     return categories.sort().map((category, i) => {
-        const functionsInCategory = Object.keys(functions).filter((key) => functions[key].category === category);
+        const functionsInCategory = Object.keys(functions).filter((key) => functions[key].category === category && !functions[key].hidden);
         return {
             name: category,
             children: createFormulaFunctions(functionsInCategory),
@@ -28378,11 +28414,9 @@ css /* scss */ `
     }
     .o-cell-is-operator {
       margin-bottom: 5px;
-      width: 96%;
     }
     .o-cell-is-value {
       margin-bottom: 5px;
-      width: 96%;
     }
     .o-color-picker-widget .o-color-picker-button {
       pointer-events: all;
@@ -30216,7 +30250,11 @@ class SplitIntoColumnsPanel extends Component {
         const composerStore = useStore(ComposerStore);
         // The feature makes no sense if we are editing a cell, because then the selection isn't active
         // Stop the edition when the panel is mounted, and close the panel if the user start editing a cell
-        useEffect(this.props.onCloseSidePanel, () => [composerStore.editionMode]);
+        useEffect((editionMode) => {
+            if (editionMode !== "inactive") {
+                this.props.onCloseSidePanel();
+            }
+        }, () => [composerStore.editionMode]);
         onMounted(() => {
             composerStore.stopEdition();
         });
@@ -31084,6 +31122,9 @@ class FigureComponent extends Component {
                 el?.focus({ preventScroll: true });
             }
         }, () => [this.env.model.getters.getSelectedFigureId(), this.props.figure.id, this.figureRef.el]);
+        onWillUnmount(() => {
+            this.props.onFigureDeleted();
+        });
     }
     clickAnchor(dirX, dirY, ev) {
         this.props.onClickAnchor(dirX, dirY, ev);
@@ -31102,6 +31143,7 @@ class FigureComponent extends Component {
                 this.props.onFigureDeleted();
                 ev.stopPropagation();
                 ev.preventDefault();
+                ev.stopPropagation();
                 break;
             case "ArrowDown":
             case "ArrowLeft":
@@ -31122,6 +31164,7 @@ class FigureComponent extends Component {
                 });
                 ev.stopPropagation();
                 ev.preventDefault();
+                ev.stopPropagation();
                 break;
         }
     }
@@ -31789,6 +31832,7 @@ function compareContentToSpanElement(content, node) {
 // -----------------------------------------------------------------------------
 css /* scss */ `
   .o-formula-assistant {
+    background: #ffffff;
     .o-formula-assistant-head {
       background-color: #f2f2f2;
       padding: 10px;
@@ -31844,6 +31888,9 @@ class FunctionDescriptionProvider extends Component {
             this.assistantState.allowCellSelectionBehind = false;
         }, 2000);
     }
+    get formulaArgSeparator() {
+        return this.env.model.getters.getLocale().formulaArgSeparator + " ";
+    }
 }
 
 const functions$2 = functionRegistry.content;
@@ -32003,6 +32050,12 @@ class Composer extends Component {
         useEffect(() => {
             this.processContent();
         });
+        onPatched(() => {
+            // Required because typing '=SUM' and double-clicking another cell leaves ShowProvider/ShowDescription true
+            if (this.composerStore.editionMode === "inactive") {
+                this.processTokenAtCursor();
+            }
+        });
     }
     // ---------------------------------------------------------------------------
     // Handlers
@@ -33983,11 +34036,6 @@ css /* scss */ `
       height: 10000px;
       background-color: ${SELECTION_BORDER_COLOR};
     }
-    .o-unhide-buttons {
-      width: fit-content;
-      gap: 5px;
-      transform: translate(-50%, 0);
-    }
     .o-unhide:hover {
       z-index: ${ComponentsImportance.Grid + 1};
       background-color: lightgrey;
@@ -34149,10 +34197,6 @@ css /* scss */ `
       height: 1px;
       background-color: ${SELECTION_BORDER_COLOR};
     }
-    .o-unhide-buttons {
-      height: fit-content;
-      transform: translate(0, -50%);
-    }
     .o-unhide:hover {
       z-index: ${ComponentsImportance.Grid + 1};
       background-color: lightgrey;
@@ -34613,19 +34657,16 @@ class GridRenderer {
         for (let col = left; col <= right; col++) {
             const colZone = { left: col, right: col, top: 0, bottom: numberOfRows - 1 };
             const { x, width } = this.getters.getVisibleRect(colZone);
-            const colHasFilter = this.getters.doesZonesContainFilter(sheetId, [colZone]);
             const isColActive = activeCols.has(col);
             const isColSelected = selectedCols.has(col);
             if (isColActive) {
-                ctx.fillStyle = colHasFilter ? FILTERS_COLOR : BACKGROUND_HEADER_ACTIVE_COLOR;
+                ctx.fillStyle = BACKGROUND_HEADER_ACTIVE_COLOR;
             }
             else if (isColSelected) {
-                ctx.fillStyle = colHasFilter
-                    ? BACKGROUND_HEADER_SELECTED_FILTER_COLOR
-                    : BACKGROUND_HEADER_SELECTED_COLOR;
+                ctx.fillStyle = BACKGROUND_HEADER_SELECTED_COLOR;
             }
             else {
-                ctx.fillStyle = colHasFilter ? BACKGROUND_HEADER_FILTER_COLOR : BACKGROUND_HEADER_COLOR;
+                ctx.fillStyle = BACKGROUND_HEADER_COLOR;
             }
             ctx.fillRect(x, 0, width, HEADER_HEIGHT);
         }
@@ -34633,19 +34674,16 @@ class GridRenderer {
         for (let row = top; row <= bottom; row++) {
             const rowZone = { top: row, bottom: row, left: 0, right: numberOfCols - 1 };
             const { y, height } = this.getters.getVisibleRect(rowZone);
-            const rowHasFilter = this.getters.doesZonesContainFilter(sheetId, [rowZone]);
             const isRowActive = activeRows.has(row);
             const isRowSelected = selectedRows.has(row);
             if (isRowActive) {
-                ctx.fillStyle = rowHasFilter ? FILTERS_COLOR : BACKGROUND_HEADER_ACTIVE_COLOR;
+                ctx.fillStyle = BACKGROUND_HEADER_ACTIVE_COLOR;
             }
             else if (isRowSelected) {
-                ctx.fillStyle = rowHasFilter
-                    ? BACKGROUND_HEADER_SELECTED_FILTER_COLOR
-                    : BACKGROUND_HEADER_SELECTED_COLOR;
+                ctx.fillStyle = BACKGROUND_HEADER_SELECTED_COLOR;
             }
             else {
-                ctx.fillStyle = rowHasFilter ? BACKGROUND_HEADER_FILTER_COLOR : BACKGROUND_HEADER_COLOR;
+                ctx.fillStyle = BACKGROUND_HEADER_COLOR;
             }
             ctx.fillRect(0, y, HEADER_WIDTH, height);
         }
@@ -37852,7 +37890,7 @@ function convertHyperlink(link, cellValue, warningManager) {
 function getSheetDims(sheet) {
     const dims = [0, 0];
     for (let row of sheet.rows) {
-        dims[0] = Math.max(dims[0], ...row.cells.map((cell) => toCartesian(cell.xc).col));
+        dims[0] = Math.max(dims[0], largeMax(row.cells.map((cell) => toCartesian(cell.xc).col)));
         dims[1] = Math.max(dims[1], row.index);
     }
     dims[0] = Math.max(dims[0], EXCEL_IMPORT_DEFAULT_NUMBER_OF_COLS);
@@ -42359,6 +42397,9 @@ class DataValidationPlugin extends CorePlugin {
         if (newRule.criterion.type === "isBoolean") {
             this.setCenterStyleToBooleanCells(newRule);
         }
+        else if (newRule.criterion.type === "isValueInList") {
+            newRule.criterion.values = Array.from(new Set(newRule.criterion.values));
+        }
         const adaptedRules = this.removeRangesFromRules(sheetId, newRule.ranges, rules);
         const ruleIndex = adaptedRules.findIndex((rule) => rule.id === newRule.id);
         if (ruleIndex !== -1) {
@@ -42755,7 +42796,7 @@ class HeaderVisibilityPlugin extends CorePlugin {
                 if (hiddenElements.size >= elements) {
                     return "TooManyHiddenElements" /* CommandResult.TooManyHiddenElements */;
                 }
-                else if (Math.min(...cmd.elements) < 0 || Math.max(...cmd.elements) > elements) {
+                else if (largeMin(cmd.elements) < 0 || largeMax(cmd.elements) > elements) {
                     return "InvalidHeaderIndex" /* CommandResult.InvalidHeaderIndex */;
                 }
                 else {
@@ -43573,8 +43614,8 @@ class RangeAdapter {
                     let newRange = range;
                     let changeType = "NONE";
                     for (let group of groups) {
-                        const min = Math.min(...group);
-                        const max = Math.max(...group);
+                        const min = largeMin(group);
+                        const max = largeMax(group);
                         if (range.zone[start] <= min && min <= range.zone[end]) {
                             const toRemove = Math.min(range.zone[end], max) - min + 1;
                             changeType = "RESIZE";
@@ -44023,8 +44064,8 @@ class SheetPlugin extends CorePlugin {
                 }
                 return "Success" /* CommandResult.Success */;
             case "REMOVE_COLUMNS_ROWS": {
-                const min = Math.min(...cmd.elements);
-                const max = Math.max(...cmd.elements);
+                const min = largeMin(cmd.elements);
+                const max = largeMax(cmd.elements);
                 if (min < 0 || !this.doesHeaderExist(cmd.sheetId, cmd.dimension, max)) {
                     return "InvalidHeaderIndex" /* CommandResult.InvalidHeaderIndex */;
                 }
@@ -44818,7 +44859,6 @@ class SheetPlugin extends CorePlugin {
 
 class TablePlugin extends CorePlugin {
     static getters = [
-        "doesZonesContainFilter",
         "getFilter",
         "getFilters",
         "getTable",
@@ -44975,10 +45015,6 @@ class TablePlugin extends CorePlugin {
     getTablesOverlappingZones(sheetId, zones) {
         return this.getTables(sheetId).filter((table) => zones.some((zone) => overlap(table.range.zone, zone)));
     }
-    doesZonesContainFilter(sheetId, zones) {
-        return (this.getTablesOverlappingZones(sheetId, zones).filter((table) => table.config.hasFilters)
-            .length > 0);
-    }
     getFilterHeaders(sheetId) {
         const headers = [];
         for (const table of this.getTables(sheetId)) {
@@ -47493,7 +47529,7 @@ class EvaluationPlugin extends UIPlugin {
                 ? getItemId(newFormat, data.formats)
                 : exportedCellData.format;
             let content;
-            if (formulaCell instanceof FormulaCellWithDependencies) {
+            if (isFormula && formulaCell instanceof FormulaCellWithDependencies) {
                 content = formulaCell.contentWithFixedReferences;
             }
             else {
@@ -47942,13 +47978,13 @@ class EvaluationConditionalFormatPlugin extends UIPlugin {
             .map((cell) => cell.value);
         switch (threshold.type) {
             case "value":
-                const result = functionName === "max" ? Math.max(...rangeValues) : Math.min(...rangeValues);
+                const result = functionName === "max" ? largeMax(rangeValues) : largeMin(rangeValues);
                 return result;
             case "number":
                 return Number(threshold.value);
             case "percentage":
-                const min = Math.min(...rangeValues);
-                const max = Math.max(...rangeValues);
+                const min = largeMin(rangeValues);
+                const max = largeMax(rangeValues);
                 const delta = max - min;
                 return min + (delta * Number(threshold.value)) / 100;
             case "percentile":
@@ -49021,13 +49057,13 @@ class AutomaticSumPlugin extends UIPlugin {
         const cells = this.getters.getEvaluatedCellsInZone(sheet.id, zone);
         const cellPositions = range(end, -1, -1);
         const invalidCells = cellPositions.filter((position) => cells[position] && !cells[position].isAutoSummable);
-        const maxValidPosition = Math.max(...invalidCells);
+        const maxValidPosition = largeMax(invalidCells);
         const numberSequences = groupConsecutive(cellPositions.filter((position) => this.isNumber(cells[position])));
         const firstSequence = numberSequences[0] || [];
-        if (Math.max(...firstSequence) < maxValidPosition) {
+        if (largeMax(firstSequence) < maxValidPosition) {
             return Infinity;
         }
-        return Math.min(...firstSequence);
+        return largeMin(firstSequence);
     }
     shouldFindData(sheetId, zone) {
         return this.getters.isEmpty(sheetId, zone) || this.getters.isSingleCellOrMerge(sheetId, zone);
@@ -50768,7 +50804,7 @@ class SheetUIPlugin extends UIPlugin {
     getColMaxWidth(sheetId, index) {
         const cellsPositions = positions(this.getters.getColsZone(sheetId, index, index));
         const sizes = cellsPositions.map((position) => this.getCellWidth({ sheetId, ...position }));
-        return Math.max(0, ...sizes);
+        return Math.max(0, largeMax(sizes));
     }
     /**
      * Check that any "sheetId" in the command matches an existing
@@ -52473,6 +52509,7 @@ class GridSelectionPlugin extends UIPlugin {
                 this.gridSelection.zones = this.gridSelection.zones.map((z) => this.getters.expandZone(sheetId, z));
                 this.gridSelection.anchor.zone = this.getters.expandZone(sheetId, this.gridSelection.anchor.zone);
                 this.setSelectionMixin(this.gridSelection.anchor, this.gridSelection.zones);
+                this.selectedFigureId = null;
                 break;
         }
         /** Any change to the selection has to be  reflected in the selection processor. */
@@ -53663,7 +53700,7 @@ class SheetViewPlugin extends UIPlugin {
      * column of the current viewport
      */
     getColDimensionsInViewport(sheetId, col) {
-        const left = Math.min(...this.getters.getSheetViewVisibleCols());
+        const left = largeMin(this.getters.getSheetViewVisibleCols());
         const start = this.getters.getColRowOffsetInViewport("COL", left, col);
         const size = this.getters.getColSize(sheetId, col);
         const isColHidden = this.getters.isColHidden(sheetId, col);
@@ -53678,7 +53715,7 @@ class SheetViewPlugin extends UIPlugin {
      * of the current viewport
      */
     getRowDimensionsInViewport(sheetId, row) {
-        const top = Math.min(...this.getters.getSheetViewVisibleRows());
+        const top = largeMin(this.getters.getSheetViewVisibleRows());
         const start = this.getters.getColRowOffsetInViewport("ROW", top, row);
         const size = this.getters.getRowSize(sheetId, row);
         const isRowHidden = this.getters.isRowHidden(sheetId, row);
@@ -54391,12 +54428,14 @@ class BottomBarSheet extends Component {
         this.editionState = "initializing";
     }
     stopEdition() {
-        if (!this.state.isEditing)
+        const input = this.sheetNameRef.el;
+        if (!this.state.isEditing || !input)
             return;
         this.state.isEditing = false;
         this.editionState = "initializing";
-        this.sheetNameRef.el?.blur();
+        input.blur();
         const inputValue = this.getInputContent() || "";
+        input.innerText = inputValue;
         interactiveRenameSheet(this.env, this.props.sheetId, inputValue, () => this.startEdition());
     }
     cancelEdition() {
@@ -54687,7 +54726,7 @@ class BottomBar extends Component {
         this.sheetListRef.el.scrollTo({ top: 0, left: scroll, behavior: "smooth" });
     }
     onSheetMouseDown(sheetId, event) {
-        if (event.button !== 0)
+        if (event.button !== 0 || this.env.model.getters.isReadonly())
             return;
         this.closeMenu();
         const visibleSheets = this.getVisibleSheets();
@@ -56091,9 +56130,6 @@ css /* scss */ `
     .text-muted {
       color: grey !important;
     }
-    button {
-      color: #333;
-    }
     .o-disabled {
       opacity: 0.4;
       pointer: default;
@@ -56214,17 +56250,17 @@ css /* scss */ `
   }
 
   .o-button {
-    border: 1px solid lightgrey;
+    border: 1px solid;
     padding: 0px 20px 0px 20px;
     border-radius: 4px;
     font-weight: 500;
     font-size: 14px;
     height: 30px;
     line-height: 16px;
-    background: white;
     margin-right: 8px;
-    &:hover:enabled {
-      background-color: rgba(0, 0, 0, 0.08);
+
+    &:not(:hover) {
+      background-color: transparent;
     }
 
     &:enabled {
@@ -56238,6 +56274,15 @@ css /* scss */ `
     &:last-child {
       margin-right: 0px;
     }
+
+    &.o-button-grey {
+      border-color: lightgrey;
+      background: #ffffff;
+      color: #333;
+      &:hover:enabled {
+        background-color: rgba(0, 0, 0, 0.08);
+      }
+    }
   }
 
   .o-input {
@@ -58135,7 +58180,7 @@ function addLineChart(chart) {
 }
 function addDoughnutChart(chart, chartSheetIndex, data, { holeSize } = { holeSize: 50 }) {
     const colors = new ChartColors();
-    const maxLength = Math.max(...chart.dataSets.map((ds) => getRangeSize(ds.range, chartSheetIndex, data)));
+    const maxLength = largeMax(chart.dataSets.map((ds) => getRangeSize(ds.range, chartSheetIndex, data)));
     const doughnutColors = range(0, maxLength).map(() => toXlsxHexColor(colors.next()));
     const dataSetsNodes = [];
     for (const [dsIndex, dataset] of Object.entries(chart.dataSets).reverse()) {
@@ -60124,6 +60169,6 @@ const constants = {
 export { AbstractCellClipboardHandler, AbstractChart, AbstractFigureClipboardHandler, CellErrorType, CommandResult, CorePlugin, DispatchResult, EvaluationError, Model, Registry, Revision, SPREADSHEET_DIMENSIONS, Spreadsheet, UIPlugin, __info__, addFunction, addRenderingLayer, astToFormula, compile, compileTokens, components, constants, convertAstNodes, coreTypes, findCellInNewZone, functionCache, helpers, hooks, invalidateCFEvaluationCommands, invalidateDependenciesCommands, invalidateEvaluationCommands, iterateAstNodes, links, load, parse, parseTokens, readonlyAllowedCommands, registries, setDefaultSheetViewSize, setTranslationMethod, stores, tokenColors, tokenize };
 
 
-__info__.version = "17.2.0";
-__info__.date = "2024-03-20T08:12:44.398Z";
-__info__.hash = "1869c24";
+__info__.version = "17.2.2";
+__info__.date = "2024-04-05T14:01:19.824Z";
+__info__.hash = "c15836d";