@odoo/o-spreadsheet 17.4.20 → 17.4.22

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 17.4.20
- * @date 2025-01-31T08:00:07.103Z
- * @hash 54a344a
+ * @version 17.4.22
+ * @date 2025-02-10T09:14:48.889Z
+ * @hash 667f2b1
  */
 
 'use strict';
@@ -748,9 +748,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.
@@ -18883,22 +18890,23 @@ const HLOOKUP = {
     description: _t("Horizontal lookup"),
     args: [
         arg("search_key (any)", _t("The value to search for. For example, 42, 'Cats', or I24.")),
-        arg("range (range)", _t("The range to consider for the search. The first row in the range is searched for the key specified in search_key.")),
+        arg("range (any, range)", _t("The range to consider for the search. The first row in the range is searched for the key specified in search_key.")),
         arg("index (number)", _t("The row index of the value to be returned, where the first row in range is numbered 1.")),
         arg(`is_sorted (boolean, default=${DEFAULT_IS_SORTED})`, _t("Indicates whether the row to be searched (the first row of the specified range) is sorted, in which case the closest match for search_key will be returned.")),
     ],
     compute: function (searchKey, range, index, isSorted = { value: DEFAULT_IS_SORTED }) {
         const _index = Math.trunc(toNumber(index?.value, this.locale));
-        assert(() => 1 <= _index && _index <= range[0].length, _t("[[FUNCTION_NAME]] evaluates to an out of bounds range."));
+        const _range = toMatrix(range);
+        assert(() => 1 <= _index && _index <= _range[0].length, _t("[[FUNCTION_NAME]] evaluates to an out of bounds range."));
         if (searchKey && isEvaluationError(searchKey.value)) {
             return searchKey;
         }
         const getValueFromRange = (range, index) => range[index][0].value;
         const _isSorted = toBoolean(isSorted.value);
         const colIndex = _isSorted
-            ? dichotomicSearch(range, searchKey, "nextSmaller", "asc", range.length, getValueFromRange)
-            : linearSearch(range, searchKey, "wildcard", range.length, getValueFromRange);
-        const col = range[colIndex];
+            ? dichotomicSearch(_range, searchKey, "nextSmaller", "asc", _range.length, getValueFromRange)
+            : linearSearch(_range, searchKey, "wildcard", _range.length, getValueFromRange);
+        const col = _range[colIndex];
         if (col === undefined) {
             return valueNotAvailable(searchKey);
         }
@@ -18997,35 +19005,37 @@ const LOOKUP = {
     description: _t("Look up a value."),
     args: [
         arg("search_key (any)", _t("The value to search for. For example, 42, 'Cats', or I24.")),
-        arg("search_array (range)", _t("One method of using this function is to provide a single sorted row or column search_array to look through for the search_key with a second argument result_range. The other way is to combine these two arguments into one search_array where the first row or column is searched and a value is returned from the last row or column in the array. If search_key is not found, a non-exact match may be returned.")),
-        arg("result_range (range, optional)", _t("The range from which to return a result. The value returned corresponds to the location where search_key is found in search_range. This range must be only a single row or column and should not be used if using the search_result_array method.")),
+        arg("search_array (any, range)", _t("One method of using this function is to provide a single sorted row or column search_array to look through for the search_key with a second argument result_range. The other way is to combine these two arguments into one search_array where the first row or column is searched and a value is returned from the last row or column in the array. If search_key is not found, a non-exact match may be returned.")),
+        arg("result_range (any, range, optional)", _t("The range from which to return a result. The value returned corresponds to the location where search_key is found in search_range. This range must be only a single row or column and should not be used if using the search_result_array method.")),
     ],
     compute: function (searchKey, searchArray, resultRange) {
-        let nbCol = searchArray.length;
-        let nbRow = searchArray[0].length;
+        const _searchArray = toMatrix(searchArray);
+        const _resultRange = toMatrix(resultRange);
+        let nbCol = _searchArray.length;
+        let nbRow = _searchArray[0].length;
         const verticalSearch = nbRow >= nbCol;
         const getElement = verticalSearch
             ? (range, index) => range[0][index].value
             : (range, index) => range[index][0].value;
         const rangeLength = verticalSearch ? nbRow : nbCol;
-        const index = dichotomicSearch(searchArray, searchKey, "nextSmaller", "asc", rangeLength, getElement);
+        const index = dichotomicSearch(_searchArray, searchKey, "nextSmaller", "asc", rangeLength, getElement);
         if (index === -1 ||
-            (verticalSearch && searchArray[0][index] === undefined) ||
-            (!verticalSearch && searchArray[index][nbRow - 1] === undefined)) {
+            (verticalSearch && _searchArray[0][index] === undefined) ||
+            (!verticalSearch && _searchArray[index][nbRow - 1] === undefined)) {
             return valueNotAvailable(searchKey);
         }
-        if (resultRange === undefined) {
-            return verticalSearch ? searchArray[nbCol - 1][index] : searchArray[index][nbRow - 1];
+        if (_resultRange[0].length === 0) {
+            return verticalSearch ? _searchArray[nbCol - 1][index] : _searchArray[index][nbRow - 1];
         }
-        nbCol = resultRange.length;
-        nbRow = resultRange[0].length;
+        nbCol = _resultRange.length;
+        nbRow = _resultRange[0].length;
         assert(() => nbCol === 1 || nbRow === 1, _t("The result_range must be a single row or a single column."));
         if (nbCol > 1) {
             assert(() => index <= nbCol - 1, _t("[[FUNCTION_NAME]] evaluates to an out of range row value %s.", (index + 1).toString()));
-            return resultRange[index][0];
+            return _resultRange[index][0];
         }
         assert(() => index <= nbRow - 1, _t("[[FUNCTION_NAME]] evaluates to an out of range column value %s.", (index + 1).toString()));
-        return resultRange[0][index];
+        return _resultRange[0][index];
     },
     isExported: true,
 };
@@ -19042,28 +19052,29 @@ const MATCH = {
     ],
     compute: function (searchKey, range, searchType = { value: DEFAULT_SEARCH_TYPE }) {
         let _searchType = toNumber(searchType, this.locale);
-        const nbCol = range.length;
-        const nbRow = range[0].length;
+        const _range = toMatrix(range);
+        const nbCol = _range.length;
+        const nbRow = _range[0].length;
         assert(() => nbCol === 1 || nbRow === 1, _t("The range must be a single row or a single column."));
         let index = -1;
         const getElement = nbCol === 1
-            ? (range, index) => range[0][index].value
-            : (range, index) => range[index][0].value;
-        const rangeLen = nbCol === 1 ? range[0].length : range.length;
+            ? (_range, index) => _range[0][index].value
+            : (_range, index) => _range[index][0].value;
+        const rangeLen = nbCol === 1 ? _range[0].length : _range.length;
         _searchType = Math.sign(_searchType);
         switch (_searchType) {
             case 1:
-                index = dichotomicSearch(range, searchKey, "nextSmaller", "asc", rangeLen, getElement);
+                index = dichotomicSearch(_range, searchKey, "nextSmaller", "asc", rangeLen, getElement);
                 break;
             case 0:
-                index = linearSearch(range, searchKey, "wildcard", rangeLen, getElement);
+                index = linearSearch(_range, searchKey, "wildcard", rangeLen, getElement);
                 break;
             case -1:
-                index = dichotomicSearch(range, searchKey, "nextGreater", "desc", rangeLen, getElement);
+                index = dichotomicSearch(_range, searchKey, "nextGreater", "desc", rangeLen, getElement);
                 break;
         }
-        if ((nbCol === 1 && range[0][index] === undefined) ||
-            (nbCol !== 1 && range[index] === undefined)) {
+        if ((nbCol === 1 && _range[0][index] === undefined) ||
+            (nbCol !== 1 && _range[index] === undefined)) {
             return valueNotAvailable(searchKey);
         }
         return index + 1;
@@ -19117,16 +19128,17 @@ const VLOOKUP = {
     ],
     compute: function (searchKey, range, index, isSorted = { value: DEFAULT_IS_SORTED }) {
         const _index = Math.trunc(toNumber(index?.value, this.locale));
-        assert(() => 1 <= _index && _index <= range.length, _t("[[FUNCTION_NAME]] evaluates to an out of bounds range."));
+        const _range = toMatrix(range);
+        assert(() => 1 <= _index && _index <= _range.length, _t("[[FUNCTION_NAME]] evaluates to an out of bounds range."));
         if (searchKey && isEvaluationError(searchKey.value)) {
             return searchKey;
         }
         const getValueFromRange = (range, index) => range[0][index].value;
         const _isSorted = toBoolean(isSorted.value);
         const rowIndex = _isSorted
-            ? dichotomicSearch(range, searchKey, "nextSmaller", "asc", range[0].length, getValueFromRange)
-            : linearSearch(range, searchKey, "wildcard", range[0].length, getValueFromRange);
-        const value = range[_index - 1][rowIndex];
+            ? dichotomicSearch(_range, searchKey, "nextSmaller", "asc", _range[0].length, getValueFromRange)
+            : linearSearch(_range, searchKey, "wildcard", _range[0].length, getValueFromRange);
+        const value = _range[_index - 1][rowIndex];
         if (value === undefined) {
             return valueNotAvailable(searchKey);
         }
@@ -19163,30 +19175,32 @@ const XLOOKUP = {
     compute: function (searchKey, lookupRange, returnRange, defaultValue, matchMode = { value: DEFAULT_MATCH_MODE }, searchMode = { value: DEFAULT_SEARCH_MODE }) {
         const _matchMode = Math.trunc(toNumber(matchMode.value, this.locale));
         const _searchMode = Math.trunc(toNumber(searchMode.value, this.locale));
-        assert(() => lookupRange.length === 1 || lookupRange[0].length === 1, _t("lookup_range should be either a single row or single column."));
+        const _lookupRange = toMatrix(lookupRange);
+        const _returnRange = toMatrix(returnRange);
+        assert(() => _lookupRange.length === 1 || _lookupRange[0].length === 1, _t("lookup_range should be either a single row or single column."));
         assert(() => [-1, 1, -2, 2].includes(_searchMode), _t("search_mode should be a value in [-1, 1, -2, 2]."));
         assert(() => [-1, 0, 1, 2].includes(_matchMode), _t("match_mode should be a value in [-1, 0, 1, 2]."));
-        const lookupDirection = lookupRange.length === 1 ? "col" : "row";
+        const lookupDirection = _lookupRange.length === 1 ? "col" : "row";
         assert(() => !(_matchMode === 2 && [-2, 2].includes(_searchMode)), _t("the search and match mode combination is not supported for XLOOKUP evaluation."));
         assert(() => lookupDirection === "col"
-            ? returnRange[0].length === lookupRange[0].length
-            : returnRange.length === lookupRange.length, _t("return_range should have the same dimensions as lookup_range."));
+            ? _returnRange[0].length === _lookupRange[0].length
+            : _returnRange.length === _lookupRange.length, _t("return_range should have the same dimensions as lookup_range."));
         if (searchKey && isEvaluationError(searchKey.value)) {
             return [[searchKey]];
         }
         const getElement = lookupDirection === "col"
             ? (range, index) => range[0][index].value
             : (range, index) => range[index][0].value;
-        const rangeLen = lookupDirection === "col" ? lookupRange[0].length : lookupRange.length;
+        const rangeLen = lookupDirection === "col" ? _lookupRange[0].length : _lookupRange.length;
         const mode = MATCH_MODE[_matchMode];
         const reverseSearch = _searchMode === -1;
         const index = _searchMode === 2 || _searchMode === -2
-            ? dichotomicSearch(lookupRange, searchKey, mode, _searchMode === 2 ? "asc" : "desc", rangeLen, getElement)
-            : linearSearch(lookupRange, searchKey, mode, rangeLen, getElement, reverseSearch);
+            ? dichotomicSearch(_lookupRange, searchKey, mode, _searchMode === 2 ? "asc" : "desc", rangeLen, getElement)
+            : linearSearch(_lookupRange, searchKey, mode, rangeLen, getElement, reverseSearch);
         if (index !== -1) {
             return lookupDirection === "col"
-                ? returnRange.map((col) => [col[index]])
-                : [returnRange[index]];
+                ? _returnRange.map((col) => [col[index]])
+                : [_returnRange[index]];
         }
         if (defaultValue === undefined) {
             return valueNotAvailable(searchKey);
@@ -33540,8 +33554,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,
@@ -33552,8 +33566,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(() => {
@@ -34001,7 +34015,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),
         });
     }
@@ -36013,6 +36027,7 @@ class PivotLayoutConfigurator extends owl.Component {
         unusedMeasureFields: Array,
         unusedDateTimeGranularities: Object,
         allGranularities: Array,
+        getScrollableContainerEl: { type: Function, optional: true },
     };
     dimensionsRef = owl.useRef("pivot-dimensions");
     dragAndDrop = useDragAndDropListItems();
@@ -36041,7 +36056,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) {
@@ -36081,7 +36096,7 @@ class PivotLayoutConfigurator extends owl.Component {
             draggedItemId: measure.name,
             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) {
@@ -37520,6 +37535,7 @@ class PivotSpreadsheetSidePanel extends owl.Component {
     };
     store;
     state;
+    pivotSidePanelRef = owl.useRef("pivotSidePanel");
     setup() {
         this.store = useLocalStore(PivotSidePanelStore, this.props.pivotId);
         this.state = owl.useState({
@@ -37548,6 +37564,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];
@@ -41220,11 +41239,10 @@ class GridRenderer {
         switch (layer) {
             case "Background":
                 this.drawGlobalBackground(renderingContext);
-                for (const zone of this.getters.getAllActiveViewportsZones()) {
+                for (const { zone, rect } of this.getters.getAllActiveViewportsZonesAndRect()) {
                     const { ctx } = renderingContext;
                     ctx.save();
                     ctx.beginPath();
-                    const rect = this.getters.getVisibleRect(zone);
                     ctx.rect(rect.x, rect.y, rect.width, rect.height);
                     ctx.clip();
                     const boxes = this.getGridBoxes(zone);
@@ -41494,10 +41512,8 @@ class GridRenderer {
         const { ctx, thinLineWidth } = renderingContext;
         const visibleCols = this.getters.getSheetViewVisibleCols();
         const left = visibleCols[0];
-        const right = visibleCols[visibleCols.length - 1];
         const visibleRows = this.getters.getSheetViewVisibleRows();
         const top = visibleRows[0];
-        const bottom = visibleRows[visibleRows.length - 1];
         const { width, height } = this.getters.getSheetViewDimensionWithHeaders();
         const selection = this.getters.getSelectedZones();
         const selectedCols = getZonesCols(selection);
@@ -41513,7 +41529,7 @@ class GridRenderer {
         ctx.lineWidth = thinLineWidth;
         ctx.strokeStyle = "#333";
         // Columns headers background
-        for (let col = left; col <= right; col++) {
+        for (const col of visibleCols) {
             const colZone = { left: col, right: col, top: 0, bottom: numberOfRows - 1 };
             const { x, width } = this.getters.getVisibleRect(colZone);
             const isColActive = activeCols.has(col);
@@ -41530,7 +41546,7 @@ class GridRenderer {
             ctx.fillRect(x, 0, width, HEADER_HEIGHT);
         }
         // Rows headers background
-        for (let row = top; row <= bottom; row++) {
+        for (const row of visibleRows) {
             const rowZone = { top: row, bottom: row, left: 0, right: numberOfCols - 1 };
             const { y, height } = this.getters.getVisibleRect(rowZone);
             const isRowActive = activeRows.has(row);
@@ -41556,21 +41572,21 @@ class GridRenderer {
         ctx.stroke();
         ctx.beginPath();
         // column text + separator
-        for (const i of visibleCols) {
-            const colSize = this.getters.getColSize(sheetId, i);
-            const colName = numberToLetters(i);
-            ctx.fillStyle = activeCols.has(i) ? "#fff" : TEXT_HEADER_COLOR;
-            let colStart = this.getHeaderOffset("COL", left, i);
+        for (const col of visibleCols) {
+            const colSize = this.getters.getColSize(sheetId, col);
+            const colName = numberToLetters(col);
+            ctx.fillStyle = activeCols.has(col) ? "#fff" : TEXT_HEADER_COLOR;
+            let colStart = this.getHeaderOffset("COL", left, col);
             ctx.fillText(colName, colStart + colSize / 2, HEADER_HEIGHT / 2);
             ctx.moveTo(colStart + colSize, 0);
             ctx.lineTo(colStart + colSize, HEADER_HEIGHT);
         }
         // row text + separator
-        for (const i of visibleRows) {
-            const rowSize = this.getters.getRowSize(sheetId, i);
-            ctx.fillStyle = activeRows.has(i) ? "#fff" : TEXT_HEADER_COLOR;
-            let rowStart = this.getHeaderOffset("ROW", top, i);
-            ctx.fillText(String(i + 1), HEADER_WIDTH / 2, rowStart + rowSize / 2);
+        for (const row of visibleRows) {
+            const rowSize = this.getters.getRowSize(sheetId, row);
+            ctx.fillStyle = activeRows.has(row) ? "#fff" : TEXT_HEADER_COLOR;
+            let rowStart = this.getHeaderOffset("ROW", top, row);
+            ctx.fillText(String(row + 1), HEADER_WIDTH / 2, rowStart + rowSize / 2);
             ctx.moveTo(0, rowStart + rowSize);
             ctx.lineTo(HEADER_WIDTH, rowStart + rowSize);
         }
@@ -41868,6 +41884,9 @@ function useGridDrawing(refName, model, canvasSize) {
         canvas.width = width * dpr;
         canvas.height = height * dpr;
         canvas.setAttribute("style", `width:${width}px;height:${height}px;`);
+        if (width === 0 || height === 0) {
+            return;
+        }
         // Imagine each pixel as a large square. The whole-number coordinates (0, 1, 2…)
         // are the edges of the squares. If you draw a one-unit-wide line between whole-number
         // coordinates, it will overlap opposite sides of the pixel square, and the resulting
@@ -47600,7 +47619,7 @@ class BordersPlugin extends CorePlugin {
     getCommonSides(border1, border2) {
         const commonBorder = {};
         for (let side of ["top", "bottom", "left", "right"]) {
-            if (border1[side] && border1[side] === border2[side]) {
+            if (border1[side] && deepEquals(border1[side], border2[side])) {
                 commonBorder[side] = border1[side];
             }
         }
@@ -60943,8 +60962,17 @@ class InternalViewport {
         this.getters = getters;
         this.sheetId = sheetId;
         this.boundaries = boundaries;
-        this.viewportWidth = sizeInGrid.width;
-        this.viewportHeight = sizeInGrid.height;
+        if (sizeInGrid.width < 0 || sizeInGrid.height < 0) {
+            throw new Error("Viewport size cannot be negative");
+        }
+        this.viewportWidth = sizeInGrid.height && sizeInGrid.width;
+        this.viewportHeight = sizeInGrid.width && sizeInGrid.height;
+        this.top = boundaries.top;
+        this.bottom = boundaries.bottom;
+        this.left = boundaries.left;
+        this.right = boundaries.right;
+        this.offsetX = offsets.x;
+        this.offsetY = offsets.y;
         this.offsetScrollbarX = offsets.x;
         this.offsetScrollbarY = offsets.y;
         this.canScrollVertically = options.canScrollVertically;
@@ -60987,9 +61015,9 @@ class InternalViewport {
             Math.min(topRowSize, this.viewportHeight - lastRowSize) // Add pixels that allows the snapping at maximum vertical scroll
             );
             height = Math.max(height, this.viewportHeight); // if the viewport grid size is smaller than its client height, return client height
-        }
-        if (lastRowEnd + FOOTER_HEIGHT > height && !this.getters.isReadonly()) {
-            height += FOOTER_HEIGHT;
+            if (lastRowEnd + FOOTER_HEIGHT > height && !this.getters.isReadonly()) {
+                height += FOOTER_HEIGHT;
+            }
         }
         return { width, height };
     }
@@ -61130,6 +61158,9 @@ class InternalViewport {
             !this.getters.isRowHidden(this.sheetId, row));
     }
     searchHeaderIndex(dimension, position, startIndex = 0) {
+        if (this.viewportWidth <= 0 || this.viewportHeight <= 0) {
+            return -1;
+        }
         const sheetId = this.sheetId;
         const headers = this.getters.getNumberHeaders(sheetId, dimension);
         // using a binary search:
@@ -61166,7 +61197,7 @@ class InternalViewport {
         this.adjustViewportZoneY();
     }
     /** Corrects the viewport's horizontal offset based on the current structure
-     *  To make sure that at least on column is visible inside the viewport.
+     *  To make sure that at least one column is visible inside the viewport.
      */
     adjustViewportOffsetX() {
         if (this.canScrollHorizontally) {
@@ -61178,7 +61209,7 @@ class InternalViewport {
         this.adjustViewportZoneX();
     }
     /** Corrects the viewport's vertical offset based on the current structure
-     *  To make sure that at least on row is visible inside the viewport.
+     *  To make sure that at least one row is visible inside the viewport.
      */
     adjustViewportOffsetY() {
         if (this.canScrollVertically) {
@@ -61195,11 +61226,14 @@ class InternalViewport {
         const sheetId = this.sheetId;
         this.left = this.searchHeaderIndex("COL", this.offsetScrollbarX, this.boundaries.left);
         this.right = Math.min(this.boundaries.right, this.searchHeaderIndex("COL", this.viewportWidth, this.left));
+        if (!this.viewportWidth) {
+            return;
+        }
         if (this.left === -1) {
             this.left = this.boundaries.left;
         }
         if (this.right === -1) {
-            this.right = this.getters.getNumberCols(sheetId) - 1;
+            this.right = this.boundaries.right;
         }
         this.offsetX =
             this.getters.getColDimensions(sheetId, this.left).start -
@@ -61211,11 +61245,14 @@ class InternalViewport {
         const sheetId = this.sheetId;
         this.top = this.searchHeaderIndex("ROW", this.offsetScrollbarY, this.boundaries.top);
         this.bottom = Math.min(this.boundaries.bottom, this.searchHeaderIndex("ROW", this.viewportHeight, this.top));
+        if (!this.viewportHeight) {
+            return;
+        }
         if (this.top === -1) {
             this.top = this.boundaries.top;
         }
         if (this.bottom === -1) {
-            this.bottom = this.getters.getNumberRows(sheetId) - 1;
+            this.bottom = this.boundaries.bottom;
         }
         this.offsetY =
             this.getters.getRowDimensions(sheetId, this.top).start -
@@ -61289,7 +61326,7 @@ class SheetViewPlugin extends UIPlugin {
         "isPositionVisible",
         "getColDimensionsInViewport",
         "getRowDimensionsInViewport",
-        "getAllActiveViewportsZones",
+        "getAllActiveViewportsZonesAndRect",
         "getRect",
     ];
     viewports = {};
@@ -61522,12 +61559,12 @@ class SheetViewPlugin extends UIPlugin {
         const sheetId = this.getters.getActiveSheetId();
         const viewports = this.getSubViewports(sheetId);
         //TODO ake another commit to eimprove this
-        return [...new Set(viewports.map((v) => range(v.left, v.right + 1)).flat())].filter((col) => !this.getters.isHeaderHidden(sheetId, "COL", col));
+        return [...new Set(viewports.map((v) => range(v.left, v.right + 1)).flat())].filter((col) => col >= 0 && !this.getters.isHeaderHidden(sheetId, "COL", col));
     }
     getSheetViewVisibleRows() {
         const sheetId = this.getters.getActiveSheetId();
         const viewports = this.getSubViewports(sheetId);
-        return [...new Set(viewports.map((v) => range(v.top, v.bottom + 1)).flat())].filter((row) => !this.getters.isHeaderHidden(sheetId, "ROW", row));
+        return [...new Set(viewports.map((v) => range(v.top, v.bottom + 1)).flat())].filter((row) => row >= 0 && !this.getters.isHeaderHidden(sheetId, "ROW", row));
     }
     /**
      * Get the positions of all the cells that are visible in the viewport, taking merges into account.
@@ -61570,19 +61607,19 @@ class SheetViewPlugin extends UIPlugin {
             maxOffsetY: Math.max(0, height - viewport.viewportHeight + 1),
         };
     }
-    getColRowOffsetInViewport(dimension, referenceIndex, index) {
-        const sheetId = this.getters.getActiveSheetId();
-        const visibleCols = this.getters.getSheetViewVisibleCols();
-        const visibleRows = this.getters.getSheetViewVisibleRows();
-        if (index < referenceIndex) {
-            return -this.getColRowOffsetInViewport(dimension, index, referenceIndex);
+    getColRowOffsetInViewport(dimension, referenceHeaderIndex, targetHeaderIndex) {
+        if (targetHeaderIndex < referenceHeaderIndex) {
+            return -this.getColRowOffsetInViewport(dimension, targetHeaderIndex, referenceHeaderIndex);
         }
+        const sheetId = this.getters.getActiveSheetId();
+        const visibleHeaders = dimension === "COL"
+            ? this.getters.getSheetViewVisibleCols()
+            : this.getters.getSheetViewVisibleRows();
+        const startIndex = visibleHeaders.findIndex((header) => referenceHeaderIndex >= header);
+        const endIndex = visibleHeaders.findIndex((header) => targetHeaderIndex <= header);
+        const relevantIndexes = visibleHeaders.slice(startIndex, endIndex);
         let offset = 0;
-        const visibleIndexes = dimension === "COL" ? visibleCols : visibleRows;
-        for (let i = referenceIndex; i < index; i++) {
-            if (!visibleIndexes.includes(i)) {
-                continue;
-            }
+        for (const i of relevantIndexes) {
             offset += this.getters.getHeaderSize(sheetId, dimension, i);
         }
         return offset;
@@ -61629,7 +61666,7 @@ class SheetViewPlugin extends UIPlugin {
         }
         return { canEdgeScroll, direction, delay };
     }
-    getEdgeScrollRow(y, previousY, tartingY) {
+    getEdgeScrollRow(y, previousY, startingY) {
         let canEdgeScroll = false;
         let direction = 0;
         let delay = 0;
@@ -61650,7 +61687,7 @@ class SheetViewPlugin extends UIPlugin {
             delay = scrollDelay(y - height);
             direction = 1;
         }
-        else if (y < offsetCorrectionY && tartingY >= offsetCorrectionY && currentOffsetY > 0) {
+        else if (y < offsetCorrectionY && startingY >= offsetCorrectionY && currentOffsetY > 0) {
             // 2
             canEdgeScroll = true;
             delay = scrollDelay(offsetCorrectionY - y);
@@ -61676,13 +61713,7 @@ class SheetViewPlugin extends UIPlugin {
      */
     getVisibleRectWithoutHeaders(zone) {
         const sheetId = this.getters.getActiveSheetId();
-        const viewportRects = this.getSubViewports(sheetId)
-            .map((viewport) => viewport.getVisibleRect(zone))
-            .filter(isDefined);
-        if (viewportRects.length === 0) {
-            return { x: 0, y: 0, width: 0, height: 0 };
-        }
-        return this.recomposeRect(viewportRects);
+        return this.mapViewportsToRect(sheetId, (viewport) => viewport.getVisibleRect(zone));
     }
     /**
      * Computes the actual size and position (:Rect) of the zone on the canvas
@@ -61690,13 +61721,7 @@ class SheetViewPlugin extends UIPlugin {
      */
     getRect(zone) {
         const sheetId = this.getters.getActiveSheetId();
-        const viewportRects = this.getSubViewports(sheetId)
-            .map((viewport) => viewport.getFullRect(zone))
-            .filter(isDefined);
-        if (viewportRects.length === 0) {
-            return { x: 0, y: 0, width: 0, height: 0 };
-        }
-        const rect = this.recomposeRect(viewportRects);
+        const rect = this.mapViewportsToRect(sheetId, (viewport) => viewport.getFullRect(zone));
         return { ...rect, x: rect.x + this.gridOffsetX, y: rect.y + this.gridOffsetY };
     }
     /**
@@ -61741,9 +61766,18 @@ class SheetViewPlugin extends UIPlugin {
             end: start + (isRowHidden ? 0 : size),
         };
     }
-    getAllActiveViewportsZones() {
+    getAllActiveViewportsZonesAndRect() {
         const sheetId = this.getters.getActiveSheetId();
-        return this.getSubViewports(sheetId);
+        return this.getSubViewports(sheetId).map((viewport) => {
+            return {
+                zone: viewport,
+                rect: {
+                    x: viewport.offsetCorrectionX + this.gridOffsetX,
+                    y: viewport.offsetCorrectionY + this.gridOffsetY,
+                    ...viewport.getMaxSize(),
+                },
+            };
+        });
     }
     // ---------------------------------------------------------------------------
     // Private
@@ -61802,12 +61836,11 @@ class SheetViewPlugin extends UIPlugin {
     }
     /** gets rid of deprecated sheetIds */
     cleanViewports() {
-        const sheetIds = this.getters.getSheetIds();
-        for (let sheetId of Object.keys(this.viewports)) {
-            if (!sheetIds.includes(sheetId)) {
-                delete this.viewports[sheetId];
-            }
+        const newViewport = {};
+        for (const sheetId of this.getters.getSheetIds()) {
+            newViewport[sheetId] = this.viewports[sheetId];
         }
+        this.viewports = newViewport;
     }
     resizeSheetView(height, width, gridOffsetX = 0, gridOffsetY = 0) {
         this.sheetViewHeight = height;
@@ -61817,7 +61850,7 @@ class SheetViewPlugin extends UIPlugin {
         this.recomputeViewports();
     }
     recomputeViewports() {
-        for (let sheetId of Object.keys(this.viewports)) {
+        for (const sheetId of this.getters.getSheetIds()) {
             this.resetViewports(sheetId);
         }
     }
@@ -61839,8 +61872,10 @@ class SheetViewPlugin extends UIPlugin {
         const { xSplit, ySplit } = this.getters.getPaneDivisions(sheetId);
         const nCols = this.getters.getNumberCols(sheetId);
         const nRows = this.getters.getNumberRows(sheetId);
-        const colOffset = this.getters.getColRowOffset("COL", 0, xSplit, sheetId);
-        const rowOffset = this.getters.getColRowOffset("ROW", 0, ySplit, sheetId);
+        const colOffset = Math.min(this.getters.getColRowOffset("COL", 0, xSplit, sheetId), this.sheetViewWidth);
+        const rowOffset = Math.min(this.getters.getColRowOffset("ROW", 0, ySplit, sheetId), this.sheetViewHeight);
+        const unfrozenWidth = Math.max(this.sheetViewWidth - colOffset, 0);
+        const unfrozenHeight = Math.max(this.sheetViewHeight - rowOffset, 0);
         const { xRatio, yRatio } = this.getFrozenSheetViewRatio(sheetId);
         const canScrollHorizontally = xRatio < 1.0;
         const canScrollVertically = yRatio < 1.0;
@@ -61851,14 +61886,14 @@ class SheetViewPlugin extends UIPlugin {
                 new InternalViewport(this.getters, sheetId, { left: 0, right: xSplit - 1, top: 0, bottom: ySplit - 1 }, { width: colOffset, height: rowOffset }, { canScrollHorizontally: false, canScrollVertically: false }, { x: 0, y: 0 })) ||
                 undefined,
             topRight: (ySplit &&
-                new InternalViewport(this.getters, sheetId, { left: xSplit, right: nCols - 1, top: 0, bottom: ySplit - 1 }, { width: this.sheetViewWidth - colOffset, height: rowOffset }, { canScrollHorizontally, canScrollVertically: false }, { x: canScrollHorizontally ? previousOffset.x : 0, y: 0 })) ||
+                new InternalViewport(this.getters, sheetId, { left: xSplit, right: nCols - 1, top: 0, bottom: ySplit - 1 }, { width: unfrozenWidth, height: rowOffset }, { canScrollHorizontally, canScrollVertically: false }, { x: canScrollHorizontally ? previousOffset.x : 0, y: 0 })) ||
                 undefined,
             bottomLeft: (xSplit &&
-                new InternalViewport(this.getters, sheetId, { left: 0, right: xSplit - 1, top: ySplit, bottom: nRows - 1 }, { width: colOffset, height: this.sheetViewHeight - rowOffset }, { canScrollHorizontally: false, canScrollVertically }, { x: 0, y: canScrollVertically ? previousOffset.y : 0 })) ||
+                new InternalViewport(this.getters, sheetId, { left: 0, right: xSplit - 1, top: ySplit, bottom: nRows - 1 }, { width: colOffset, height: unfrozenHeight }, { canScrollHorizontally: false, canScrollVertically }, { x: 0, y: canScrollVertically ? previousOffset.y : 0 })) ||
                 undefined,
             bottomRight: new InternalViewport(this.getters, sheetId, { left: xSplit, right: nCols - 1, top: ySplit, bottom: nRows - 1 }, {
-                width: this.sheetViewWidth - colOffset,
-                height: this.sheetViewHeight - rowOffset,
+                width: unfrozenWidth,
+                height: unfrozenHeight,
             }, { canScrollHorizontally, canScrollVertically }, {
                 x: canScrollHorizontally ? previousOffset.x : 0,
                 y: canScrollVertically ? previousOffset.y : 0,
@@ -61935,12 +61970,26 @@ class SheetViewPlugin extends UIPlugin {
         const height = this.sheetViewHeight + this.gridOffsetY;
         return { xRatio: offsetCorrectionX / width, yRatio: offsetCorrectionY / height };
     }
-    recomposeRect(viewportRects) {
-        const x = Math.min(...viewportRects.map((rect) => rect.x));
-        const y = Math.min(...viewportRects.map((rect) => rect.y));
-        const width = Math.max(...viewportRects.map((rect) => rect.x + rect.width)) - x;
-        const height = Math.max(...viewportRects.map((rect) => rect.y + rect.height)) - y;
-        return { x, y, width, height };
+    mapViewportsToRect(sheetId, rectCallBack) {
+        let x = Infinity;
+        let y = Infinity;
+        let width = 0;
+        let height = 0;
+        let hasViewports = false;
+        for (const viewport of this.getSubViewports(sheetId)) {
+            const rect = rectCallBack(viewport);
+            if (rect) {
+                hasViewports = true;
+                x = Math.min(x, rect.x);
+                y = Math.min(y, rect.y);
+                width = Math.max(width, rect.x + rect.width);
+                height = Math.max(height, rect.y + rect.height);
+            }
+        }
+        if (!hasViewports) {
+            return { x: 0, y: 0, width: 0, height: 0 };
+        }
+        return { x, y, width: width - x, height: height - y };
     }
 }
 
@@ -62911,7 +62960,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),
         });
     }
@@ -68873,6 +68922,6 @@ exports.tokenColors = tokenColors;
 exports.tokenize = tokenize;
 
 
-__info__.version = "17.4.20";
-__info__.date = "2025-01-31T08:00:07.103Z";
-__info__.hash = "54a344a";
+__info__.version = "17.4.22";
+__info__.date = "2025-02-10T09:14:48.889Z";
+__info__.hash = "667f2b1";