@odoo/o-spreadsheet 18.1.6 → 18.1.8

package/dist/o-spreadsheet.cjs.js

@@ -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 18.1.6
- * @date 2025-02-05T07:18:57.089Z
- * @hash f5b97e0
+ * @version 18.1.8
+ * @date 2025-02-14T08:42:08.322Z
+ * @hash 02682f4
  */
 
 'use strict';
@@ -770,9 +770,16 @@ function deepEqualsArray(arr1, arr2) {
     }
     return true;
 }
-/** Check if the given array contains all the values of the other array. */
+/**
+ * Check if the given array contains all the values of the other array.
+ * It makes the assumption that both array do not contain duplicates.
+ */
 function includesAll(arr, values) {
-    return values.every((value) => arr.includes(value));
+    if (arr.length < values.length) {
+        return false;
+    }
+    const set = new Set(arr);
+    return values.every((value) => set.has(value));
 }
 /**
  * Return an object with all the keys in the object that have a falsy value removed.
@@ -6474,6 +6481,33 @@ function drawDecoratedText(context, text, position, underline = false, strikethr
  * https://stackoverflow.com/questions/105034/create-guid-uuid-in-javascript
  * */
 class UuidGenerator {
+    /**
+     * Generates a custom UUID using a simple 36^12 method (8-character alphanumeric string with lowercase letters)
+     * This has a higher chance of collision than a UUIDv4, but not only faster to generate than an UUIDV4,
+     * it also has a smaller size, which is preferable to alleviate the overall data size.
+     *
+     * This method is preferable when generating uuids for the core data (sheetId, figureId, etc)
+     * as they will appear several times in the revisions and local history.
+     *
+     */
+    smallUuid() {
+        //@ts-ignore
+        if (window.crypto && window.crypto.getRandomValues) {
+            //@ts-ignore
+            return ([1e7] + -1e3).replace(/[018]/g, (c) => (c ^ (crypto.getRandomValues(new Uint8Array(1))[0] & (15 >> (c / 4)))).toString(16));
+        }
+        else {
+            // mainly for jest and other browsers that do not have the crypto functionality
+            return "xxxxxxxx-xxxx".replace(/[xy]/g, function (c) {
+                const r = (Math.random() * 16) | 0, v = c == "x" ? r : (r & 0x3) | 0x8;
+                return v.toString(16);
+            });
+        }
+    }
+    /**
+     * Generates an UUIDV4, has astronomically low chance of collision, but is larger in size than the smallUuid.
+     * This method should be used when you need to avoid collisions at all costs, like the id of a revision.
+     */
     uuidv4() {
         //@ts-ignore
         if (window.crypto && window.crypto.getRandomValues) {
@@ -8508,7 +8542,7 @@ class ChartClipboardHandler extends AbstractFigureClipboardHandler {
         };
     }
     getPasteTarget(sheetId, target, content, options) {
-        const newId = new UuidGenerator().uuidv4();
+        const newId = new UuidGenerator().smallUuid();
         return { zones: [], figureId: newId, sheetId };
     }
     paste(target, clippedContent, options) {
@@ -8674,7 +8708,7 @@ class ConditionalFormatClipboardHandler extends AbstractCellClipboardHandler {
         if (!targetCF && queuedCfs) {
             targetCF = queuedCfs.find((queued) => queued.cf.stopIfTrue === originCF.stopIfTrue && deepEquals(queued.cf.rule, originCF.rule))?.cf;
         }
-        return targetCF || { ...originCF, id: this.uuidGenerator.uuidv4(), ranges: [] };
+        return targetCF || { ...originCF, id: this.uuidGenerator.smallUuid(), ranges: [] };
     }
 }
 
@@ -8767,7 +8801,7 @@ class DataValidationClipboardHandler extends AbstractCellClipboardHandler {
         }
         return (targetRule || {
             ...originRule,
-            id: newId ? this.uuidGenerator.uuidv4() : originRule.id,
+            id: newId ? this.uuidGenerator.smallUuid() : originRule.id,
             ranges: [],
         });
     }
@@ -8829,7 +8863,7 @@ class ImageClipboardHandler extends AbstractFigureClipboardHandler {
         };
     }
     getPasteTarget(sheetId, target, content, options) {
-        const newId = new UuidGenerator().uuidv4();
+        const newId = new UuidGenerator().smallUuid();
         return { sheetId, zones: [], figureId: newId };
     }
     paste(target, clippedContent, options) {
@@ -15104,7 +15138,7 @@ const SORTN = {
             }
         }
     },
-    isExported: true,
+    isExported: false,
 };
 // -----------------------------------------------------------------------------
 // UNIQUE
@@ -27617,7 +27651,7 @@ function forceUnicityOfFigure(data) {
     for (const sheet of data.sheets || []) {
         for (const figure of sheet.figures || []) {
             if (figureIds.has(figure.id)) {
-                figure.id += uuidGenerator.uuidv4();
+                figure.id += uuidGenerator.smallUuid();
             }
             figureIds.add(figure.id);
         }
@@ -28202,9 +28236,7 @@ function getBarChartData(definition, dataSets, labelRange, getters) {
     const labelValues = getChartLabelValues(getters, dataSets, labelRange);
     let labels = labelValues.formattedValues;
     let dataSetsValues = getChartDatasetValues(getters, dataSets);
-    if (definition.dataSetsHaveTitle &&
-        dataSetsValues[0] &&
-        labels.length > dataSetsValues[0].data.length) {
+    if (shouldRemoveFirstLabel(labelRange, dataSets[0], definition.dataSetsHaveTitle || false)) {
         labels.shift();
     }
     ({ labels, dataSetsValues } = filterInvalidDataPoints(labels, dataSetsValues));
@@ -28253,13 +28285,12 @@ function getPyramidChartData(definition, dataSets, labelRange, getters) {
     };
 }
 function getLineChartData(definition, dataSets, labelRange, getters) {
-    const axisType = getChartAxisType(definition, labelRange, getters);
+    const axisType = getChartAxisType(definition, dataSets, labelRange, getters);
     const labelValues = getChartLabelValues(getters, dataSets, labelRange);
     let labels = axisType === "linear" ? labelValues.values : labelValues.formattedValues;
     let dataSetsValues = getChartDatasetValues(getters, dataSets);
-    if (definition.dataSetsHaveTitle &&
-        dataSetsValues[0] &&
-        labels.length > dataSetsValues[0].data.length) {
+    const removeFirstLabel = shouldRemoveFirstLabel(labelRange, dataSets[0], definition.dataSetsHaveTitle || false);
+    if (removeFirstLabel) {
         labels.shift();
     }
     ({ labels, dataSetsValues } = filterInvalidDataPoints(labels, dataSetsValues));
@@ -28271,7 +28302,7 @@ function getLineChartData(definition, dataSets, labelRange, getters) {
     }
     const leftAxisFormat = getChartDatasetFormat(getters, dataSets, "left");
     const rightAxisFormat = getChartDatasetFormat(getters, dataSets, "right");
-    const labelsFormat = getChartLabelFormat(getters, labelRange);
+    const labelsFormat = getChartLabelFormat(getters, labelRange, removeFirstLabel);
     const axisFormats = { y: leftAxisFormat, y1: rightAxisFormat, x: labelsFormat };
     const trendDataSetsValues = [];
     for (const index in dataSetsValues) {
@@ -28307,9 +28338,7 @@ function getPieChartData(definition, dataSets, labelRange, getters) {
     const labelValues = getChartLabelValues(getters, dataSets, labelRange);
     let labels = labelValues.formattedValues;
     let dataSetsValues = getChartDatasetValues(getters, dataSets);
-    if (definition.dataSetsHaveTitle &&
-        dataSetsValues[0] &&
-        labels.length > dataSetsValues[0].data.length) {
+    if (shouldRemoveFirstLabel(labelRange, dataSets[0], definition.dataSetsHaveTitle || false)) {
         labels.shift();
     }
     ({ labels, dataSetsValues } = filterInvalidDataPoints(labels, dataSetsValues));
@@ -28329,9 +28358,7 @@ function getRadarChartData(definition, dataSets, labelRange, getters) {
     const labelValues = getChartLabelValues(getters, dataSets, labelRange);
     let labels = labelValues.formattedValues;
     let dataSetsValues = getChartDatasetValues(getters, dataSets);
-    if (definition.dataSetsHaveTitle &&
-        dataSetsValues[0] &&
-        labels.length > dataSetsValues[0].data.length) {
+    if (shouldRemoveFirstLabel(labelRange, dataSets[0], definition.dataSetsHaveTitle || false)) {
         labels.shift();
     }
     ({ labels, dataSetsValues } = filterInvalidDataPoints(labels, dataSetsValues));
@@ -28351,7 +28378,7 @@ function getRadarChartData(definition, dataSets, labelRange, getters) {
 function getGeoChartData(definition, dataSets, labelRange, getters) {
     const labelValues = getChartLabelValues(getters, dataSets, labelRange);
     let labels = labelValues.formattedValues;
-    if (definition.dataSetsHaveTitle) {
+    if (shouldRemoveFirstLabel(labelRange, dataSets[0], definition.dataSetsHaveTitle || false)) {
         labels.shift();
     }
     let dataSetsValues = getChartDatasetValues(getters, dataSets);
@@ -28512,36 +28539,41 @@ function normalizeLabels(labels, newLabels, config) {
     }
     return { normalizedLabels, normalizedNewLabels };
 }
-function getChartAxisType(chart, labelRange, getters) {
-    if (isDateChart(chart, labelRange, getters) && isLuxonTimeAdapterInstalled()) {
+function getChartAxisType(definition, dataSets, labelRange, getters) {
+    if (isDateChart(definition, dataSets, labelRange, getters) && isLuxonTimeAdapterInstalled()) {
         return "time";
     }
-    if (isLinearChart(chart, labelRange, getters)) {
+    if (isLinearChart(definition, dataSets, labelRange, getters)) {
         return "linear";
     }
     return "category";
 }
-function isDateChart(definition, labelRange, getters) {
-    return !definition.labelsAsText && canBeDateChart(labelRange, getters);
+function isDateChart(definition, dataSets, labelRange, getters) {
+    return !definition.labelsAsText && canBeDateChart(definition, dataSets, labelRange, getters);
 }
-function isLinearChart(definition, labelRange, getters) {
-    return !definition.labelsAsText && canBeLinearChart(labelRange, getters);
+function isLinearChart(definition, dataSets, labelRange, getters) {
+    return !definition.labelsAsText && canBeLinearChart(definition, dataSets, labelRange, getters);
 }
-function canChartParseLabels(labelRange, getters) {
-    return canBeDateChart(labelRange, getters) || canBeLinearChart(labelRange, getters);
+function canChartParseLabels(definition, dataSets, labelRange, getters) {
+    return (canBeDateChart(definition, dataSets, labelRange, getters) ||
+        canBeLinearChart(definition, dataSets, labelRange, getters));
 }
-function canBeDateChart(labelRange, getters) {
-    if (!labelRange || !canBeLinearChart(labelRange, getters)) {
+function canBeDateChart(definition, dataSets, labelRange, getters) {
+    if (!labelRange || !canBeLinearChart(definition, dataSets, labelRange, getters)) {
         return false;
     }
-    const labelFormat = getChartLabelFormat(getters, labelRange);
+    const removeFirstLabel = shouldRemoveFirstLabel(labelRange, dataSets[0], definition.dataSetsHaveTitle || false);
+    const labelFormat = getChartLabelFormat(getters, labelRange, removeFirstLabel);
     return Boolean(labelFormat && timeFormatLuxonCompatible.test(labelFormat));
 }
-function canBeLinearChart(labelRange, getters) {
+function canBeLinearChart(definition, dataSets, labelRange, getters) {
     if (!labelRange) {
         return false;
     }
     const labels = getters.getRangeValues(labelRange);
+    if (shouldRemoveFirstLabel(labelRange, dataSets[0], definition.dataSetsHaveTitle || false)) {
+        labels.shift();
+    }
     if (labels.some((label) => isNaN(Number(label)) && label)) {
         return false;
     }
@@ -28650,17 +28682,15 @@ function aggregateDataForLabels(labels, datasets) {
         })),
     };
 }
-function getChartLabelFormat(getters, range) {
+function getChartLabelFormat(getters, range, shouldRemoveFirstLabel) {
     if (!range)
         return undefined;
-    const { sheetId, zone: { left, top, bottom }, } = range;
-    for (let row = top; row <= bottom; row++) {
-        const format = getters.getEvaluatedCell({ sheetId, col: left, row }).format;
-        if (format) {
-            return format;
-        }
+    const { sheetId, zone } = range;
+    const formats = positions(zone).map((position) => getters.getEvaluatedCell({ sheetId, ...position }).format);
+    if (shouldRemoveFirstLabel) {
+        formats.shift();
     }
-    return undefined;
+    return formats.find((format) => format !== undefined);
 }
 function getChartLabelValues(getters, dataSets, labelRange) {
     let labels = { values: [], formattedValues: [] };
@@ -32893,7 +32923,7 @@ const linkSheet = {
 const deleteSheet = {
     name: _t("Delete"),
     isVisible: (env) => {
-        return env.model.getters.getSheetIds().length > 1;
+        return env.model.getters.getVisibleSheetIds().length > 1;
     },
     execute: (env) => env.askConfirmation(_t("Are you sure you want to delete this sheet?"), () => {
         env.model.dispatch("DELETE_SHEET", { sheetId: env.model.getters.getActiveSheetId() });
@@ -32904,7 +32934,7 @@ const duplicateSheet = {
     name: _t("Duplicate"),
     execute: (env) => {
         const sheetIdFrom = env.model.getters.getActiveSheetId();
-        const sheetIdTo = env.model.uuidGenerator.uuidv4();
+        const sheetIdTo = env.model.uuidGenerator.smallUuid();
         env.model.dispatch("DUPLICATE_SHEET", {
             sheetId: sheetIdFrom,
             sheetIdTo,
@@ -33607,20 +33637,21 @@ function getSmartChartDefinition(zone, getters) {
     }
     // Only display legend for several datasets.
     const newLegendPos = dataSetZone.right === dataSetZone.left ? "none" : "top";
-    const labelRange = labelRangeXc ? getters.getRangeFromSheetXC(sheetId, labelRangeXc) : undefined;
-    if (canChartParseLabels(labelRange, getters)) {
-        return {
-            title: {},
-            dataSets,
-            labelsAsText: false,
-            stacked: false,
-            aggregated: false,
-            cumulative: false,
-            labelRange: labelRangeXc,
-            type: "line",
-            dataSetsHaveTitle,
-            legendPosition: newLegendPos,
-        };
+    const lineChartDefinition = {
+        title: {},
+        dataSets,
+        labelsAsText: false,
+        stacked: false,
+        aggregated: false,
+        cumulative: false,
+        labelRange: labelRangeXc,
+        type: "line",
+        dataSetsHaveTitle,
+        legendPosition: newLegendPos,
+    };
+    const chart = new LineChart(lineChartDefinition, sheetId, getters);
+    if (canChartParseLabels(lineChartDefinition, chart.dataSets, chart.labelRange, getters)) {
+        return lineChartDefinition;
     }
     const _dataSets = createDataSets(getters, dataSets, sheetId, dataSetsHaveTitle);
     if (singleColumn &&
@@ -34034,7 +34065,7 @@ const HIDE_ROWS_NAME = (env) => {
 //------------------------------------------------------------------------------
 const CREATE_CHART = (env) => {
     const getters = env.model.getters;
-    const id = env.model.uuidGenerator.uuidv4();
+    const id = env.model.uuidGenerator.smallUuid();
     const sheetId = getters.getActiveSheetId();
     if (getZoneArea(env.model.getters.getSelectedZone()) === 1) {
         env.model.selection.selectTableAroundSelection();
@@ -34057,8 +34088,8 @@ const CREATE_CHART = (env) => {
 // Pivots
 //------------------------------------------------------------------------------
 const CREATE_PIVOT = (env) => {
-    const pivotId = env.model.uuidGenerator.uuidv4();
-    const newSheetId = env.model.uuidGenerator.uuidv4();
+    const pivotId = env.model.uuidGenerator.smallUuid();
+    const newSheetId = env.model.uuidGenerator.smallUuid();
     const result = env.model.dispatch("INSERT_NEW_PIVOT", { pivotId, newSheetId });
     if (result.isSuccessful) {
         env.openSidePanel("PivotSidePanel", { pivotId });
@@ -34117,7 +34148,7 @@ async function requestImage(env) {
 const CREATE_IMAGE = async (env) => {
     if (env.imageProvider) {
         const sheetId = env.model.getters.getActiveSheetId();
-        const figureId = env.model.uuidGenerator.uuidv4();
+        const figureId = env.model.uuidGenerator.smallUuid();
         const image = await requestImage(env);
         if (!image) {
             throw new Error("No image provider was given to the environment");
@@ -34670,7 +34701,7 @@ const insertCheckbox = {
             ranges,
             sheetId,
             rule: {
-                id: env.model.uuidGenerator.uuidv4(),
+                id: env.model.uuidGenerator.smallUuid(),
                 criterion: {
                     type: "isBoolean",
                     values: [],
@@ -34686,7 +34717,7 @@ const insertDropdown = {
         const zones = env.model.getters.getSelectedZones();
         const sheetId = env.model.getters.getActiveSheetId();
         const ranges = zones.map((zone) => env.model.getters.getRangeDataFromZone(sheetId, zone));
-        const ruleID = env.model.uuidGenerator.uuidv4();
+        const ruleID = env.model.uuidGenerator.smallUuid();
         env.model.dispatch("ADD_DATA_VALIDATION_RULE", {
             ranges,
             sheetId,
@@ -34717,7 +34748,7 @@ const insertSheet = {
     execute: (env) => {
         const activeSheetId = env.model.getters.getActiveSheetId();
         const position = env.model.getters.getSheetIds().indexOf(activeSheetId) + 1;
-        const sheetId = env.model.uuidGenerator.uuidv4();
+        const sheetId = env.model.uuidGenerator.smallUuid();
         env.model.dispatch("CREATE_SHEET", { sheetId, position });
         env.model.dispatch("ACTIVATE_SHEET", { sheetIdFrom: activeSheetId, sheetIdTo: sheetId });
     },
@@ -39206,7 +39237,7 @@ class LineConfigPanel extends GenericChartConfigPanel {
     get canTreatLabelsAsText() {
         const chart = this.env.model.getters.getChart(this.props.figureId);
         if (chart && chart instanceof LineChart) {
-            return canChartParseLabels(chart.labelRange, this.env.model.getters);
+            return canChartParseLabels(chart.getDefinition(), chart.dataSets, chart.labelRange, this.env.model.getters);
         }
         return false;
     }
@@ -39283,7 +39314,7 @@ class ScatterConfigPanel extends GenericChartConfigPanel {
     get canTreatLabelsAsText() {
         const chart = this.env.model.getters.getChart(this.props.figureId);
         if (chart && chart instanceof ScatterChart) {
-            return canChartParseLabels(chart.labelRange, this.env.model.getters);
+            return canChartParseLabels(chart.getDefinition(), chart.dataSets, chart.labelRange, this.env.model.getters);
         }
         return false;
     }
@@ -41031,8 +41062,8 @@ function useDragAndDropListItems() {
         document.body.style.cursor = "move";
         state.draggedItemId = args.draggedItemId;
         const container = direction === "horizontal"
-            ? new HorizontalContainer(args.containerEl)
-            : new VerticalContainer(args.containerEl);
+            ? new HorizontalContainer(args.scrollableContainerEl)
+            : new VerticalContainer(args.scrollableContainerEl);
         dndHelper = new DOMDndHelper({
             ...args,
             container,
@@ -41043,8 +41074,8 @@ function useDragAndDropListItems() {
         const stopListening = startDnd(dndHelper.onMouseMove.bind(dndHelper), dndHelper.onMouseUp.bind(dndHelper));
         cleanupFns.push(stopListening);
         const onScroll = dndHelper.onScroll.bind(dndHelper);
-        args.containerEl.addEventListener("scroll", onScroll);
-        cleanupFns.push(() => args.containerEl.removeEventListener("scroll", onScroll));
+        args.scrollableContainerEl.addEventListener("scroll", onScroll);
+        cleanupFns.push(() => args.scrollableContainerEl.removeEventListener("scroll", onScroll));
         cleanupFns.push(dndHelper.destroy.bind(dndHelper));
     };
     owl.onWillUnmount(() => {
@@ -41501,7 +41532,7 @@ class ConditionalFormatPreviewList extends owl.Component {
             draggedItemId: cf.id,
             initialMousePosition: event.clientY,
             items: items,
-            containerEl: this.cfListRef.el,
+            scrollableContainerEl: this.cfListRef.el,
             onDragEnd: (cfId, finalIndex) => this.onDragEnd(cfId, finalIndex),
         });
     }
@@ -42061,7 +42092,7 @@ class ConditionalFormattingPanel extends owl.Component {
         this.originalEditedCf = undefined;
     }
     addConditionalFormat() {
-        const cfId = this.env.model.uuidGenerator.uuidv4();
+        const cfId = this.env.model.uuidGenerator.smallUuid();
         this.env.model.dispatch("ADD_CONDITIONAL_FORMAT", {
             sheetId: this.activeSheetId,
             ranges: this.env.model.getters
@@ -43331,7 +43362,7 @@ class DataValidationEditor extends owl.Component {
             .getSelectedZones()
             .map((zone) => zoneToXc(this.env.model.getters.getUnboundedZone(sheetId, zone)));
         return {
-            id: this.env.model.uuidGenerator.uuidv4(),
+            id: this.env.model.uuidGenerator.smallUuid(),
             criterion: { type: "textContains", values: [""] },
             ranges,
         };
@@ -44683,6 +44714,7 @@ class PivotLayoutConfigurator extends owl.Component {
         unusedGranularities: Object,
         dateGranularities: Array,
         datetimeGranularities: Array,
+        getScrollableContainerEl: { type: Function, optional: true },
         pivotId: String,
     };
     dimensionsRef = owl.useRef("pivot-dimensions");
@@ -44716,7 +44748,7 @@ class PivotLayoutConfigurator extends owl.Component {
             draggedItemId: dimension.nameWithGranularity,
             initialMousePosition: event.clientY,
             items: draggableItems,
-            containerEl: this.dimensionsRef.el,
+            scrollableContainerEl: this.props.getScrollableContainerEl?.() || this.dimensionsRef.el,
             onDragEnd: (dimensionName, finalIndex) => {
                 const originalIndex = draggableIds.findIndex((id) => id === dimensionName);
                 if (originalIndex === finalIndex) {
@@ -44765,7 +44797,7 @@ class PivotLayoutConfigurator extends owl.Component {
             draggedItemId: measure.id,
             initialMousePosition: event.clientY,
             items: draggableItems,
-            containerEl: this.dimensionsRef.el,
+            scrollableContainerEl: this.props.getScrollableContainerEl?.() || this.dimensionsRef.el,
             onDragEnd: (measureName, finalIndex) => {
                 const originalIndex = draggableIds.findIndex((id) => id === measureName);
                 if (originalIndex === finalIndex) {
@@ -44948,8 +44980,8 @@ class PivotTitleSection extends owl.Component {
         return this.env.model.getters.getPivotDisplayName(this.props.pivotId);
     }
     duplicatePivot() {
-        const newPivotId = this.env.model.uuidGenerator.uuidv4();
-        const newSheetId = this.env.model.uuidGenerator.uuidv4();
+        const newPivotId = this.env.model.uuidGenerator.smallUuid();
+        const newSheetId = this.env.model.uuidGenerator.smallUuid();
         const result = this.env.model.dispatch("DUPLICATE_PIVOT_IN_NEW_SHEET", {
             pivotId: this.props.pivotId,
             newPivotId,
@@ -45398,7 +45430,7 @@ class SpreadsheetPivotTable {
     rowTreeToRows(tree, parentRow) {
         return tree.flatMap((node) => {
             const row = {
-                indent: parentRow ? parentRow.indent + 1 : 0,
+                indent: parentRow ? parentRow.indent + 1 : 1,
                 fields: [...(parentRow?.fields || []), node.field],
                 values: [...(parentRow?.values || []), node.value],
             };
@@ -45454,7 +45486,7 @@ function dataEntriesToRows(dataEntries, index, rows, fields, values) {
         pivotTableRows.push({
             fields: _fields,
             values: _values,
-            indent: index,
+            indent: index + 1,
         });
         const record = groups[value];
         if (record) {
@@ -46452,6 +46484,7 @@ class PivotSpreadsheetSidePanel extends owl.Component {
     };
     store;
     state;
+    pivotSidePanelRef = owl.useRef("pivotSidePanel");
     setup() {
         this.store = useLocalStore(PivotSidePanelStore, this.props.pivotId);
         this.state = owl.useState({
@@ -46480,6 +46513,9 @@ class PivotSpreadsheetSidePanel extends owl.Component {
     get definition() {
         return this.store.definition;
     }
+    getScrollableContainerEl() {
+        return this.pivotSidePanelRef.el;
+    }
     onSelectionChanged(ranges) {
         this.state.rangeHasChanged = true;
         this.state.range = ranges[0];
@@ -47570,7 +47606,7 @@ class TableStyleEditorPanel extends owl.Component {
         this.state.selectedTemplateName = templateName;
     }
     onConfirm() {
-        const tableStyleId = this.props.styleId || this.env.model.uuidGenerator.uuidv4();
+        const tableStyleId = this.props.styleId || this.env.model.uuidGenerator.smallUuid();
         this.env.model.dispatch("CREATE_TABLE_STYLE", {
             tableStyleId,
             tableStyleName: this.state.styleName,
@@ -56144,7 +56180,7 @@ class SheetPlugin extends CorePlugin {
                     ? "Success" /* CommandResult.Success */
                     : "InvalidColor" /* CommandResult.InvalidColor */;
             case "DELETE_SHEET":
-                return this.orderedSheetIds.length > 1
+                return this.getVisibleSheetIds().length > 1
                     ? "Success" /* CommandResult.Success */
                     : "NotEnoughSheets" /* CommandResult.NotEnoughSheets */;
             case "ADD_COLUMNS_ROWS":
@@ -63503,6 +63539,15 @@ class Session extends EventBus {
         }
         this.sendPendingMessage();
     }
+    dropPendingRevision(revisionId) {
+        this.revisions.drop(revisionId);
+        const revisionIds = this.pendingMessages
+            .filter((message) => message.type === "REMOTE_REVISION")
+            .map((message) => message.nextRevisionId);
+        this.trigger("pending-revisions-dropped", { revisionIds });
+        this.waitingAck = false;
+        this.waitingUndoRedoAck = false;
+    }
     /**
      * Send the next pending message
      */
@@ -63517,13 +63562,7 @@ class Session extends EventBus {
                  * The command is empty, we have to drop all the next local revisions
                  * to avoid issues with undo/redo
                  */
-                this.revisions.drop(revision.id);
-                const revisionIds = this.pendingMessages
-                    .filter((message) => message.type === "REMOTE_REVISION")
-                    .map((message) => message.nextRevisionId);
-                this.trigger("pending-revisions-dropped", { revisionIds });
-                this.waitingAck = false;
-                this.waitingUndoRedoAck = false;
+                this.dropPendingRevision(revision.id);
                 this.pendingMessages = [];
                 return;
             }
@@ -63550,7 +63589,6 @@ class Session extends EventBus {
         switch (message.type) {
             case "REMOTE_REVISION":
             case "REVISION_REDONE":
-            case "REVISION_UNDONE":
             case "SNAPSHOT_CREATED":
                 this.waitingAck = false;
                 this.pendingMessages = this.pendingMessages.filter((msg) => msg.nextRevisionId !== message.nextRevisionId);
@@ -63559,6 +63597,27 @@ class Session extends EventBus {
                 this.lastRevisionMessage = message;
                 this.sendPendingMessage();
                 break;
+            case "REVISION_UNDONE": {
+                this.waitingAck = false;
+                this.pendingMessages = this.pendingMessages.filter((msg) => msg.nextRevisionId !== message.nextRevisionId);
+                const pendingRemoteRevisions = this.pendingMessages.filter((message) => message.type === "REMOTE_REVISION");
+                const firstTransformedRevisionIndex = pendingRemoteRevisions.findIndex((message) => !deepEquals(message.commands, this.revisions.get(message.nextRevisionId).commands));
+                if (firstTransformedRevisionIndex !== -1) {
+                    /**
+                     * Some revisions undergo transformations that may cause issues with
+                     * undo/redo if the transformation is destructive (we don't get back
+                     * the original command by transforming it with the inverse).
+                     * To prevent these problems, we must discard all subsequent local
+                     * revisions.
+                     */
+                    this.dropPendingRevision(this.pendingMessages[firstTransformedRevisionIndex].nextRevisionId);
+                    this.pendingMessages = this.pendingMessages.slice(0, firstTransformedRevisionIndex);
+                }
+                this.serverRevisionId = message.nextRevisionId;
+                this.processedRevisions.add(message.nextRevisionId);
+                this.sendPendingMessage();
+                break;
+            }
         }
     }
     isAlreadyProcessed(message) {
@@ -65029,23 +65088,23 @@ const uuidGenerator = new UuidGenerator();
 function repeatCreateChartCommand(getters, cmd) {
     return {
         ...repeatSheetDependantCommand(getters, cmd),
-        id: uuidGenerator.uuidv4(),
+        id: uuidGenerator.smallUuid(),
     };
 }
 function repeatCreateImageCommand(getters, cmd) {
     return {
         ...repeatSheetDependantCommand(getters, cmd),
-        figureId: uuidGenerator.uuidv4(),
+        figureId: uuidGenerator.smallUuid(),
     };
 }
 function repeatCreateFigureCommand(getters, cmd) {
     const newCmd = repeatSheetDependantCommand(getters, cmd);
-    newCmd.figure.id = uuidGenerator.uuidv4();
+    newCmd.figure.id = uuidGenerator.smallUuid();
     return newCmd;
 }
 function repeatCreateSheetCommand(getters, cmd) {
     const newCmd = deepCopy(cmd);
-    newCmd.sheetId = uuidGenerator.uuidv4();
+    newCmd.sheetId = uuidGenerator.smallUuid();
     const sheetName = cmd.name || getters.getSheet(getters.getActiveSheetId()).name;
     // Extract the prefix of the sheet name (everything before the number at the end of the name)
     const namePrefix = sheetName.match(/(.+?)\d*$/)?.[1] || sheetName;
@@ -66508,23 +66567,7 @@ class GridSelectionPlugin extends UIPlugin {
                         gridSelection: deepCopy(gridSelection),
                     };
                 }
-                if (!this.getters.tryGetSheet(this.getters.getActiveSheetId())) {
-                    const currentSheetIds = this.getters.getVisibleSheetIds();
-                    this.activeSheet = this.getters.getSheet(currentSheetIds[0]);
-                    if (this.activeSheet.id in this.sheetsData) {
-                        const { anchor } = this.clipSelection(this.activeSheet.id, this.sheetsData[this.activeSheet.id].gridSelection);
-                        this.selectCell(anchor.cell.col, anchor.cell.row);
-                    }
-                    else {
-                        this.selectCell(0, 0);
-                    }
-                    const { col, row } = this.gridSelection.anchor.cell;
-                    this.moveClient({
-                        sheetId: this.getters.getActiveSheetId(),
-                        col,
-                        row,
-                    });
-                }
+                this.fallbackToVisibleSheet();
                 const sheetId = this.getters.getActiveSheetId();
                 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);
@@ -66534,6 +66577,7 @@ class GridSelectionPlugin extends UIPlugin {
         }
     }
     finalize() {
+        this.fallbackToVisibleSheet();
         /** Any change to the selection has to be  reflected in the selection processor. */
         this.selection.resetDefaultAnchor(this, deepCopy(this.gridSelection.anchor));
     }
@@ -66844,6 +66888,25 @@ class GridSelectionPlugin extends UIPlugin {
         }
         return "Success" /* CommandResult.Success */;
     }
+    fallbackToVisibleSheet() {
+        if (!this.getters.tryGetSheet(this.getters.getActiveSheetId())) {
+            const currentSheetIds = this.getters.getVisibleSheetIds();
+            this.activeSheet = this.getters.getSheet(currentSheetIds[0]);
+            if (this.activeSheet.id in this.sheetsData) {
+                const { anchor } = this.clipSelection(this.activeSheet.id, this.sheetsData[this.activeSheet.id].gridSelection);
+                this.selectCell(anchor.cell.col, anchor.cell.row);
+            }
+            else {
+                this.selectCell(0, 0);
+            }
+            const { col, row } = this.gridSelection.anchor.cell;
+            this.moveClient({
+                sheetId: this.getters.getActiveSheetId(),
+                col,
+                row,
+            });
+        }
+    }
     //-------------------------------------------
     // Helpers for extensions
     // ------------------------------------------
@@ -68844,7 +68907,7 @@ class BottomBar extends owl.Component {
     clickAddSheet(ev) {
         const activeSheetId = this.env.model.getters.getActiveSheetId();
         const position = this.env.model.getters.getSheetIds().findIndex((sheetId) => sheetId === activeSheetId) + 1;
-        const sheetId = this.env.model.uuidGenerator.uuidv4();
+        const sheetId = this.env.model.uuidGenerator.smallUuid();
         const name = this.env.model.getters.getNextSheetName(_t("Sheet"));
         this.env.model.dispatch("CREATE_SHEET", { sheetId, position, name });
         this.env.model.dispatch("ACTIVATE_SHEET", { sheetIdFrom: activeSheetId, sheetIdTo: sheetId });
@@ -68961,7 +69024,7 @@ class BottomBar extends owl.Component {
             draggedItemId: sheetId,
             initialMousePosition: event.clientX,
             items: sheets,
-            containerEl: this.sheetListRef.el,
+            scrollableContainerEl: this.sheetListRef.el,
             onDragEnd: (sheetId, finalIndex) => this.onDragEnd(sheetId, finalIndex),
         });
     }
@@ -69859,6 +69922,10 @@ const FX_SVG = /*xml*/ `
 </svg>
 `;
 css /* scss */ `
+  .o-topbar-composer-container {
+    height: ${TOPBAR_TOOLBAR_HEIGHT}px;
+  }
+
   .o-topbar-composer {
     height: fit-content;
     margin-top: -1px;
@@ -71132,7 +71199,7 @@ class Tree {
     }
     /**
      * Drop the operation and all following operations in every
-     * branch
+     * branches
      */
     drop(operationId) {
         for (const branch of this.branches) {
@@ -74577,7 +74644,7 @@ class Model extends EventBus {
     }
     setupConfig(config) {
         const client = config.client || {
-            id: this.uuidGenerator.uuidv4(),
+            id: this.uuidGenerator.smallUuid(),
             name: _t("Anonymous").toString(),
         };
         const transportService = config.transportService || new LocalTransportService();
@@ -75100,6 +75167,6 @@ exports.tokenColors = tokenColors;
 exports.tokenize = tokenize;
 
 
-__info__.version = "18.1.6";
-__info__.date = "2025-02-05T07:18:57.089Z";
-__info__.hash = "f5b97e0";
+__info__.version = "18.1.8";
+__info__.date = "2025-02-14T08:42:08.322Z";
+__info__.hash = "02682f4";