@odoo/o-spreadsheet 16.0.54 → 16.0.56

package/dist/o-spreadsheet.cjs.js

@@ -9974,6 +9974,9 @@ class ConditionalFormattingPanel extends owl.Component {
         }
     }
     setColorScaleColor(target, color) {
+        if (!isColorValid(color)) {
+            return;
+        }
         const point = this.state.rules.colorScale[target];
         if (point) {
             point.color = Number.parseInt(color.substr(1), 16);
@@ -10685,15 +10688,23 @@ class ChartFigure extends owl.Component {
 ChartFigure.template = "o-spreadsheet-ChartFigure";
 ChartFigure.components = { Menu };
 
+/**
+ * Start listening to pointer events and apply the given callbacks.
+ *
+ * @returns A function to remove the listeners.
+ */
 function startDnd(onMouseMove, onMouseUp, onMouseDown = () => { }) {
-    const _onMouseUp = (ev) => {
-        onMouseUp(ev);
+    const removeListeners = () => {
         window.removeEventListener("mousedown", onMouseDown);
         window.removeEventListener("mouseup", _onMouseUp);
         window.removeEventListener("dragstart", _onDragStart);
         window.removeEventListener("mousemove", onMouseMove);
         window.removeEventListener("wheel", onMouseMove);
     };
+    const _onMouseUp = (ev) => {
+        onMouseUp(ev);
+        removeListeners();
+    };
     function _onDragStart(ev) {
         ev.preventDefault();
     }
@@ -10702,6 +10713,7 @@ function startDnd(onMouseMove, onMouseUp, onMouseDown = () => { }) {
     window.addEventListener("dragstart", _onDragStart);
     window.addEventListener("mousemove", onMouseMove);
     window.addEventListener("wheel", onMouseMove);
+    return removeListeners;
 }
 /**
  * Function to be used during a mousedown event, this function allows to
@@ -18396,13 +18408,12 @@ function tokenizeSymbol(chars) {
 }
 const whiteSpaceRegexp = /\s/;
 function tokenizeSpace(chars) {
-    let length = 0;
+    let spaces = "";
     while (chars[0] && chars[0].match(whiteSpaceRegexp)) {
-        length++;
-        chars.shift();
+        spaces += chars.shift();
     }
-    if (length) {
-        return { type: "SPACE", value: " ".repeat(length) };
+    if (spaces) {
+        return { type: "SPACE", value: spaces };
     }
     return null;
 }
@@ -20429,11 +20440,11 @@ css /* scss */ `
   }
 
   /* Custom css to highlight topbar composer on focus */
-  .o-topbar-toolbar .o-composer-container:focus-within {
-    border: 1px solid ${SELECTION_BORDER_COLOR};
+  .o-topbar-toolbar .o-composer-container[active] {
+    border: 1px solid ${SELECTION_BORDER_COLOR} !important;
   }
 
-  .o-topbar-toolbar .o-composer-container {
+  .o-topbar-toolbar .o-composer-container[active] {
     z-index: ${ComponentsImportance.TopBarComposer};
   }
 `;
@@ -21382,6 +21393,7 @@ class FiguresContainer extends owl.Component {
             y: 0,
             width: 0,
             height: 0,
+            cancelDnd: undefined,
         });
     }
     setup() {
@@ -21395,11 +21407,24 @@ class FiguresContainer extends owl.Component {
             // new rendering
             this.render();
         });
+        owl.onWillUpdateProps(() => {
+            const sheetId = this.env.model.getters.getActiveSheetId();
+            if (this.dnd.figId && !this.env.model.getters.getFigure(sheetId, this.dnd.figId)) {
+                if (this.dnd.cancelDnd) {
+                    this.dnd.cancelDnd();
+                }
+                this.dnd.figId = undefined;
+                this.dnd.cancelDnd = undefined;
+            }
+        });
     }
     getVisibleFigures() {
         const visibleFigures = this.env.model.getters.getVisibleFigures();
         if (this.dnd.figId && !visibleFigures.some((figure) => figure.id === this.dnd.figId)) {
-            visibleFigures.push(this.env.model.getters.getFigure(this.env.model.getters.getActiveSheetId(), this.dnd.figId));
+            const draggedFigure = this.env.model.getters.getFigure(this.env.model.getters.getActiveSheetId(), this.dnd.figId);
+            if (draggedFigure) {
+                visibleFigures.push(draggedFigure);
+            }
         }
         return visibleFigures;
     }
@@ -21512,7 +21537,7 @@ class FiguresContainer extends owl.Component {
             this.dnd.figId = undefined;
             this.env.model.dispatch("UPDATE_FIGURE", { sheetId, id: figure.id, x, y });
         };
-        startDnd(onMouseMove, onMouseUp);
+        this.dnd.cancelDnd = startDnd(onMouseMove, onMouseUp);
     }
     startResize(figure, dirX, dirY, ev) {
         ev.stopPropagation();
@@ -21566,7 +21591,7 @@ class FiguresContainer extends owl.Component {
                 ...update,
             });
         };
-        startDnd(onMouseMove, onMouseUp);
+        this.dnd.cancelDnd = startDnd(onMouseMove, onMouseUp);
     }
     getDndFigure() {
         const figure = this.getVisibleFigures().find((fig) => fig.id === this.dnd.figId);
@@ -27935,7 +27960,6 @@ class BordersPlugin extends CorePlugin {
 }
 BordersPlugin.getters = ["getCellBorder"];
 
-const nbspRegexp = new RegExp(String.fromCharCode(160), "g");
 const LINK_STYLE = { textColor: LINK_COLOR };
 /**
  * Core Plugin
@@ -28269,13 +28293,10 @@ class CellPlugin extends CorePlugin {
         return id;
     }
     updateCell(sheetId, col, row, after) {
-        var _a;
         const before = this.getters.getCell(sheetId, col, row);
         const hasContent = "content" in after || "formula" in after;
         // Compute the new cell properties
-        const afterContent = hasContent
-            ? ((_a = after.content) === null || _a === void 0 ? void 0 : _a.replace(nbspRegexp, "")) || ""
-            : (before === null || before === void 0 ? void 0 : before.content) || "";
+        const afterContent = (hasContent ? after.content : before === null || before === void 0 ? void 0 : before.content) || "";
         let style;
         if (after.style !== undefined) {
             style = after.style || undefined;
@@ -37159,6 +37180,7 @@ class SheetViewPlugin extends UIPlugin {
                 break;
             case "DELETE_SHEET":
                 this.cleanViewports();
+                this.sheetsWithDirtyViewports.delete(cmd.sheetId);
                 break;
             case "ACTIVATE_SHEET":
                 this.setViewports();
@@ -37511,6 +37533,9 @@ class SheetViewPlugin extends UIPlugin {
         };
     }
     resetViewports(sheetId) {
+        if (!this.getters.tryGetSheet(sheetId)) {
+            return;
+        }
         const { xSplit, ySplit } = this.getters.getPaneDivisions(sheetId);
         const nCols = this.getters.getNumberCols(sheetId);
         const nRows = this.getters.getNumberRows(sheetId);
@@ -43459,6 +43484,6 @@ exports.setTranslationMethod = setTranslationMethod;
 exports.tokenize = tokenize;
 
 
-__info__.version = '16.0.54';
-__info__.date = '2024-11-08T12:11:01.946Z';
-__info__.hash = 'fdfa7dc';
+__info__.version = '16.0.56';
+__info__.date = '2024-12-19T07:45:47.250Z';
+__info__.hash = '9d0e335';

package/dist/o-spreadsheet.d.ts

@@ -4961,6 +4961,7 @@ interface ColorPickerProps {
     dropdownDirection?: "left" | "right" | "center";
     onColorPicked: (color: Color) => void;
     currentColor: Color;
+    disableNoColor?: boolean;
 }
 declare class ColorPicker extends Component<ColorPickerProps, SpreadsheetChildEnv> {
     static template: string;
@@ -5720,6 +5721,7 @@ interface DndState {
     y: Pixel;
     width: Pixel;
     height: Pixel;
+    cancelDnd: (() => void) | undefined;
 }
 interface Props$e {
     sidePanelIsOpen: Boolean;

package/dist/o-spreadsheet.esm.js

@@ -9970,6 +9970,9 @@ class ConditionalFormattingPanel extends Component {
         }
     }
     setColorScaleColor(target, color) {
+        if (!isColorValid(color)) {
+            return;
+        }
         const point = this.state.rules.colorScale[target];
         if (point) {
             point.color = Number.parseInt(color.substr(1), 16);
@@ -10681,15 +10684,23 @@ class ChartFigure extends Component {
 ChartFigure.template = "o-spreadsheet-ChartFigure";
 ChartFigure.components = { Menu };
 
+/**
+ * Start listening to pointer events and apply the given callbacks.
+ *
+ * @returns A function to remove the listeners.
+ */
 function startDnd(onMouseMove, onMouseUp, onMouseDown = () => { }) {
-    const _onMouseUp = (ev) => {
-        onMouseUp(ev);
+    const removeListeners = () => {
         window.removeEventListener("mousedown", onMouseDown);
         window.removeEventListener("mouseup", _onMouseUp);
         window.removeEventListener("dragstart", _onDragStart);
         window.removeEventListener("mousemove", onMouseMove);
         window.removeEventListener("wheel", onMouseMove);
     };
+    const _onMouseUp = (ev) => {
+        onMouseUp(ev);
+        removeListeners();
+    };
     function _onDragStart(ev) {
         ev.preventDefault();
     }
@@ -10698,6 +10709,7 @@ function startDnd(onMouseMove, onMouseUp, onMouseDown = () => { }) {
     window.addEventListener("dragstart", _onDragStart);
     window.addEventListener("mousemove", onMouseMove);
     window.addEventListener("wheel", onMouseMove);
+    return removeListeners;
 }
 /**
  * Function to be used during a mousedown event, this function allows to
@@ -18392,13 +18404,12 @@ function tokenizeSymbol(chars) {
 }
 const whiteSpaceRegexp = /\s/;
 function tokenizeSpace(chars) {
-    let length = 0;
+    let spaces = "";
     while (chars[0] && chars[0].match(whiteSpaceRegexp)) {
-        length++;
-        chars.shift();
+        spaces += chars.shift();
     }
-    if (length) {
-        return { type: "SPACE", value: " ".repeat(length) };
+    if (spaces) {
+        return { type: "SPACE", value: spaces };
     }
     return null;
 }
@@ -20425,11 +20436,11 @@ css /* scss */ `
   }
 
   /* Custom css to highlight topbar composer on focus */
-  .o-topbar-toolbar .o-composer-container:focus-within {
-    border: 1px solid ${SELECTION_BORDER_COLOR};
+  .o-topbar-toolbar .o-composer-container[active] {
+    border: 1px solid ${SELECTION_BORDER_COLOR} !important;
   }
 
-  .o-topbar-toolbar .o-composer-container {
+  .o-topbar-toolbar .o-composer-container[active] {
     z-index: ${ComponentsImportance.TopBarComposer};
   }
 `;
@@ -21378,6 +21389,7 @@ class FiguresContainer extends Component {
             y: 0,
             width: 0,
             height: 0,
+            cancelDnd: undefined,
         });
     }
     setup() {
@@ -21391,11 +21403,24 @@ class FiguresContainer extends Component {
             // new rendering
             this.render();
         });
+        onWillUpdateProps(() => {
+            const sheetId = this.env.model.getters.getActiveSheetId();
+            if (this.dnd.figId && !this.env.model.getters.getFigure(sheetId, this.dnd.figId)) {
+                if (this.dnd.cancelDnd) {
+                    this.dnd.cancelDnd();
+                }
+                this.dnd.figId = undefined;
+                this.dnd.cancelDnd = undefined;
+            }
+        });
     }
     getVisibleFigures() {
         const visibleFigures = this.env.model.getters.getVisibleFigures();
         if (this.dnd.figId && !visibleFigures.some((figure) => figure.id === this.dnd.figId)) {
-            visibleFigures.push(this.env.model.getters.getFigure(this.env.model.getters.getActiveSheetId(), this.dnd.figId));
+            const draggedFigure = this.env.model.getters.getFigure(this.env.model.getters.getActiveSheetId(), this.dnd.figId);
+            if (draggedFigure) {
+                visibleFigures.push(draggedFigure);
+            }
         }
         return visibleFigures;
     }
@@ -21508,7 +21533,7 @@ class FiguresContainer extends Component {
             this.dnd.figId = undefined;
             this.env.model.dispatch("UPDATE_FIGURE", { sheetId, id: figure.id, x, y });
         };
-        startDnd(onMouseMove, onMouseUp);
+        this.dnd.cancelDnd = startDnd(onMouseMove, onMouseUp);
     }
     startResize(figure, dirX, dirY, ev) {
         ev.stopPropagation();
@@ -21562,7 +21587,7 @@ class FiguresContainer extends Component {
                 ...update,
             });
         };
-        startDnd(onMouseMove, onMouseUp);
+        this.dnd.cancelDnd = startDnd(onMouseMove, onMouseUp);
     }
     getDndFigure() {
         const figure = this.getVisibleFigures().find((fig) => fig.id === this.dnd.figId);
@@ -27931,7 +27956,6 @@ class BordersPlugin extends CorePlugin {
 }
 BordersPlugin.getters = ["getCellBorder"];
 
-const nbspRegexp = new RegExp(String.fromCharCode(160), "g");
 const LINK_STYLE = { textColor: LINK_COLOR };
 /**
  * Core Plugin
@@ -28265,13 +28289,10 @@ class CellPlugin extends CorePlugin {
         return id;
     }
     updateCell(sheetId, col, row, after) {
-        var _a;
         const before = this.getters.getCell(sheetId, col, row);
         const hasContent = "content" in after || "formula" in after;
         // Compute the new cell properties
-        const afterContent = hasContent
-            ? ((_a = after.content) === null || _a === void 0 ? void 0 : _a.replace(nbspRegexp, "")) || ""
-            : (before === null || before === void 0 ? void 0 : before.content) || "";
+        const afterContent = (hasContent ? after.content : before === null || before === void 0 ? void 0 : before.content) || "";
         let style;
         if (after.style !== undefined) {
             style = after.style || undefined;
@@ -37155,6 +37176,7 @@ class SheetViewPlugin extends UIPlugin {
                 break;
             case "DELETE_SHEET":
                 this.cleanViewports();
+                this.sheetsWithDirtyViewports.delete(cmd.sheetId);
                 break;
             case "ACTIVATE_SHEET":
                 this.setViewports();
@@ -37507,6 +37529,9 @@ class SheetViewPlugin extends UIPlugin {
         };
     }
     resetViewports(sheetId) {
+        if (!this.getters.tryGetSheet(sheetId)) {
+            return;
+        }
         const { xSplit, ySplit } = this.getters.getPaneDivisions(sheetId);
         const nCols = this.getters.getNumberCols(sheetId);
         const nRows = this.getters.getNumberRows(sheetId);
@@ -43427,6 +43452,6 @@ function addFunction(functionName, functionDescription) {
 export { AbstractChart, CommandResult, CorePlugin, DATETIME_FORMAT, DispatchResult, EvaluationError, Model, Registry, Revision, SPREADSHEET_DIMENSIONS, Spreadsheet, UIPlugin, __info__, addFunction, astToFormula, cellTypes, compile, components, convertAstNodes, coreTypes, findCellInNewZone, functionCache, helpers, invalidateEvaluationCommands, load, parse, readonlyAllowedCommands, registries, setTranslationMethod, tokenize };
 
 
-__info__.version = '16.0.54';
-__info__.date = '2024-11-08T12:11:01.946Z';
-__info__.hash = 'fdfa7dc';
+__info__.version = '16.0.56';
+__info__.date = '2024-12-19T07:45:47.250Z';
+__info__.hash = '9d0e335';

package/dist/o-spreadsheet.iife.js

@@ -9971,6 +9971,9 @@
           }
       }
       setColorScaleColor(target, color) {
+          if (!isColorValid(color)) {
+              return;
+          }
           const point = this.state.rules.colorScale[target];
           if (point) {
               point.color = Number.parseInt(color.substr(1), 16);
@@ -10682,15 +10685,23 @@
   ChartFigure.template = "o-spreadsheet-ChartFigure";
   ChartFigure.components = { Menu };
 
+  /**
+   * Start listening to pointer events and apply the given callbacks.
+   *
+   * @returns A function to remove the listeners.
+   */
   function startDnd(onMouseMove, onMouseUp, onMouseDown = () => { }) {
-      const _onMouseUp = (ev) => {
-          onMouseUp(ev);
+      const removeListeners = () => {
           window.removeEventListener("mousedown", onMouseDown);
           window.removeEventListener("mouseup", _onMouseUp);
           window.removeEventListener("dragstart", _onDragStart);
           window.removeEventListener("mousemove", onMouseMove);
           window.removeEventListener("wheel", onMouseMove);
       };
+      const _onMouseUp = (ev) => {
+          onMouseUp(ev);
+          removeListeners();
+      };
       function _onDragStart(ev) {
           ev.preventDefault();
       }
@@ -10699,6 +10710,7 @@
       window.addEventListener("dragstart", _onDragStart);
       window.addEventListener("mousemove", onMouseMove);
       window.addEventListener("wheel", onMouseMove);
+      return removeListeners;
   }
   /**
    * Function to be used during a mousedown event, this function allows to
@@ -18393,13 +18405,12 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
   }
   const whiteSpaceRegexp = /\s/;
   function tokenizeSpace(chars) {
-      let length = 0;
+      let spaces = "";
       while (chars[0] && chars[0].match(whiteSpaceRegexp)) {
-          length++;
-          chars.shift();
+          spaces += chars.shift();
       }
-      if (length) {
-          return { type: "SPACE", value: " ".repeat(length) };
+      if (spaces) {
+          return { type: "SPACE", value: spaces };
       }
       return null;
   }
@@ -20426,11 +20437,11 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
   }
 
   /* Custom css to highlight topbar composer on focus */
-  .o-topbar-toolbar .o-composer-container:focus-within {
-    border: 1px solid ${SELECTION_BORDER_COLOR};
+  .o-topbar-toolbar .o-composer-container[active] {
+    border: 1px solid ${SELECTION_BORDER_COLOR} !important;
   }
 
-  .o-topbar-toolbar .o-composer-container {
+  .o-topbar-toolbar .o-composer-container[active] {
     z-index: ${ComponentsImportance.TopBarComposer};
   }
 `;
@@ -21379,6 +21390,7 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
               y: 0,
               width: 0,
               height: 0,
+              cancelDnd: undefined,
           });
       }
       setup() {
@@ -21392,11 +21404,24 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
               // new rendering
               this.render();
           });
+          owl.onWillUpdateProps(() => {
+              const sheetId = this.env.model.getters.getActiveSheetId();
+              if (this.dnd.figId && !this.env.model.getters.getFigure(sheetId, this.dnd.figId)) {
+                  if (this.dnd.cancelDnd) {
+                      this.dnd.cancelDnd();
+                  }
+                  this.dnd.figId = undefined;
+                  this.dnd.cancelDnd = undefined;
+              }
+          });
       }
       getVisibleFigures() {
           const visibleFigures = this.env.model.getters.getVisibleFigures();
           if (this.dnd.figId && !visibleFigures.some((figure) => figure.id === this.dnd.figId)) {
-              visibleFigures.push(this.env.model.getters.getFigure(this.env.model.getters.getActiveSheetId(), this.dnd.figId));
+              const draggedFigure = this.env.model.getters.getFigure(this.env.model.getters.getActiveSheetId(), this.dnd.figId);
+              if (draggedFigure) {
+                  visibleFigures.push(draggedFigure);
+              }
           }
           return visibleFigures;
       }
@@ -21509,7 +21534,7 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
               this.dnd.figId = undefined;
               this.env.model.dispatch("UPDATE_FIGURE", { sheetId, id: figure.id, x, y });
           };
-          startDnd(onMouseMove, onMouseUp);
+          this.dnd.cancelDnd = startDnd(onMouseMove, onMouseUp);
       }
       startResize(figure, dirX, dirY, ev) {
           ev.stopPropagation();
@@ -21563,7 +21588,7 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
                   ...update,
               });
           };
-          startDnd(onMouseMove, onMouseUp);
+          this.dnd.cancelDnd = startDnd(onMouseMove, onMouseUp);
       }
       getDndFigure() {
           const figure = this.getVisibleFigures().find((fig) => fig.id === this.dnd.figId);
@@ -27932,7 +27957,6 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
   }
   BordersPlugin.getters = ["getCellBorder"];
 
-  const nbspRegexp = new RegExp(String.fromCharCode(160), "g");
   const LINK_STYLE = { textColor: LINK_COLOR };
   /**
    * Core Plugin
@@ -28266,13 +28290,10 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
           return id;
       }
       updateCell(sheetId, col, row, after) {
-          var _a;
           const before = this.getters.getCell(sheetId, col, row);
           const hasContent = "content" in after || "formula" in after;
           // Compute the new cell properties
-          const afterContent = hasContent
-              ? ((_a = after.content) === null || _a === void 0 ? void 0 : _a.replace(nbspRegexp, "")) || ""
-              : (before === null || before === void 0 ? void 0 : before.content) || "";
+          const afterContent = (hasContent ? after.content : before === null || before === void 0 ? void 0 : before.content) || "";
           let style;
           if (after.style !== undefined) {
               style = after.style || undefined;
@@ -37156,6 +37177,7 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
                   break;
               case "DELETE_SHEET":
                   this.cleanViewports();
+                  this.sheetsWithDirtyViewports.delete(cmd.sheetId);
                   break;
               case "ACTIVATE_SHEET":
                   this.setViewports();
@@ -37508,6 +37530,9 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
           };
       }
       resetViewports(sheetId) {
+          if (!this.getters.tryGetSheet(sheetId)) {
+              return;
+          }
           const { xSplit, ySplit } = this.getters.getPaneDivisions(sheetId);
           const nCols = this.getters.getNumberCols(sheetId);
           const nRows = this.getters.getNumberRows(sheetId);
@@ -43458,9 +43483,9 @@ day_count_convention (number, default=${DEFAULT_DAY_COUNT_CONVENTION} ) ${_lt("A
   Object.defineProperty(exports, '__esModule', { value: true });
 
 
-  __info__.version = '16.0.54';
-  __info__.date = '2024-11-08T12:11:01.946Z';
-  __info__.hash = 'fdfa7dc';
+  __info__.version = '16.0.56';
+  __info__.date = '2024-12-19T07:45:47.250Z';
+  __info__.hash = '9d0e335';
 
 
 })(this.o_spreadsheet = this.o_spreadsheet || {}, owl);