@odoo/o-spreadsheet 17.1.0-alpha.4 → 17.1.0-alpha.5

package/dist/o-spreadsheet.esm.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 17.1.0-alpha.4
- * @date 2023-11-24T13:12:24.882Z
- * @hash 255821b
+ * @version 17.1.0-alpha.5
+ * @date 2023-12-05T09:51:40.034Z
+ * @hash c2823eb
  */
 
 import { Component, useRef, onMounted, useEffect, onWillPatch, useState, onWillUpdateProps, onPatched, useComponent, useExternalListener, onWillUnmount, onWillStart, xml, useChildSubEnv, useSubEnv, markRaw } from '@odoo/owl';
@@ -3386,8 +3386,10 @@ function convertInternalFormatToFormat(internalFormat) {
 const cellReference = new RegExp(/\$?([A-Z]{1,3})\$?([0-9]{1,7})/, "i");
 // Same as above, but matches the exact string (nothing before or after)
 const singleCellReference = new RegExp(/^\$?([A-Z]{1,3})\$?([0-9]{1,7})$/, "i");
-/** Reference of a column header (eg. A, AB) */
-const colHeader = new RegExp(/^([A-Z]{1,3})+$/, "i");
+/** Reference of a column header (eg. A, AB, $A) */
+const colHeader = new RegExp(/^\$?([A-Z]{1,3})+$/, "i");
+/** Reference of a row header (eg. 1, $1) */
+const rowHeader = new RegExp(/^\$?([0-9]{1,7})+$/, "i");
 /** Reference of a column (eg. A, $CA, Sheet1!B) */
 const colReference = new RegExp(/^\s*('.+'!|[^']+!)?\$?([A-Z]{1,3})$/, "i");
 /** Reference of a row (eg. 1, 59, Sheet1!9) */
@@ -3417,6 +3419,9 @@ function isRowReference(xc) {
 function isColHeader(str) {
     return colHeader.test(str);
 }
+function isRowHeader(str) {
+    return rowHeader.test(str);
+}
 /**
  * Return true if the given xc is the reference of a single cell,
  * without any specified sheet (e.g. A1)
@@ -4107,18 +4112,10 @@ class RangeImpl {
         if (isFullCol) {
             parts[0].rowFixed = parts[0].rowFixed || parts[1].rowFixed;
             parts[1].rowFixed = parts[0].rowFixed || parts[1].rowFixed;
-            if (zone.left === zone.right) {
-                parts[0].colFixed = parts[0].colFixed || parts[1].colFixed;
-                parts[1].colFixed = parts[0].colFixed || parts[1].colFixed;
-            }
         }
         if (isFullRow) {
             parts[0].colFixed = parts[0].colFixed || parts[1].colFixed;
             parts[1].colFixed = parts[0].colFixed || parts[1].colFixed;
-            if (zone.top === zone.bottom) {
-                parts[0].rowFixed = parts[0].rowFixed || parts[1].rowFixed;
-                parts[1].rowFixed = parts[0].rowFixed || parts[1].rowFixed;
-            }
         }
         return parts;
     }
@@ -4360,7 +4357,7 @@ function computeTextLinesHeight(textLineHeight, numberOfLines = 1) {
 function getDefaultCellHeight(ctx, cell, colSize) {
     if (!cell || !cell.content)
         return DEFAULT_CELL_HEIGHT;
-    const maxWidth = cell.style?.wrapping ? colSize - 2 * MIN_CELL_TEXT_MARGIN : undefined;
+    const maxWidth = cell.style?.wrapping === "wrap" ? colSize - 2 * MIN_CELL_TEXT_MARGIN : undefined;
     const numberOfLines = cell.isFormula
         ? 1
         : splitTextToWidth(ctx, cell.content, cell.style, maxWidth).length;
@@ -6257,10 +6254,6 @@ css /* scss */ `
 
     .o-error-tooltip-message {
       overflow: hidden;
-      display: -webkit-box; /* Limit to 3 lines */
-      -webkit-line-clamp: 3;
-      line-clamp: 3;
-      -webkit-box-orient: vertical;
     }
   }
 `;
@@ -6668,6 +6661,9 @@ function keyboardEventToShortcutString(ev, mode = "key") {
     keyDownString += letterRegex.test(key) ? key.toUpperCase() : key;
     return keyDownString;
 }
+function isMacOS() {
+    return navigator.userAgent.toUpperCase().indexOf("MAC") >= 0;
+}
 
 /**
  * Return the o-spreadsheet element position relative
@@ -17947,6 +17943,10 @@ function setXcToFixedReferenceType(xc, referenceType) {
             return xc.slice(0, indexOfNumber) + "$" + xc.slice(indexOfNumber);
         case "colrow":
             indexOfNumber = xc.search(/[0-9]/);
+            if (indexOfNumber === -1 || indexOfNumber === 0) {
+                // no row number (eg. A) or no column (eg. 1)
+                return "$" + xc;
+            }
             xc = xc.slice(0, indexOfNumber) + "$" + xc.slice(indexOfNumber);
             return "$" + xc;
         case "none":
@@ -28517,6 +28517,7 @@ class AbstractResizer extends Component {
         draggerShadowThickness: 0,
         delta: 0,
         base: 0,
+        position: "before",
     });
     _computeHandleDisplay(ev) {
         const position = this._getEvOffset(ev);
@@ -28641,11 +28642,13 @@ class AbstractResizer extends Component {
                     this.state.draggerLinePosition = dimensions.start;
                     this.state.draggerShadowPosition = dimensions.start;
                     this.state.base = elementIndex;
+                    this.state.position = "before";
                 }
                 else if (this._getSelectedZoneEnd() < elementIndex) {
                     this.state.draggerLinePosition = dimensions.end;
                     this.state.draggerShadowPosition = dimensions.end - this.state.draggerShadowThickness;
-                    this.state.base = elementIndex + 1;
+                    this.state.base = elementIndex;
+                    this.state.position = "after";
                 }
                 else {
                     this.state.draggerLinePosition = startDimensions.start;
@@ -28820,6 +28823,7 @@ class ColResizer extends AbstractResizer {
             dimension: "COL",
             base: this.state.base,
             elements,
+            position: this.state.position,
         });
         if (!result.isSuccessful && result.reasons.includes("WillRemoveExistingMerge" /* CommandResult.WillRemoveExistingMerge */)) {
             this.env.raiseError(MergeErrorMessage);
@@ -28984,6 +28988,7 @@ class RowResizer extends AbstractResizer {
             dimension: "ROW",
             base: this.state.base,
             elements,
+            position: this.state.position,
         });
         if (!result.isSuccessful && result.reasons.includes("WillRemoveExistingMerge" /* CommandResult.WillRemoveExistingMerge */)) {
             this.env.raiseError(MergeErrorMessage);
@@ -29091,8 +29096,8 @@ function useWheelHandler(handler) {
         return val * (deltaMode === 0 ? 1 : DEFAULT_CELL_HEIGHT);
     }
     const onMouseWheel = (ev) => {
-        const deltaX = normalize(ev.shiftKey ? ev.deltaY : ev.deltaX, ev.deltaMode);
-        const deltaY = normalize(ev.shiftKey ? ev.deltaX : ev.deltaY, ev.deltaMode);
+        const deltaX = normalize(ev.shiftKey && !isMacOS() ? ev.deltaY : ev.deltaX, ev.deltaMode);
+        const deltaY = normalize(ev.shiftKey && !isMacOS() ? ev.deltaX : ev.deltaY, ev.deltaMode);
         handler(deltaX, deltaY);
     };
     return onMouseWheel;
@@ -33529,7 +33534,7 @@ const machine = {
         SPACE: goTo(State.RightRef),
         NUMBER: goTo(State.Found),
         REFERENCE: goTo(State.Found, (token) => isSingleCellReference(token.value)),
-        SYMBOL: goTo(State.Found, (token) => isColHeader(token.value)),
+        SYMBOL: goTo(State.Found, (token) => isColHeader(token.value) || isRowHeader(token.value)),
     },
     [State.RightColumnRef]: {
         SPACE: goTo(State.RightColumnRef),
@@ -33540,6 +33545,7 @@ const machine = {
         SPACE: goTo(State.RightRowRef),
         NUMBER: goTo(State.Found),
         REFERENCE: goTo(State.Found, (token) => isSingleCellReference(token.value)),
+        SYMBOL: goTo(State.Found, (token) => isRowHeader(token.value)),
     },
     [State.Found]: {},
 };
@@ -37519,7 +37525,16 @@ class HeaderVisibilityPlugin extends CorePlugin {
         return consecutiveIndexes;
     }
     getAllVisibleHeaders(sheetId, dimension) {
-        return range(0, this.hiddenHeaders[sheetId][dimension].length).filter((i) => !this.hiddenHeaders[sheetId][dimension][i]);
+        const headers = range(0, this.getters.getNumberHeaders(sheetId, dimension));
+        const foldedHeaders = [];
+        this.getters.getHeaderGroups(sheetId, dimension).forEach((group) => {
+            if (group.isFolded) {
+                foldedHeaders.push(...range(group.start, group.end + 1));
+            }
+        });
+        return headers.filter((i) => {
+            return !this.hiddenHeaders[sheetId][dimension][i] && !foldedHeaders.includes(i);
+        });
     }
     import(data) {
         for (let sheet of data.sheets) {
@@ -39980,10 +39995,13 @@ class CompilationParametersBuilder {
         if (evaluatedCell === undefined) {
             return { value: null, format: this.getters.getCell(position)?.format };
         }
+        if (evaluatedCell.type === CellValueType.error) {
+            throw evaluatedCell.error;
+        }
         return evaluatedCell;
     }
     getEvaluatedCellIfNotEmpty(position) {
-        const evaluatedCell = this.getEvaluatedCell(position);
+        const evaluatedCell = this.computeCell(position);
         if (evaluatedCell.type === CellValueType.empty) {
             const cell = this.getters.getCell(position);
             if (!cell || (!cell.isFormula && cell.content === "")) {
@@ -39992,13 +40010,6 @@ class CompilationParametersBuilder {
         }
         return evaluatedCell;
     }
-    getEvaluatedCell(position) {
-        const evaluatedCell = this.computeCell(position);
-        if (evaluatedCell.type === CellValueType.error) {
-            throw evaluatedCell.error;
-        }
-        return evaluatedCell;
-    }
     /**
      * Return the values of the cell(s) used in reference, but always in the format of a range even
      * if a single cell is referenced. It is a list of col values. This is useful for the formulas that describe parameters as
@@ -40021,7 +40032,11 @@ class CompilationParametersBuilder {
         const { top, left, bottom, right } = zone;
         const cacheKey = `${sheetId}-${top}-${left}-${bottom}-${right}`;
         if (cacheKey in this.rangeCache) {
-            return this.rangeCache[cacheKey];
+            const result = this.rangeCache[cacheKey];
+            if (result instanceof EvaluationError) {
+                throw result;
+            }
+            return result;
         }
         const height = _zone.bottom - _zone.top + 1;
         const width = _zone.right - _zone.left + 1;
@@ -40031,6 +40046,11 @@ class CompilationParametersBuilder {
             const colIndex = col - _zone.left;
             matrix[colIndex] = new Array(height);
             for (let row = _zone.top; row <= _zone.bottom; row++) {
+                const evaluatedCell = this.getEvaluatedCellIfNotEmpty({ sheetId, col, row });
+                if (evaluatedCell?.type === CellValueType.error) {
+                    this.rangeCache[cacheKey] = evaluatedCell.error;
+                    throw evaluatedCell.error;
+                }
                 const rowIndex = row - _zone.top;
                 matrix[colIndex][rowIndex] = this.readCell({ sheetId, col, row });
             }
@@ -41132,7 +41152,7 @@ class EvaluationConditionalFormatPlugin extends UIPlugin {
     // ---------------------------------------------------------------------------
     handle(cmd) {
         if (invalidateCFEvaluationCommands.has(cmd.type) ||
-            (cmd.type === "UPDATE_CELL" && "content" in cmd)) {
+            (cmd.type === "UPDATE_CELL" && ("content" in cmd || "format" in cmd))) {
             this.isStale = true;
         }
     }
@@ -41458,7 +41478,7 @@ class EvaluationDataValidationPlugin extends UIPlugin {
     handle(cmd) {
         if (invalidateEvaluationCommands.has(cmd.type) ||
             cmd.type === "EVALUATE_CELLS" ||
-            (cmd.type === "UPDATE_CELL" && "content" in cmd)) {
+            (cmd.type === "UPDATE_CELL" && ("content" in cmd || "format" in cmd))) {
             this.validationResults = {};
             return;
         }
@@ -41996,19 +42016,21 @@ class AutofillPlugin extends UIPlugin {
         let col = zone.left;
         let row = zone.bottom;
         if (col > 0) {
-            let left = this.getters.getEvaluatedCell({ sheetId, col: col - 1, row });
-            while (left.type !== CellValueType.empty) {
+            let leftPosition = { sheetId, col: col - 1, row };
+            while (this.getters.getEvaluatedCell(leftPosition).type !== CellValueType.empty ||
+                this.getters.getCell(leftPosition)?.content) {
                 row += 1;
-                left = this.getters.getEvaluatedCell({ sheetId, col: col - 1, row });
+                leftPosition = { sheetId, col: col - 1, row };
             }
         }
         if (row === zone.bottom) {
             col = zone.right;
             if (col <= this.getters.getNumberCols(sheetId)) {
-                let right = this.getters.getEvaluatedCell({ sheetId, col: col + 1, row });
-                while (right.type !== CellValueType.empty) {
+                let rightPosition = { sheetId, col: col + 1, row };
+                while (this.getters.getEvaluatedCell(rightPosition).type !== CellValueType.empty ||
+                    this.getters.getCell(rightPosition)?.content) {
                     row += 1;
-                    right = this.getters.getEvaluatedCell({ sheetId, col: col + 1, row });
+                    rightPosition = { sheetId, col: col + 1, row };
                 }
             }
         }
@@ -43000,6 +43022,7 @@ class Session extends EventBus {
     isReplayingInitialRevisions = false;
     processedRevisions = new Set();
     uuidGenerator = new UuidGenerator();
+    lastLocalOperation;
     /**
      * Manages the collaboration between multiple users on the same spreadsheet.
      * It can forward local state changes to other users to ensure they all eventually
@@ -43032,6 +43055,11 @@ class Session extends EventBus {
             return;
         const revision = new Revision(this.uuidGenerator.uuidv4(), this.clientId, commands, rootCommand, changes, Date.now());
         this.revisions.append(revision.id, revision);
+        // REQUEST_REDO just repeats the last operation, the
+        // last operation is still the same and should not change.
+        if (rootCommand.type !== "REQUEST_REDO") {
+            this.lastLocalOperation = revision;
+        }
         this.trigger("new-local-state-update", { id: revision.id });
         this.sendUpdateMessage({
             type: "REMOTE_REVISION",
@@ -43128,19 +43156,10 @@ class Session extends EventBus {
         return this.pendingMessages.length === 0;
     }
     /**
-     * Get the last local revision whose root command isn't in the given list of ignored commands
+     * Get the last local revision
      * */
-    getLastLocalNonEmptyRevision(ignoredRootCommands) {
-        const revisions = this.revisions.getRevertedExecution();
-        for (const rev of revisions) {
-            if (rev.rootCommand === "SNAPSHOT")
-                return undefined;
-            if (!rev.rootCommand || rev.rootCommand === "REMOTE")
-                continue;
-            if (!ignoredRootCommands.includes(rev.rootCommand?.type) && rev.commands.length)
-                return rev;
-        }
-        return undefined;
+    getLastLocalNonEmptyRevision() {
+        return this.lastLocalOperation;
     }
     _move(position) {
         // this method is debounced and might be called after the client
@@ -43200,7 +43219,7 @@ class Session extends EventBus {
                 break;
             case "REMOTE_REVISION":
                 const { clientId, commands, timestamp } = message;
-                const revision = new Revision(message.nextRevisionId, clientId, commands, "REMOTE", undefined, timestamp);
+                const revision = new Revision(message.nextRevisionId, clientId, commands, undefined, undefined, timestamp);
                 if (revision.clientId !== this.clientId) {
                     this.revisions.insert(revision.id, revision, message.serverRevisionId);
                     const pendingCommands = this.pendingMessages
@@ -43213,10 +43232,11 @@ class Session extends EventBus {
                 }
                 break;
             case "SNAPSHOT_CREATED": {
-                const revision = new Revision(message.nextRevisionId, "server", [], "SNAPSHOT", undefined, Date.now());
+                const revision = new Revision(message.nextRevisionId, "server", [], undefined, undefined, Date.now());
                 this.revisions.insert(revision.id, revision, message.serverRevisionId);
                 this.dropPendingHistoryMessages();
                 this.trigger("snapshot");
+                this.lastLocalOperation = undefined;
                 break;
             }
         }
@@ -45861,6 +45881,9 @@ class SelectionInputsManagerPlugin extends UIPlugin {
     // Other
     // ---------------------------------------------------------------------------
     initInput(id, initialRanges, inputHasSingleRange = false) {
+        if (this.inputs[id]) {
+            this.unfocus();
+        }
         this.inputs[id] = new SelectionInputPlugin(this.config, initialRanges, inputHasSingleRange);
         if (initialRanges.length === 0) {
             const input = this.inputs[id];
@@ -46569,7 +46592,7 @@ class HistoryPlugin extends UIPlugin {
      * Ignore standard undo/redo revisions (that are empty)
      */
     getPossibleRevisionToRepeat() {
-        return this.session.getLastLocalNonEmptyRevision(["REQUEST_REDO"]);
+        return this.session.getLastLocalNonEmptyRevision();
     }
 }
 
@@ -48040,6 +48063,12 @@ class FilterEvaluationPlugin extends UIPlugin {
                 break;
             case "HIDE_COLUMNS_ROWS":
             case "UNHIDE_COLUMNS_ROWS":
+            case "GROUP_HEADERS":
+            case "UNGROUP_HEADERS":
+            case "FOLD_HEADER_GROUP":
+            case "UNFOLD_HEADER_GROUP":
+            case "FOLD_ALL_HEADER_GROUPS":
+            case "UNFOLD_ALL_HEADER_GROUPS":
                 this.updateHiddenRows();
                 break;
             case "UPDATE_FILTER":
@@ -48741,7 +48770,7 @@ class GridSelectionPlugin extends UIPlugin {
             sheetId: cmd.sheetId,
             base: cmd.base,
             quantity: thickness,
-            position: "before",
+            position: cmd.position,
         });
         const isCol = cmd.dimension === "COL";
         const start = cmd.elements[0];
@@ -48758,12 +48787,13 @@ class GridSelectionPlugin extends UIPlugin {
             },
         ];
         const state = new ClipboardCellsState(target, "CUT", this.getters, this.dispatch, this.selection);
+        const base = isBasedBefore ? cmd.base : cmd.base + 1;
         const pasteTarget = [
             {
-                left: isCol ? cmd.base : 0,
-                right: isCol ? cmd.base + thickness - 1 : this.getters.getNumberCols(cmd.sheetId) - 1,
-                top: !isCol ? cmd.base : 0,
-                bottom: !isCol ? cmd.base + thickness - 1 : this.getters.getNumberRows(cmd.sheetId) - 1,
+                left: isCol ? base : 0,
+                right: isCol ? base + thickness - 1 : this.getters.getNumberCols(cmd.sheetId) - 1,
+                top: !isCol ? base : 0,
+                bottom: !isCol ? base + thickness - 1 : this.getters.getNumberRows(cmd.sheetId) - 1,
             },
         ];
         state.paste(pasteTarget, { selectTarget: true });
@@ -48798,6 +48828,11 @@ class GridSelectionPlugin extends UIPlugin {
             doesElementsHaveCommonMerges(id, cmd.base - 1, cmd.base)) {
             return "WillRemoveExistingMerge" /* CommandResult.WillRemoveExistingMerge */;
         }
+        const headers = [cmd.base, ...cmd.elements];
+        const maxHeaderValue = isCol ? this.getters.getNumberCols(id) : this.getters.getNumberRows(id);
+        if (headers.some((h) => h < 0 || h >= maxHeaderValue)) {
+            return "InvalidHeaderIndex" /* CommandResult.InvalidHeaderIndex */;
+        }
         return "Success" /* CommandResult.Success */;
     }
     //-------------------------------------------
@@ -48975,9 +49010,6 @@ class InternalViewport {
      */
     adjustPosition(position) {
         const sheetId = this.sheetId;
-        if (!position) {
-            position = this.getters.getSheetPosition(sheetId);
-        }
         const mainCellPosition = this.getters.getMainCellPosition({ sheetId, ...position });
         const { col, row } = this.getters.getNextVisibleCellPosition(mainCellPosition);
         if (isInside(col, this.boundaries.top, this.boundaries)) {
@@ -48990,56 +49022,50 @@ class InternalViewport {
     adjustPositionX(targetCol) {
         const sheetId = this.sheetId;
         const { end } = this.getters.getColDimensions(sheetId, targetCol);
-        const maxCol = this.getters.getNumberCols(sheetId);
         if (this.offsetX + this.offsetCorrectionX + this.viewportWidth < end) {
+            const maxCol = this.getters.getNumberCols(sheetId);
             let finalTarget = targetCol;
-            while (this.getters.isColHidden(sheetId, finalTarget) && targetCol < maxCol) {
+            while (this.getters.isColHidden(sheetId, finalTarget) && finalTarget < maxCol) {
                 finalTarget++;
             }
             const finalTargetEnd = this.getters.getColDimensions(sheetId, finalTarget).end;
             const startIndex = this.searchHeaderIndex("COL", finalTargetEnd - this.viewportWidth - this.offsetCorrectionX, this.boundaries.left);
-            this.offsetX =
+            this.offsetScrollbarX =
                 this.getters.getColDimensions(sheetId, startIndex).end - this.offsetCorrectionX;
-            this.offsetScrollbarX = this.offsetX;
-            this.adjustViewportZoneX();
         }
         else if (this.left > targetCol) {
             let finalTarget = targetCol;
-            while (this.getters.isColHidden(sheetId, finalTarget) && targetCol > 0) {
+            while (this.getters.isColHidden(sheetId, finalTarget) && finalTarget > 0) {
                 finalTarget--;
             }
-            this.offsetX =
+            this.offsetScrollbarX =
                 this.getters.getColDimensions(sheetId, finalTarget).start - this.offsetCorrectionX;
-            this.offsetScrollbarX = this.offsetX;
-            this.adjustViewportZoneX();
         }
+        this.adjustViewportZoneX();
     }
     adjustPositionY(targetRow) {
         const sheetId = this.sheetId;
         const { end } = this.getters.getRowDimensions(sheetId, targetRow);
-        const maxRow = this.getters.getNumberRows(sheetId);
         if (this.offsetY + this.viewportHeight + this.offsetCorrectionY < end) {
+            const maxRow = this.getters.getNumberRows(sheetId);
             let finalTarget = targetRow;
-            while (this.getters.isRowHidden(sheetId, finalTarget) && targetRow < maxRow) {
+            while (this.getters.isRowHidden(sheetId, finalTarget) && finalTarget < maxRow) {
                 finalTarget++;
             }
             const finalTargetEnd = this.getters.getRowDimensions(sheetId, finalTarget).end;
             const startIndex = this.searchHeaderIndex("ROW", finalTargetEnd - this.viewportHeight - this.offsetCorrectionY, this.boundaries.top);
-            this.offsetY =
+            this.offsetScrollbarY =
                 this.getters.getRowDimensions(sheetId, startIndex).end - this.offsetCorrectionY;
-            this.offsetScrollbarY = this.offsetY;
-            this.adjustViewportZoneY();
         }
         else if (this.top > targetRow) {
             let finalTarget = targetRow;
-            while (this.getters.isRowHidden(sheetId, finalTarget) && targetRow > 0) {
+            while (this.getters.isRowHidden(sheetId, finalTarget) && finalTarget > 0) {
                 finalTarget--;
             }
-            this.offsetY =
+            this.offsetScrollbarY =
                 this.getters.getRowDimensions(sheetId, finalTarget).start - this.offsetCorrectionY;
-            this.offsetScrollbarY = this.offsetY;
-            this.adjustViewportZoneY();
         }
+        this.adjustViewportZoneY();
     }
     setViewportOffset(offsetX, offsetY) {
         this.setViewportOffsetX(offsetX);
@@ -49055,11 +49081,10 @@ class InternalViewport {
      * @returns Computes the absolute coordinate of a given zone inside the viewport
      */
     getRect(zone) {
-        const targetZone = intersection(zone, this.zone);
+        const targetZone = intersection(zone, this);
         if (targetZone) {
-            const x = this.getters.getColRowOffset("COL", this.zone.left, targetZone.left) +
-                this.offsetCorrectionX;
-            const y = this.getters.getColRowOffset("ROW", this.zone.top, targetZone.top) + this.offsetCorrectionY;
+            const x = this.getters.getColRowOffset("COL", this.left, targetZone.left) + this.offsetCorrectionX;
+            const y = this.getters.getColRowOffset("ROW", this.top, targetZone.top) + this.offsetCorrectionY;
             const width = Math.min(this.getters.getColRowOffset("COL", targetZone.left, targetZone.right + 1), this.viewportWidth);
             const height = Math.min(this.getters.getColRowOffset("ROW", targetZone.top, targetZone.bottom + 1), this.viewportHeight);
             return {
@@ -49069,9 +49094,7 @@ class InternalViewport {
                 height,
             };
         }
-        else {
-            return undefined;
-        }
+        return undefined;
     }
     isVisible(col, row) {
         const isInside = row <= this.bottom && row >= this.top && col >= this.left && col <= this.right;
@@ -49079,7 +49102,6 @@ class InternalViewport {
             !this.getters.isColHidden(this.sheetId, col) &&
             !this.getters.isRowHidden(this.sheetId, row));
     }
-    // PRIVATE
     searchHeaderIndex(dimension, position, startIndex = 0) {
         const sheetId = this.sheetId;
         const headers = this.getters.getNumberHeaders(sheetId, dimension);
@@ -49102,9 +49124,6 @@ class InternalViewport {
         }
         return -1;
     }
-    get zone() {
-        return { left: this.left, right: this.right, top: this.top, bottom: this.bottom };
-    }
     setViewportOffsetX(offsetX) {
         if (!this.canScrollHorizontally) {
             return;
@@ -49129,11 +49148,6 @@ class InternalViewport {
                 this.offsetScrollbarX = Math.max(0, viewportWidth - this.viewportWidth);
             }
         }
-        this.left = this.getColIndex(this.offsetScrollbarX);
-        this.right = this.getColIndex(this.offsetScrollbarX + this.viewportWidth);
-        if (this.right === -1) {
-            this.right = this.boundaries.right;
-        }
         this.adjustViewportZoneX();
     }
     /** Corrects the viewport's vertical offset based on the current structure
@@ -49146,11 +49160,6 @@ class InternalViewport {
                 this.offsetScrollbarY = Math.max(0, paneHeight - this.viewportHeight);
             }
         }
-        this.top = this.getRowIndex(this.offsetScrollbarY);
-        this.bottom = this.getRowIndex(this.offsetScrollbarY + this.viewportHeight);
-        if (this.bottom === -1) {
-            this.bottom = this.boundaries.bottom;
-        }
         this.adjustViewportZoneY();
     }
     /** Updates the pane zone and snapped offset based on its horizontal
@@ -49528,7 +49537,7 @@ class SheetViewPlugin extends UIPlugin {
     isVisibleInViewport({ sheetId, col, row }) {
         return this.getSubViewports(sheetId).some((pane) => pane.isVisible(col, row));
     }
-    // => return s the new offset
+    // => returns the new offset
     getEdgeScrollCol(x, previousX, startingX) {
         let canEdgeScroll = false;
         let direction = 0;
@@ -49843,10 +49852,13 @@ class HeaderPositionsUIPlugin extends UIPlugin {
                 }
                 break;
             case "UPDATE_CELL":
-                if ("content" in cmd || "format" in cmd || cmd.style?.fontSize !== undefined) {
+                if ("content" in cmd || "format" in cmd) {
                     this.headerPositions = {};
                     this.isDirty = true;
                 }
+                else {
+                    this.headerPositions[cmd.sheetId] = this.computeHeaderPositionsOfSheet(cmd.sheetId);
+                }
                 break;
             case "UPDATE_FILTER":
             case "REMOVE_FILTER_TABLE":
@@ -52003,6 +52015,8 @@ css /* scss */ `
     *:before,
     *:after {
       box-sizing: content-box;
+      /** rtl not supported ATM */
+      direction: ltr;
     }
     .o-separator {
       border-bottom: ${MENU_SEPARATOR_BORDER_WIDTH}px solid ${SEPARATOR_COLOR};
@@ -56008,6 +56022,6 @@ const constants = {
 export { AbstractChart, CommandResult, CorePlugin, DispatchResult, EvaluationError, Model, Registry, Revision, SPREADSHEET_DIMENSIONS, Spreadsheet, UIPlugin, __info__, addFunction, astToFormula, compile, compileTokens, components, constants, convertAstNodes, coreTypes, findCellInNewZone, functionCache, helpers, hooks, invalidateCFEvaluationCommands, invalidateDependenciesCommands, invalidateEvaluationCommands, iterateAstNodes, links, load, parse, parseTokens, readonlyAllowedCommands, registries, setDefaultSheetViewSize, setTranslationMethod, tokenize };
 
 
-__info__.version = "17.1.0-alpha.4";
-__info__.date = "2023-11-24T13:12:24.882Z";
-__info__.hash = "255821b";
+__info__.version = "17.1.0-alpha.5";
+__info__.date = "2023-12-05T09:51:40.034Z";
+__info__.hash = "c2823eb";