@odoo/o-spreadsheet 18.1.7 → 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.7
- * @date 2025-02-10T09:00:28.556Z
- * @hash 338d8a1
+ * @version 18.1.8
+ * @date 2025-02-14T08:42:08.322Z
+ * @hash 02682f4
  */
 
 'use strict';
@@ -6481,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) {
@@ -8515,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) {
@@ -8681,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: [] };
     }
 }
 
@@ -8774,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: [],
         });
     }
@@ -8836,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) {
@@ -15111,7 +15138,7 @@ const SORTN = {
             }
         }
     },
-    isExported: true,
+    isExported: false,
 };
 // -----------------------------------------------------------------------------
 // UNIQUE
@@ -27624,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);
         }
@@ -28209,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));
@@ -28260,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));
@@ -28278,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) {
@@ -28314,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));
@@ -28336,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));
@@ -28358,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);
@@ -28519,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;
     }
@@ -28657,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: [] };
@@ -32900,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() });
@@ -32911,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,
@@ -33614,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 &&
@@ -34041,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();
@@ -34064,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 });
@@ -34124,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");
@@ -34677,7 +34701,7 @@ const insertCheckbox = {
             ranges,
             sheetId,
             rule: {
-                id: env.model.uuidGenerator.uuidv4(),
+                id: env.model.uuidGenerator.smallUuid(),
                 criterion: {
                     type: "isBoolean",
                     values: [],
@@ -34693,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,
@@ -34724,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 });
     },
@@ -39213,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;
     }
@@ -39290,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;
     }
@@ -42068,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
@@ -43338,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,
         };
@@ -44956,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,
@@ -47582,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,
@@ -56156,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":
@@ -63515,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
      */
@@ -63529,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;
             }
@@ -63562,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);
@@ -63571,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) {
@@ -65041,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;
@@ -66520,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);
@@ -66546,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));
     }
@@ -66856,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
     // ------------------------------------------
@@ -68856,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 });
@@ -69871,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;
@@ -71144,7 +71199,7 @@ class Tree {
     }
     /**
      * Drop the operation and all following operations in every
-     * branch
+     * branches
      */
     drop(operationId) {
         for (const branch of this.branches) {
@@ -74589,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();
@@ -75112,6 +75167,6 @@ exports.tokenColors = tokenColors;
 exports.tokenize = tokenize;
 
 
-__info__.version = "18.1.7";
-__info__.date = "2025-02-10T09:00:28.556Z";
-__info__.hash = "338d8a1";
+__info__.version = "18.1.8";
+__info__.date = "2025-02-14T08:42:08.322Z";
+__info__.hash = "02682f4";

package/dist/o-spreadsheet.d.ts

@@ -1247,6 +1247,7 @@ declare class Session extends EventBus<CollaborativeEvent> {
     private onClientJoined;
     private onClientLeft;
     private sendUpdateMessage;
+    private dropPendingRevision;
     /**
      * Send the next pending message
      */
@@ -1446,6 +1447,20 @@ declare function splitReference(ref: string): {
 declare function computeTextWidth(context: CanvasRenderingContext2D, text: string, style: Style, fontUnit?: "px" | "pt"): number;
 
 declare 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(): string;
+    /**
+     * 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(): string;
 }
 
@@ -2046,6 +2061,7 @@ declare class GridSelectionPlugin extends UIPlugin {
     private onAddElements;
     private onMoveElements;
     private isMoveElementAllowed;
+    private fallbackToVisibleSheet;
     /**
      * Clip the selection if it spans outside the sheet
      */
@@ -12668,9 +12684,9 @@ declare const chartHelpers: {
     getGeoChartData(definition: GeoChartDefinition, dataSets: DataSet[], labelRange: Range | undefined, getters: Getters): GeoChartRuntimeGenerationArgs;
     getTrendDatasetForBarChart(config: TrendConfiguration, data: any[]): chart_js.Point[] | undefined;
     getTrendDatasetForLineChart(config: TrendConfiguration, data: any[], labels: string[], axisType: AxisType, locale: Locale): chart_js.Point[] | undefined;
-    canChartParseLabels(labelRange: Range | undefined, getters: Getters): boolean;
+    canChartParseLabels(definition: GenericDefinition<LineChartDefinition>, dataSets: DataSet[], labelRange: Range | undefined, getters: Getters): boolean;
     getData(getters: Getters, ds: DataSet): (CellValue | undefined)[];
-    getChartLabelFormat(getters: Getters, range: Range | undefined): Format | undefined;
+    getChartLabelFormat(getters: Getters, range: Range | undefined, shouldRemoveFirstLabel: boolean): Format | undefined;
     getBarChartDatasets(definition: GenericDefinition<BarChartDefinition>, args: ChartRuntimeGenerationArgs): chart_js.ChartDataset<"line" | "bar">[];
     getWaterfallDatasetAndLabels(definition: GenericDefinition<WaterfallChartDefinition>, args: ChartRuntimeGenerationArgs): {
         datasets: chart_js.ChartDataset[];