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

package/dist/o-spreadsheet.iife.js

@@ -3,9 +3,9 @@
 /**
  * This file is generated by o-spreadsheet build tools. Do not edit it.
  * @see https://github.com/odoo/o-spreadsheet
- * @version 17.2.0-alpha.4
- * @date 2024-02-09T13:10:23.460Z
- * @hash 1b33cf4
+ * @version 17.2.0-alpha.5
+ * @date 2024-02-16T14:27:52.323Z
+ * @hash 0c69dd8
  */
 
 (function (exports, owl) {
@@ -454,8 +454,8 @@
    * If the given item does not exist in the dictionary, it creates one with a new id.
    */
   function getItemId(item, itemsDic) {
-      for (let [key, value] of Object.entries(itemsDic)) {
-          if (deepEquals(value, item)) {
+      for (const key in itemsDic) {
+          if (deepEquals(itemsDic[key], item)) {
               return parseInt(key, 10);
           }
       }
@@ -559,11 +559,13 @@
       if (typeof o1 !== "object")
           return o1 === o2;
       // Objects can have different keys if the values are undefined
-      const keys = new Set();
-      Object.keys(o1).forEach((key) => keys.add(key));
-      Object.keys(o2).forEach((key) => keys.add(key));
-      for (let key of keys) {
-          if (typeof o1[key] !== typeof o1[key])
+      for (const key in o2) {
+          if (!(key in o1) && o2[key] !== undefined) {
+              return false;
+          }
+      }
+      for (const key in o1) {
+          if (typeof o1[key] !== typeof o2[key])
               return false;
           if (typeof o1[key] === "object") {
               if (!deepEquals(o1[key], o2[key]))
@@ -633,18 +635,18 @@
       return true;
   }
   class JetSet extends Set {
-      add(...iterable) {
+      addMany(iterable) {
           for (const element of iterable) {
               super.add(element);
           }
           return this;
       }
-      delete(...iterable) {
-          let deleted = false;
+      deleteMany(iterable) {
+          let wasDeleted = false;
           for (const element of iterable) {
-              deleted ||= super.delete(element);
+              wasDeleted ||= super.delete(element);
           }
-          return deleted;
+          return wasDeleted;
       }
   }
   /**
@@ -1946,7 +1948,7 @@
        * Static helper which returns a successful DispatchResult
        */
       static get Success() {
-          return new DispatchResult();
+          return SUCCESS;
       }
       get isSuccessful() {
           return this.reasons.length === 0;
@@ -1959,6 +1961,7 @@
           return this.reasons.includes(reason);
       }
   }
+  const SUCCESS = new DispatchResult();
   exports.CommandResult = void 0;
   (function (CommandResult) {
       CommandResult["Success"] = "Success";
@@ -2265,7 +2268,7 @@
           .normalize("NFD")
           .replace(/[\u0300-\u036f]/g, "");
   });
-  const expectBooleanValueError = (value) => _t("The function [[FUNCTION_NAME]] expects a boolean value, but '%s' is a text, and cannot be coerced to a number.", value);
+  const expectBooleanValueError = (value) => _t("The function [[FUNCTION_NAME]] expects a boolean value, but '%s' is a text, and cannot be coerced to a boolean.", value);
   function toBoolean(data) {
       const value = toValue(data);
       switch (typeof value) {
@@ -7529,6 +7532,9 @@
        * Also useful for mocking a store.
        */
       inject(Store, instance) {
+          if (this.dependencies.has(Store) && this.dependencies.get(Store) !== instance) {
+              throw new Error(`Store ${Store.name} already has an instance`);
+          }
           this.dependencies.set(Store, instance);
       }
       /**
@@ -11357,8 +11363,10 @@ stores.inject(MyMetaStore, storeInstance);
       };
       static components = {};
       onDoubleClick() {
-          this.env.model.dispatch("SELECT_FIGURE", { id: this.props.figure.id });
-          this.env.openSidePanel("ChartPanel");
+          const result = this.env.model.dispatch("SELECT_FIGURE", { id: this.props.figure.id });
+          if (result.isSuccessful) {
+              this.env.openSidePanel("ChartPanel");
+          }
       }
       get chartType() {
           return this.env.model.getters.getChartType(this.props.figure.id);
@@ -13915,6 +13923,12 @@ stores.inject(MyMetaStore, storeInstance);
       let { col, row } = env.model.getters.getActivePosition();
       env.getStore(CellPopoverStore).open({ col, row }, "LinkEditor");
   };
+  const INSERT_LINK_NAME = (env) => {
+      const sheetId = env.model.getters.getActiveSheetId();
+      const { col, row } = env.model.getters.getActivePosition();
+      const cell = env.model.getters.getEvaluatedCell({ sheetId, col, row });
+      return cell && cell.link ? _t("Edit link") : _t("Insert link");
+  };
   //------------------------------------------------------------------------------
   // Filters action
   //------------------------------------------------------------------------------
@@ -21721,6 +21735,68 @@ stores.inject(MyMetaStore, storeInstance);
       isExported: true,
   };
   // -----------------------------------------------------------------------------
+  // INDEX
+  // -----------------------------------------------------------------------------
+  const INDIRECT = {
+      description: _t("Returns the content of a cell, specified by a string."),
+      args: [
+          arg("reference (string)", _t("The range of cells from which the values are returned.")),
+          arg("use_a1_notation (boolean, default=TRUE)", _t("A boolean indicating whether to use A1 style notation (TRUE) or R1C1 style notation (FALSE).")),
+      ],
+      returns: ["ANY"],
+      compute: function (reference, useA1Notation = { value: true }) {
+          let _reference = reference?.value?.toString();
+          if (!_reference) {
+              throw new InvalidReferenceError(_t("Reference should be defined."));
+          }
+          const _useA1Notation = toBoolean(useA1Notation);
+          if (!_useA1Notation) {
+              throw new EvaluationError(_t("R1C1 notation is not supported."));
+          }
+          const sheetId = this.__originSheetId;
+          let originPosition;
+          const __originCellXC = this.__originCellXC?.();
+          if (__originCellXC) {
+              const cellZone = toZone(__originCellXC);
+              originPosition = {
+                  sheetId,
+                  col: cellZone.left,
+                  row: cellZone.top,
+              };
+              // The following line is used to reset the dependencies of the cell, to avoid
+              // keeping dependencies from previous evaluation of the INDIRECT formula (i.e.
+              // in case the reference has been changed).
+              this.updateDependencies?.(originPosition);
+          }
+          const range = this.getters.getRangeFromSheetXC(sheetId, _reference);
+          if (range === undefined || range.invalidXc || range.invalidSheetName) {
+              throw new InvalidReferenceError();
+          }
+          if (originPosition) {
+              this.addDependencies?.(originPosition, [range]);
+          }
+          const values = [];
+          for (let col = range.zone.left; col <= range.zone.right; col++) {
+              const colValues = [];
+              for (let row = range.zone.top; row <= range.zone.bottom; row++) {
+                  const position = { sheetId: range.sheetId, col, row };
+                  // The below 'value' is typed as CellValue because the evaluateFormula will have to
+                  // evaluate a string containing the A1 reference of a single cell, so this will
+                  // return a CellValue.
+                  const value = this.getters.evaluateFormula(range.sheetId, toXC(col, row));
+                  const evaluatedCell = this.getters.getEvaluatedCell(position);
+                  colValues.push({
+                      ...evaluatedCell,
+                      value,
+                  });
+              }
+              values.push(colValues);
+          }
+          return values.length === 1 && values[0].length === 1 ? values[0][0] : values;
+      },
+      isExported: true,
+  };
+  // -----------------------------------------------------------------------------
   // LOOKUP
   // -----------------------------------------------------------------------------
   const LOOKUP = {
@@ -21926,6 +22002,7 @@ stores.inject(MyMetaStore, storeInstance);
     COLUMNS: COLUMNS,
     HLOOKUP: HLOOKUP,
     INDEX: INDEX,
+    INDIRECT: INDIRECT,
     LOOKUP: LOOKUP,
     MATCH: MATCH,
     ROW: ROW,
@@ -22901,7 +22978,7 @@ stores.inject(MyMetaStore, storeInstance);
   };
   const insertFunction = {
       name: _t("Function"),
-      icon: "o-spreadsheet-Icon.SHOW_HIDE_FORMULA",
+      icon: "o-spreadsheet-Icon.FORMULA",
   };
   const insertFunctionSum = {
       name: _t("SUM"),
@@ -23123,7 +23200,7 @@ stores.inject(MyMetaStore, storeInstance);
   })
       .add("insert_link", {
       ...insertLink,
-      name: _t("Insert link"),
+      name: INSERT_LINK_NAME,
       sequence: 150,
       separator: true,
   });
@@ -23788,9 +23865,7 @@ stores.inject(MyMetaStore, storeInstance);
       isReadonlyAllowed: true,
   };
   const viewGridlines = {
-      name: (env) => env.model.getters.getGridLinesVisibility(env.model.getters.getActiveSheetId())
-          ? _t("Hide gridlines")
-          : _t("Show gridlines"),
+      name: _t("Gridlines"),
       execute: (env) => {
           const sheetId = env.model.getters.getActiveSheetId();
           env.model.dispatch("SET_GRID_LINES_VISIBILITY", {
@@ -23798,13 +23873,16 @@ stores.inject(MyMetaStore, storeInstance);
               areGridLinesVisible: !env.model.getters.getGridLinesVisibility(sheetId),
           });
       },
-      icon: "o-spreadsheet-Icon.SHOW_HIDE_GRID",
+      isActive: (env) => {
+          const sheetId = env.model.getters.getActiveSheetId();
+          return env.model.getters.getGridLinesVisibility(sheetId);
+      },
   };
   const viewFormulas = {
-      name: (env) => env.model.getters.shouldShowFormulas() ? "Hide formulas" : _t("Show formulas"),
+      name: _t("Formulas"),
+      isActive: (env) => env.model.getters.shouldShowFormulas(),
       execute: (env) => env.model.dispatch("SET_FORMULA_VISIBILITY", { show: !env.model.getters.shouldShowFormulas() }),
       isReadonlyAllowed: true,
-      icon: "o-spreadsheet-Icon.SHOW_HIDE_FORMULA",
   };
   const createRemoveFilter = {
       name: (env) => selectionContainsFilter(env) ? _t("Remove selected filters") : _t("Create filter"),
@@ -24361,13 +24439,18 @@ stores.inject(MyMetaStore, storeInstance);
       isVisible: (env) => canUngroupHeaders(env, "ROW"),
       sequence: 20,
   })
-      .addChild("view_gridlines", ["view"], {
+      .addChild("show", ["view"], {
+      name: _t("Show"),
+      sequence: 1,
+      icon: "o-spreadsheet-Icon.SHOW",
+  })
+      .addChild("view_gridlines", ["view", "show"], {
       ...viewGridlines,
-      sequence: 15,
+      sequence: 5,
   })
-      .addChild("view_formulas", ["view"], {
+      .addChild("view_formulas", ["view", "show"], {
       ...viewFormulas,
-      sequence: 20,
+      sequence: 10,
   })
       // ---------------------------------------------------------------------
       // INSERT MENU ITEMS
@@ -25350,6 +25433,17 @@ stores.inject(MyMetaStore, storeInstance);
           onValueSelected: Function,
           onValueHovered: Function,
       };
+      autoCompleteListRef = owl.useRef("autoCompleteList");
+      setup() {
+          owl.useEffect(() => {
+              const selectedIndex = this.props.selectedIndex;
+              if (selectedIndex === undefined) {
+                  return;
+              }
+              const selectedElement = this.autoCompleteListRef.el?.children[selectedIndex];
+              selectedElement?.scrollIntoView?.({ block: "nearest" });
+          }, () => [this.props.selectedIndex, this.autoCompleteListRef.el]);
+      }
   }
 
   class ContentEditableHelper {
@@ -25794,6 +25888,7 @@ stores.inject(MyMetaStore, storeInstance);
       position: absolute;
       margin: 1px 4px;
       pointer-events: none;
+      overflow: auto;
 
       .o-semi-bold {
         /** FIXME: to remove in favor of Bootstrap
@@ -25853,7 +25948,10 @@ stores.inject(MyMetaStore, storeInstance);
           if (this.props.delimitation && this.props.rect) {
               const { x: cellX, y: cellY, height: cellHeight } = this.props.rect;
               const remainingHeight = this.props.delimitation.height - (cellY + cellHeight);
+              assistantStyle["max-height"] = `${remainingHeight}px`;
               if (cellY > remainingHeight) {
+                  const availableSpaceAbove = cellY;
+                  assistantStyle["max-height"] = `${availableSpaceAbove}px`;
                   // render top
                   // We compensate 2 px of margin on the assistant style + 1px for design reasons
                   assistantStyle.top = `-3px`;
@@ -29458,6 +29556,34 @@ stores.inject(MyMetaStore, storeInstance);
       }
   }
 
+  class SidePanelStore extends SpreadsheetStore {
+      isOpen = false;
+      panelProps = {};
+      componentTag = "";
+      open(componentTag, panelProps = {}) {
+          if (this.isOpen && componentTag !== this.componentTag) {
+              this.panelProps?.onCloseSidePanel?.();
+          }
+          this.componentTag = componentTag;
+          this.panelProps = panelProps;
+          this.isOpen = true;
+      }
+      toggle(componentTag, panelProps) {
+          if (this.isOpen && componentTag === this.componentTag) {
+              this.close();
+          }
+          else {
+              this.open(componentTag, panelProps);
+          }
+      }
+      close() {
+          this.panelProps.onCloseSidePanel?.();
+          this.isOpen = false;
+          this.panelProps = {};
+          this.componentTag = "";
+      }
+  }
+
   const registries$1 = {
       ROW: rowMenuRegistry,
       COL: colMenuRegistry,
@@ -29471,7 +29597,6 @@ stores.inject(MyMetaStore, storeInstance);
   class Grid extends owl.Component {
       static template = "o-spreadsheet-Grid";
       static props = {
-          sidePanelIsOpen: Boolean,
           exposeFocus: Function,
       };
       static components = {
@@ -29500,6 +29625,7 @@ stores.inject(MyMetaStore, storeInstance);
       onMouseWheel;
       canvasPosition;
       hoveredCell;
+      sidePanel;
       setup() {
           this.highlightStore = useStore(HighlightStore);
           this.menuState = owl.useState({
@@ -29513,6 +29639,7 @@ stores.inject(MyMetaStore, storeInstance);
           this.composerStore = useStore(ComposerStore);
           this.composerFocusStore = useStore(ComposerFocusStore);
           this.DOMFocusableElementStore = useStore(DOMFocusableElementStore);
+          this.sidePanel = useStore(SidePanelStore);
           owl.useChildSubEnv({ getPopoverContainerRect: () => this.getGridRect() });
           owl.useExternalListener(document.body, "cut", this.copy.bind(this, true));
           owl.useExternalListener(document.body, "copy", this.copy.bind(this, false));
@@ -29525,6 +29652,11 @@ stores.inject(MyMetaStore, storeInstance);
               this.hoveredCell.clear();
           });
           this.cellPopovers = useStore(CellPopoverStore);
+          owl.useEffect(() => {
+              if (!this.sidePanel.isOpen) {
+                  this.DOMFocusableElementStore.focus();
+              }
+          }, () => [this.sidePanel.isOpen]);
       }
       onCellHovered({ col, row }) {
           this.hoveredCell.hover({ col, row });
@@ -30901,6 +31033,7 @@ stores.inject(MyMetaStore, storeInstance);
           value: { type: Boolean, optional: true },
           className: { type: String, optional: true },
           name: { type: String, optional: true },
+          title: { type: String, optional: true },
           onChange: Function,
       };
       static defaultProps = { value: false };
@@ -31912,6 +32045,35 @@ stores.inject(MyMetaStore, storeInstance);
       design: ScorecardChartDesignPanel,
   });
 
+  class MainChartPanelStore extends SpreadsheetStore {
+      sidePanel = this.get(SidePanelStore);
+      figureId;
+      panel = "configuration";
+      constructor(get, figureId) {
+          super(get);
+          this.figureId = figureId;
+      }
+      handle(cmd) {
+          switch (cmd.type) {
+              case "DELETE_FIGURE":
+                  if (this.sidePanel.componentTag === "ChartPanel" && this.figureId === cmd.id) {
+                      this.sidePanel.close();
+                  }
+                  break;
+              case "SELECT_FIGURE":
+                  if (cmd.id) {
+                      this.figureId = cmd.id;
+                  }
+                  else if (this.sidePanel.componentTag === "ChartPanel") {
+                      this.sidePanel.close();
+                  }
+          }
+      }
+      activatePanel(panel) {
+          this.panel = panel;
+      }
+  }
+
   css /* scss */ `
   .o-chart {
     .o-panel {
@@ -31940,37 +32102,19 @@ stores.inject(MyMetaStore, storeInstance);
       static template = "o-spreadsheet-ChartPanel";
       static components = { Section };
       static props = { onCloseSidePanel: Function };
-      state;
+      store;
       get figureId() {
-          return this.state.figureId;
+          return this.store.figureId;
       }
       setup() {
-          const selectedFigureId = this.env.model.getters.getSelectedFigureId();
-          if (!selectedFigureId) {
-              this.props.onCloseSidePanel();
-              return;
-          }
-          this.state = owl.useState({
-              panel: "configuration",
-              figureId: selectedFigureId,
-          });
-          owl.onWillUpdateProps(() => {
-              const selectedFigureId = this.env.model.getters.getSelectedFigureId();
-              if (selectedFigureId && selectedFigureId !== this.state.figureId) {
-                  this.state.figureId = selectedFigureId;
-              }
-              if (!this.env.model.getters.isChartDefined(this.figureId)) {
-                  this.props.onCloseSidePanel();
-                  return;
-              }
-          });
+          this.store = useLocalStore(MainChartPanelStore, this.env.model.getters.getSelectedFigureId());
       }
       updateChart(figureId, updateDefinition) {
           if (figureId !== this.figureId) {
               return;
           }
           const definition = {
-              ...this.getChartDefinition(),
+              ...this.getChartDefinition(this.figureId),
               ...updateDefinition,
           };
           return this.env.model.dispatch("UPDATE_CHART", {
@@ -31984,7 +32128,7 @@ stores.inject(MyMetaStore, storeInstance);
               return;
           }
           const definition = {
-              ...this.getChartDefinition(),
+              ...this.getChartDefinition(this.figureId),
               ...updateDefinition,
           };
           return this.env.model.canDispatch("UPDATE_CHART", {
@@ -31994,6 +32138,9 @@ stores.inject(MyMetaStore, storeInstance);
           });
       }
       onTypeChange(type) {
+          if (!this.figureId) {
+              return;
+          }
           const context = this.env.model.getters.getContextCreationChart(this.figureId);
           if (!context) {
               throw new Error("Chart not defined.");
@@ -32006,6 +32153,9 @@ stores.inject(MyMetaStore, storeInstance);
           });
       }
       get chartPanel() {
+          if (!this.figureId) {
+              throw new Error("Chart not defined.");
+          }
           const type = this.env.model.getters.getChartType(this.figureId);
           if (!type) {
               throw new Error("Chart not defined.");
@@ -32016,15 +32166,12 @@ stores.inject(MyMetaStore, storeInstance);
           }
           return chartPanel;
       }
-      getChartDefinition(figureId = this.figureId) {
+      getChartDefinition(figureId) {
           return this.env.model.getters.getChartDefinition(figureId);
       }
       get chartTypes() {
           return getChartTypes();
       }
-      activatePanel(panel) {
-          this.state.panel = panel;
-      }
   }
 
   const BORDER_COLOR = "#8B008B";
@@ -32118,7 +32265,7 @@ stores.inject(MyMetaStore, storeInstance);
       }
       finalize() {
           if (this.isSearchDirty) {
-              this.refreshSearch();
+              this.refreshSearch(false);
               this.isSearchDirty = false;
           }
       }
@@ -32150,10 +32297,10 @@ stores.inject(MyMetaStore, storeInstance);
       /**
        * refresh the matches according to the current search options
        */
-      refreshSearch() {
+      refreshSearch(jumpToMatchSheet = true) {
           this.selectedMatchIndex = null;
           this.findMatches();
-          this.selectNextCell(Direction.current);
+          this.selectNextCell(Direction.current, jumpToMatchSheet);
       }
       getSheetsInSearchOrder() {
           switch (this.searchOptions.searchScope) {
@@ -32223,7 +32370,7 @@ stores.inject(MyMetaStore, storeInstance);
        * It is also used to keep coherence between the selected searchMatch
        * and selectedMatchIndex.
        */
-      selectNextCell(indexChange) {
+      selectNextCell(indexChange, jumpToMatchSheet = true) {
           const matches = this.searchMatches;
           if (!matches.length) {
               this.selectedMatchIndex = null;
@@ -32249,7 +32396,7 @@ stores.inject(MyMetaStore, storeInstance);
           this.selectedMatchIndex = nextIndex;
           const selectedMatch = matches[nextIndex];
           // Switch to the sheet where the match is located
-          if (this.getters.getActiveSheetId() !== selectedMatch.sheetId) {
+          if (jumpToMatchSheet && this.getters.getActiveSheetId() !== selectedMatch.sheetId) {
               this.model.dispatch("ACTIVATE_SHEET", {
                   sheetIdFrom: this.getters.getActiveSheetId(),
                   sheetIdTo: selectedMatch.sheetId,
@@ -36966,22 +37113,7 @@ stores.inject(MyMetaStore, storeInstance);
           }
       }
       export(data) {
-          // Borders
-          let borderId = 0;
           const borders = {};
-          /**
-           * Get the id of the given border. If the border does not exist, it creates
-           * one.
-           */
-          function getBorderId(border) {
-              for (let [key, value] of Object.entries(borders)) {
-                  if (deepEquals(value, border)) {
-                      return parseInt(key, 10);
-                  }
-              }
-              borders[++borderId] = border;
-              return borderId;
-          }
           for (let sheet of data.sheets) {
               for (let col = 0; col < sheet.colNumber; col++) {
                   for (let row = 0; row < sheet.rowNumber; row++) {
@@ -36989,7 +37121,7 @@ stores.inject(MyMetaStore, storeInstance);
                       if (border) {
                           const xc = toXC(col, row);
                           const cell = sheet.cells[xc];
-                          const borderId = getBorderId(border);
+                          const borderId = getItemId(border, borders);
                           if (cell) {
                               cell.border = borderId;
                           }
@@ -37686,9 +37818,9 @@ stores.inject(MyMetaStore, storeInstance);
       allowDispatch(cmd) {
           switch (cmd.type) {
               case "UPDATE_CELL":
-                  return this.checkCellOutOfSheet(cmd);
+                  return this.checkValidations(cmd, this.checkCellOutOfSheet, this.checkUselessUpdateCell);
               case "CLEAR_CELL":
-                  return this.checkValidations(cmd, this.chainValidations(this.checkCellOutOfSheet, this.checkUselessClearCell));
+                  return this.checkValidations(cmd, this.checkCellOutOfSheet, this.checkUselessClearCell);
               default:
                   return "Success" /* CommandResult.Success */;
           }
@@ -37858,8 +37990,8 @@ stores.inject(MyMetaStore, storeInstance);
       }
       removeDefaultStyleValues(style) {
           const cleanedStyle = { ...style };
-          for (const [property, defaultValue] of Object.entries(DEFAULT_STYLE)) {
-              if (cleanedStyle[property] === defaultValue) {
+          for (const property in DEFAULT_STYLE) {
+              if (cleanedStyle[property] === DEFAULT_STYLE[property]) {
                   delete cleanedStyle[property];
               }
           }
@@ -38015,10 +38147,7 @@ stores.inject(MyMetaStore, storeInstance);
           else {
               style = before ? before.style : undefined;
           }
-          const locale = this.getters.getLocale();
-          let format = ("format" in after ? after.format : before && before.format) ||
-              detectDateFormat(afterContent, locale) ||
-              detectNumberFormat(afterContent);
+          const format = "format" in after ? after.format : before && before.format;
           /* Read the following IF as:
            * we need to remove the cell if it is completely empty, but we can know if it completely empty if:
            * - the command says the new content is empty and has no border/format/style
@@ -38059,6 +38188,7 @@ stores.inject(MyMetaStore, storeInstance);
       }
       createLiteralCell(id, content, format, style) {
           const locale = this.getters.getLocale();
+          format = format || detectDateFormat(content, locale) || detectNumberFormat(content);
           if (format !== PLAIN_TEXT_FORMAT && !isEvaluationError(content)) {
               content = toString(parseLiteral(content, locale));
           }
@@ -38131,6 +38261,18 @@ stores.inject(MyMetaStore, storeInstance);
           }
           return "Success" /* CommandResult.Success */;
       }
+      checkUselessUpdateCell(cmd) {
+          const cell = this.getters.getCell(cmd);
+          const hasContent = "content" in cmd || "formula" in cmd;
+          const hasStyle = "style" in cmd;
+          const hasFormat = "format" in cmd;
+          if ((!hasContent || cell?.content === cmd.content) &&
+              (!hasStyle || deepEquals(cell?.style, cmd.style)) &&
+              (!hasFormat || cell?.format === cmd.format)) {
+              return "NoChanges" /* CommandResult.NoChanges */;
+          }
+          return "Success" /* CommandResult.Success */;
+      }
   }
   class FormulaCellWithDependencies {
       id;
@@ -42946,7 +43088,15 @@ stores.inject(MyMetaStore, storeInstance);
           if (!this.rTrees[sheetId]) {
               return;
           }
-          this.rTrees[sheetId].remove(item, deepEquals);
+          this.rTrees[sheetId].remove(item, this.rtreeItemComparer);
+      }
+      rtreeItemComparer(left, right) {
+          return (left.data == right.data &&
+              left.boundingBox.sheetId === right.boundingBox.sheetId &&
+              left.boundingBox?.zone.left === right.boundingBox.zone.left &&
+              left.boundingBox?.zone.top === right.boundingBox.zone.top &&
+              left.boundingBox?.zone.right === right.boundingBox.zone.right &&
+              left.boundingBox?.zone.bottom === right.boundingBox.zone.bottom);
       }
   }
   /**
@@ -43024,7 +43174,7 @@ stores.inject(MyMetaStore, storeInstance);
           const queue = Array.from(ranges).reverse();
           while (queue.length > 0) {
               const range = queue.pop();
-              visited.add(...this.encoder.encodeBoundingBox(range));
+              visited.addMany(this.encoder.encodeBoundingBox(range));
               const impactedPositionIds = this.rTree.search(range).map((dep) => dep.data);
               for (const positionId of impactedPositionIds) {
                   if (!visited.has(positionId)) {
@@ -43032,7 +43182,7 @@ stores.inject(MyMetaStore, storeInstance);
                   }
               }
           }
-          visited.delete(...ranges.flatMap((r) => this.encoder.encodeBoundingBox(r)));
+          visited.deleteMany(ranges.flatMap((r) => this.encoder.encodeBoundingBox(r)));
           return visited;
       }
   }
@@ -43118,6 +43268,7 @@ stores.inject(MyMetaStore, storeInstance);
   const MAX_ITERATION = 30;
   const ERROR_CYCLE_CELL = createEvaluatedCell(new CircularDependencyError());
   const EMPTY_CELL = createEvaluatedCell({ value: null });
+  const EVAL_CONTEXT_INDEX = 2;
   class Evaluator {
       context;
       getters;
@@ -43163,17 +43314,24 @@ stores.inject(MyMetaStore, storeInstance);
           const dependencies = this.getDirectDependencies(positionId);
           this.formulaDependencies().addDependencies(positionId, dependencies);
       }
+      addDependencies(position, dependencies) {
+          const positionId = this.encoder.encode(position);
+          this.formulaDependencies().addDependencies(positionId, dependencies);
+      }
       updateCompilationParameters() {
           // rebuild the compilation parameters (with a clean cache)
           this.compilationParams = buildCompilationParameters(this.context, this.getters, this.computeAndSave.bind(this));
+          this.compilationParams[EVAL_CONTEXT_INDEX].updateDependencies =
+              this.updateDependencies.bind(this);
+          this.compilationParams[EVAL_CONTEXT_INDEX].addDependencies = this.addDependencies.bind(this);
       }
       evaluateCells(positions) {
           const cells = positions.map((p) => this.encoder.encode(p));
           const cellsToCompute = new JetSet(cells);
           const arrayFormulasPositionIds = this.getArrayFormulasImpactedByChangesOf(cells);
-          cellsToCompute.add(...this.getCellsDependingOn(cells));
-          cellsToCompute.add(...arrayFormulasPositionIds);
-          cellsToCompute.add(...this.getCellsDependingOn(arrayFormulasPositionIds));
+          cellsToCompute.addMany(this.getCellsDependingOn(cells));
+          cellsToCompute.addMany(arrayFormulasPositionIds);
+          cellsToCompute.addMany(this.getCellsDependingOn(arrayFormulasPositionIds));
           this.evaluate(cellsToCompute);
       }
       getArrayFormulasImpactedByChangesOf(positionIds) {
@@ -43187,7 +43345,7 @@ stores.inject(MyMetaStore, storeInstance);
               }
               if (!content) {
                   // The previous content could have blocked some array formulas
-                  impactedPositionIds.add(...this.getArrayFormulasBlockedByOrSpreadingOn(positionId));
+                  impactedPositionIds.addMany(this.getArrayFormulasBlockedByOrSpreadingOn(positionId));
               }
           }
           return impactedPositionIds;
@@ -43239,7 +43397,7 @@ stores.inject(MyMetaStore, storeInstance);
           }
           const arrayFormulas = this.spreadingRelations.getFormulaPositionsSpreadingOn(positionId);
           const cells = new JetSet(arrayFormulas);
-          cells.add(...this.getCellsDependingOn(arrayFormulas));
+          cells.addMany(this.getCellsDependingOn(arrayFormulas));
           return cells;
       }
       nextPositionsToUpdate = new JetSet();
@@ -43367,7 +43525,7 @@ stores.inject(MyMetaStore, storeInstance);
               this.setEvaluatedCell(positionId, evaluatedCell);
               // check if formula dependencies present in the spread zone
               // if so, they need to be recomputed
-              this.nextPositionsToUpdate.add(...this.getCellsDependingOn([positionId]));
+              this.nextPositionsToUpdate.addMany(this.getCellsDependingOn([positionId]));
           };
       }
       invalidateSpreading(positionId) {
@@ -43382,8 +43540,8 @@ stores.inject(MyMetaStore, storeInstance);
                   continue;
               }
               this.evaluatedCells.delete(child);
-              this.nextPositionsToUpdate.add(...this.getCellsDependingOn([child]));
-              this.nextPositionsToUpdate.add(...this.getArrayFormulasBlockedByOrSpreadingOn(child));
+              this.nextPositionsToUpdate.addMany(this.getCellsDependingOn([child]));
+              this.nextPositionsToUpdate.addMany(this.getArrayFormulasBlockedByOrSpreadingOn(child));
           }
           this.spreadingRelations.removeNode(positionId);
       }
@@ -43512,15 +43670,15 @@ stores.inject(MyMetaStore, storeInstance);
       }
   }
   function updateEvalContextAndExecute(compiledFormula, compilationParams, sheetId, cellId) {
-      compilationParams[2].__originCellXC = lazy(() => {
+      compilationParams[EVAL_CONTEXT_INDEX].__originCellXC = lazy(() => {
           if (!cellId) {
               return undefined;
           }
           // compute the value lazily for performance reasons
-          const position = compilationParams[2].getters.getCellPosition(cellId);
+          const position = compilationParams[EVAL_CONTEXT_INDEX].getters.getCellPosition(cellId);
           return toXC(position.col, position.row);
       });
-      compilationParams[2].__originSheetId = sheetId;
+      compilationParams[EVAL_CONTEXT_INDEX].__originSheetId = sheetId;
       return compiledFormula.execute(compiledFormula.dependencies, ...compilationParams);
   }
 
@@ -43850,8 +44008,13 @@ stores.inject(MyMetaStore, storeInstance);
    */
   class CustomColorsPlugin extends UIPlugin {
       customColors = {};
+      configCustomColors;
       shouldUpdateColors = false;
       static getters = ["getCustomColors"];
+      constructor(config) {
+          super(config);
+          this.configCustomColors = config.customColors;
+      }
       handle(cmd) {
           switch (cmd.type) {
               case "UPDATE_CELL":
@@ -43873,7 +44036,7 @@ stores.inject(MyMetaStore, storeInstance);
           }
       }
       getCustomColors() {
-          let usedColors = [];
+          let usedColors = this.configCustomColors;
           for (const sheetId of this.getters.getSheetIds()) {
               usedColors = usedColors.concat(this.getColorsFromCells(sheetId), this.getFormattingColors(sheetId), this.getChartColors(sheetId));
           }
@@ -46846,7 +47009,7 @@ stores.inject(MyMetaStore, storeInstance);
           "getCellText",
           "getCellMultiLineText",
           "getContiguousZone",
-          "isCellEmpty",
+          "isEvaluatedCellEmpty",
       ];
       ctx = document.createElement("canvas").getContext("2d");
       // ---------------------------------------------------------------------------
@@ -46974,14 +47137,26 @@ stores.inject(MyMetaStore, storeInstance);
           return zone;
       }
       /**
-       * Check if a cell is empty. If the cell is part of a merge,
-       * check if the merge containing the cell is empty.
+       * Checks if a cell evaluated value is empty. If the cell is part of a merge,
+       * the check applies to the main cell of the merge.
        */
-      isCellEmpty(position) {
+      //TODORAR this does not fit the new scheme where a cell with an empty string is not empty anymore
+      // this means the navigation was acually altered with the recent change
+      // remove the getter and put it back in selection processor as nobody uses it
+      isEvaluatedCellEmpty(position) {
           const mainPosition = this.getters.getMainCellPosition(position);
           const cell = this.getters.getEvaluatedCell(mainPosition);
           return cell.type === CellValueType.empty;
       }
+      /**
+       * Checks if a cell is empty (i.e. does not have a content or a formula does not spread over it).
+       * If the cell is part of a merge, the check applies to the main cell of the merge.
+       */
+      isCellEmpty(position) {
+          const mainPosition = this.getters.getMainCellPosition(position);
+          return !(this.getters.getCorrespondingFormulaCell(mainPosition) ||
+              this.getters.getCell(mainPosition)?.content);
+      }
       getColMaxWidth(sheetId, index) {
           const cellsPositions = positions(this.getters.getColsZone(sheetId, index, index));
           const sizes = cellsPositions.map((position) => this.getCellWidth({ sheetId, ...position }));
@@ -51516,7 +51691,7 @@ stores.inject(MyMetaStore, storeInstance);
           if (ev.key === "Enter") {
               ev.preventDefault();
               ev.stopPropagation();
-              this.store.selectNextMatch();
+              ev.shiftKey ? this.store.selectPreviousMatch() : this.store.selectNextMatch();
           }
       }
       onKeydownReplace(ev) {
@@ -53212,22 +53387,20 @@ stores.inject(MyMetaStore, storeInstance);
 `;
   class SidePanel extends owl.Component {
       static template = "o-spreadsheet-SidePanel";
-      static props = {
-          component: String,
-          panelProps: { type: Object, optional: true },
-          onCloseSidePanel: Function,
-      };
-      state;
+      static props = {};
+      sidePanelStore;
       setup() {
-          this.state = owl.useState({
-              panel: sidePanelRegistry.get(this.props.component),
-          });
-          owl.onWillUpdateProps((nextProps) => (this.state.panel = sidePanelRegistry.get(nextProps.component)));
+          this.sidePanelStore = useStore(SidePanelStore);
+      }
+      get panel() {
+          return sidePanelRegistry.get(this.sidePanelStore.componentTag);
+      }
+      close() {
+          this.sidePanelStore.close();
       }
       getTitle() {
-          return typeof this.state.panel.title === "function"
-              ? this.state.panel.title(this.env)
-              : this.state.panel.title;
+          const panel = this.panel;
+          return typeof panel.title === "function" ? panel.title(this.env) : panel.title;
       }
   }
 
@@ -54161,10 +54334,13 @@ stores.inject(MyMetaStore, storeInstance);
       }
       setup() {
           const stores = useStoreProvider();
-          this.sidePanel = owl.useState({ isOpen: false, panelProps: {} });
+          stores.inject(ModelStore, this.model);
+          this.notificationStore = useStore(NotificationStore);
+          this.composerFocusStore = useStore(ComposerFocusStore);
+          this.sidePanel = useStore(SidePanelStore);
           this.keyDownMapping = {
-              "CTRL+H": () => this.toggleSidePanel("FindAndReplace", {}),
-              "CTRL+F": () => this.toggleSidePanel("FindAndReplace", {}),
+              "CTRL+H": () => this.sidePanel.toggle("FindAndReplace", {}),
+              "CTRL+F": () => this.sidePanel.toggle("FindAndReplace", {}),
           };
           const fileStore = this.model.config.external.fileStore;
           owl.useSubEnv({
@@ -54173,11 +54349,10 @@ stores.inject(MyMetaStore, storeInstance);
               loadCurrencies: this.model.config.external.loadCurrencies,
               loadLocales: this.model.config.external.loadLocales,
               isDashboard: () => this.model.getters.isDashboard(),
-              openSidePanel: this.openSidePanel.bind(this),
-              toggleSidePanel: this.toggleSidePanel.bind(this),
+              openSidePanel: this.sidePanel.open.bind(this.sidePanel),
+              toggleSidePanel: this.sidePanel.toggle.bind(this.sidePanel),
               clipboard: this.env.clipboard || instantiateClipboard(),
               startCellEdition: (content) => this.composerFocusStore.focusGridComposerCell(content),
-              //TODORAR: make a store out of this
           });
           owl.useEffect(() => {
               /**
@@ -54206,9 +54381,6 @@ stores.inject(MyMetaStore, storeInstance);
           owl.onPatched(() => {
               this.checkViewportSize();
           });
-          stores.inject(ModelStore, this.model);
-          this.notificationStore = useStore(NotificationStore);
-          this.composerFocusStore = useStore(ComposerFocusStore);
       }
       bindModelEvents() {
           this.model.on("update", this, () => this.render(true));
@@ -54241,28 +54413,6 @@ stores.inject(MyMetaStore, storeInstance);
               this.isViewportTooSmall = false;
           }
       }
-      openSidePanel(panel, panelProps) {
-          if (this.sidePanel.isOpen && panel !== this.sidePanel.component) {
-              this.sidePanel.panelProps?.onCloseSidePanel?.();
-          }
-          this.sidePanel.component = panel;
-          this.sidePanel.panelProps = panelProps;
-          this.sidePanel.isOpen = true;
-      }
-      closeSidePanel() {
-          this.sidePanel.isOpen = false;
-          this.focusGrid();
-          this.sidePanel.panelProps?.onCloseSidePanel?.();
-      }
-      toggleSidePanel(panel, panelProps) {
-          if (this.sidePanel.isOpen && panel === this.sidePanel.component) {
-              this.sidePanel.isOpen = false;
-              this.focusGrid();
-          }
-          else {
-              this.openSidePanel(panel, panelProps);
-          }
-      }
       focusGrid() {
           if (!this._focusGrid) {
               return;
@@ -55691,8 +55841,8 @@ stores.inject(MyMetaStore, storeInstance);
           let currentPosition = startPosition;
           // If both the current cell and the next cell are not empty, we want to go to the end of the cluster
           const nextCellPosition = this.getNextCellPosition(startPosition, dim, dir);
-          let mode = !this.getters.isCellEmpty({ ...currentPosition, sheetId }) &&
-              !this.getters.isCellEmpty({ ...nextCellPosition, sheetId })
+          let mode = !this.isCellSkippableInCluster({ ...currentPosition, sheetId }) &&
+              !this.isCellSkippableInCluster({ ...nextCellPosition, sheetId })
               ? "endOfCluster"
               : "nextCluster";
           while (true) {
@@ -55702,7 +55852,7 @@ stores.inject(MyMetaStore, storeInstance);
                   currentPosition.row === nextCellPosition.row) {
                   break;
               }
-              const isNextCellEmpty = this.getters.isCellEmpty({ ...nextCellPosition, sheetId });
+              const isNextCellEmpty = this.isCellSkippableInCluster({ ...nextCellPosition, sheetId });
               if (mode === "endOfCluster" && isNextCellEmpty) {
                   break;
               }
@@ -55733,6 +55883,11 @@ stores.inject(MyMetaStore, storeInstance);
       getPosition() {
           return { ...this.anchor.cell };
       }
+      isCellSkippableInCluster(position) {
+          const mainPosition = this.getters.getMainCellPosition(position);
+          const cell = this.getters.getEvaluatedCell(mainPosition);
+          return (cell.type === CellValueType.empty || (cell.type === CellValueType.text && cell.value === ""));
+      }
   }
 
   class StateObserver {
@@ -57599,6 +57754,7 @@ stores.inject(MyMetaStore, storeInstance);
               snapshotRequested: false,
               notifyUI: (payload) => this.trigger("notify-ui", payload),
               raiseBlockingErrorUI: (text) => this.trigger("raise-error-ui", { text }),
+              customColors: config.customColors || [],
           };
       }
       setupExternalConfig(external) {
@@ -57630,6 +57786,7 @@ stores.inject(MyMetaStore, storeInstance);
               uiActions: this.config,
               session: this.session,
               defaultCurrencyFormat: this.config.defaultCurrencyFormat,
+              customColors: this.config.customColors || [],
           };
       }
       // ---------------------------------------------------------------------------
@@ -57929,6 +58086,10 @@ stores.inject(MyMetaStore, storeInstance);
       deepEquals,
       overlap,
       union,
+      isInside,
+      deepCopy,
+      expandZoneOnInsertion,
+      reduceZoneOnDeletion,
   };
   const links = {
       isMarkdownLink,
@@ -57983,6 +58144,9 @@ stores.inject(MyMetaStore, storeInstance);
       RendererStore,
       SelectionInputStore,
       SpreadsheetStore,
+      useStore,
+      useLocalStore,
+      SidePanelStore,
   };
   function addFunction(functionName, functionDescription) {
       functionRegistry.add(functionName, functionDescription);
@@ -57995,7 +58159,9 @@ stores.inject(MyMetaStore, storeInstance);
       SECONDARY_COLOR,
   };
 
+  exports.AbstractCellClipboardHandler = AbstractCellClipboardHandler;
   exports.AbstractChart = AbstractChart;
+  exports.AbstractFigureClipboardHandler = AbstractFigureClipboardHandler;
   exports.CellErrorType = CellErrorType;
   exports.CorePlugin = CorePlugin;
   exports.DispatchResult = DispatchResult;
@@ -58035,9 +58201,9 @@ stores.inject(MyMetaStore, storeInstance);
   exports.tokenize = tokenize;
 
 
-  __info__.version = "17.2.0-alpha.4";
-  __info__.date = "2024-02-09T13:10:23.460Z";
-  __info__.hash = "1b33cf4";
+  __info__.version = "17.2.0-alpha.5";
+  __info__.date = "2024-02-16T14:27:52.323Z";
+  __info__.hash = "0c69dd8";
 
 
 })(this.o_spreadsheet = this.o_spreadsheet || {}, owl);