@odoo/o-spreadsheet 16.3.0 → 16.3.1

package/dist/o-spreadsheet.cjs.js

@@ -16873,6 +16873,7 @@ const unFreezePane = {
     execute: (env) => env.model.dispatch("UNFREEZE_COLUMNS_ROWS", {
         sheetId: env.model.getters.getActiveSheetId(),
     }),
+    icon: "o-spreadsheet-Icon.UNFREEZE",
 };
 const freezePane = {
     name: _lt("Freeze"),
@@ -17944,6 +17945,18 @@ css /* scss */ `
       grid-template-columns: repeat(${ITEMS_PER_LINE}, 1fr);
       grid-gap: ${ITEM_HORIZONTAL_MARGIN * 2}px;
     }
+    .o-color-picker-toggler-button {
+      display: flex;
+      .o-color-picker-toggler-sign {
+        margin: auto auto;
+        width: 55%;
+        height: 55%;
+        .o-icon {
+          width: 100%;
+          height: 100%;
+        }
+      }
+    }
     .o-color-picker-line-item {
       width: ${ITEM_EDGE_LENGTH}px;
       height: ${ITEM_EDGE_LENGTH}px;
@@ -20616,7 +20629,14 @@ class ContentEditableHelper {
             current = it.next();
         }
         if (current.value !== nodeToFind) {
-            throw new Error("Cannot find the node in the children of the element");
+            /** This situation can happen if the code is called while the selection is not currently on the ContentEditableHelper.
+             * In this case, we return 0 because we don't know the size of the text before the selection.
+             *
+             * A known occurence is triggered since the introduction of commit d4663158 (PR #2038).
+             *
+             * FIXME: find a way to test eventhough the selection API is not available in jsDOM.
+             */
+            return 0;
         }
         else {
             if (!current.value.hasChildNodes()) {
@@ -38668,7 +38688,8 @@ class SheetUIPlugin extends UIPlugin {
     getCellWidth(position) {
         const text = this.getCellText(position);
         const style = this.getters.getCellComputedStyle(position);
-        let contentWidth = this.getTextWidth(text, style);
+        const multiLineText = text.split(NEWLINE);
+        let contentWidth = Math.max(...multiLineText.map((line) => this.getTextWidth(line, style)));
         const icon = this.getters.getConditionalIcon(position);
         if (icon) {
             contentWidth += computeIconWidth(this.getters.getCellStyle(position));
@@ -43797,6 +43818,10 @@ class Spreadsheet extends owl.Component {
     }
     checkViewportSize() {
         const { xRatio, yRatio } = this.env.model.getters.getFrozenSheetViewRatio(this.env.model.getters.getActiveSheetId());
+        if (!isFinite(xRatio) || !isFinite(yRatio)) {
+            // before mounting, the ratios can be NaN or Infinity if the viewport size is 0
+            return;
+        }
         if (yRatio > MAXIMAL_FREEZABLE_RATIO || xRatio > MAXIMAL_FREEZABLE_RATIO) {
             if (this.isViewportTooSmall) {
                 return;
@@ -47558,6 +47583,6 @@ exports.setTranslationMethod = setTranslationMethod;
 exports.tokenize = tokenize;
 
 
-__info__.version = '16.3.0';
-__info__.date = '2023-05-17T14:44:57.860Z';
-__info__.hash = '33b37e4';
+__info__.version = '16.3.1';
+__info__.date = '2023-05-25T13:00:27.320Z';
+__info__.hash = 'b3cfd2c';

package/dist/o-spreadsheet.esm.js

@@ -16869,6 +16869,7 @@ const unFreezePane = {
     execute: (env) => env.model.dispatch("UNFREEZE_COLUMNS_ROWS", {
         sheetId: env.model.getters.getActiveSheetId(),
     }),
+    icon: "o-spreadsheet-Icon.UNFREEZE",
 };
 const freezePane = {
     name: _lt("Freeze"),
@@ -17940,6 +17941,18 @@ css /* scss */ `
       grid-template-columns: repeat(${ITEMS_PER_LINE}, 1fr);
       grid-gap: ${ITEM_HORIZONTAL_MARGIN * 2}px;
     }
+    .o-color-picker-toggler-button {
+      display: flex;
+      .o-color-picker-toggler-sign {
+        margin: auto auto;
+        width: 55%;
+        height: 55%;
+        .o-icon {
+          width: 100%;
+          height: 100%;
+        }
+      }
+    }
     .o-color-picker-line-item {
       width: ${ITEM_EDGE_LENGTH}px;
       height: ${ITEM_EDGE_LENGTH}px;
@@ -20612,7 +20625,14 @@ class ContentEditableHelper {
             current = it.next();
         }
         if (current.value !== nodeToFind) {
-            throw new Error("Cannot find the node in the children of the element");
+            /** This situation can happen if the code is called while the selection is not currently on the ContentEditableHelper.
+             * In this case, we return 0 because we don't know the size of the text before the selection.
+             *
+             * A known occurence is triggered since the introduction of commit d4663158 (PR #2038).
+             *
+             * FIXME: find a way to test eventhough the selection API is not available in jsDOM.
+             */
+            return 0;
         }
         else {
             if (!current.value.hasChildNodes()) {
@@ -38664,7 +38684,8 @@ class SheetUIPlugin extends UIPlugin {
     getCellWidth(position) {
         const text = this.getCellText(position);
         const style = this.getters.getCellComputedStyle(position);
-        let contentWidth = this.getTextWidth(text, style);
+        const multiLineText = text.split(NEWLINE);
+        let contentWidth = Math.max(...multiLineText.map((line) => this.getTextWidth(line, style)));
         const icon = this.getters.getConditionalIcon(position);
         if (icon) {
             contentWidth += computeIconWidth(this.getters.getCellStyle(position));
@@ -43793,6 +43814,10 @@ class Spreadsheet extends Component {
     }
     checkViewportSize() {
         const { xRatio, yRatio } = this.env.model.getters.getFrozenSheetViewRatio(this.env.model.getters.getActiveSheetId());
+        if (!isFinite(xRatio) || !isFinite(yRatio)) {
+            // before mounting, the ratios can be NaN or Infinity if the viewport size is 0
+            return;
+        }
         if (yRatio > MAXIMAL_FREEZABLE_RATIO || xRatio > MAXIMAL_FREEZABLE_RATIO) {
             if (this.isViewportTooSmall) {
                 return;
@@ -47526,6 +47551,6 @@ function addFunction(functionName, functionDescription) {
 export { AbstractChart, CommandResult, CorePlugin, DispatchResult, EvaluationError, Model, Registry, Revision, SPREADSHEET_DIMENSIONS, Spreadsheet, UIPlugin, __info__, addFunction, astToFormula, compile, components, convertAstNodes, coreTypes, findCellInNewZone, functionCache, helpers, invalidateEvaluationCommands, links, load, parse, readonlyAllowedCommands, registries, setDefaultSheetViewSize, setTranslationMethod, tokenize };
 
 
-__info__.version = '16.3.0';
-__info__.date = '2023-05-17T14:44:57.860Z';
-__info__.hash = '33b37e4';
+__info__.version = '16.3.1';
+__info__.date = '2023-05-25T13:00:27.320Z';
+__info__.hash = 'b3cfd2c';

package/dist/o-spreadsheet.iife.js

@@ -16870,6 +16870,7 @@
       execute: (env) => env.model.dispatch("UNFREEZE_COLUMNS_ROWS", {
           sheetId: env.model.getters.getActiveSheetId(),
       }),
+      icon: "o-spreadsheet-Icon.UNFREEZE",
   };
   const freezePane = {
       name: _lt("Freeze"),
@@ -17941,6 +17942,18 @@
       grid-template-columns: repeat(${ITEMS_PER_LINE}, 1fr);
       grid-gap: ${ITEM_HORIZONTAL_MARGIN * 2}px;
     }
+    .o-color-picker-toggler-button {
+      display: flex;
+      .o-color-picker-toggler-sign {
+        margin: auto auto;
+        width: 55%;
+        height: 55%;
+        .o-icon {
+          width: 100%;
+          height: 100%;
+        }
+      }
+    }
     .o-color-picker-line-item {
       width: ${ITEM_EDGE_LENGTH}px;
       height: ${ITEM_EDGE_LENGTH}px;
@@ -20613,7 +20626,14 @@
               current = it.next();
           }
           if (current.value !== nodeToFind) {
-              throw new Error("Cannot find the node in the children of the element");
+              /** This situation can happen if the code is called while the selection is not currently on the ContentEditableHelper.
+               * In this case, we return 0 because we don't know the size of the text before the selection.
+               *
+               * A known occurence is triggered since the introduction of commit d4663158 (PR #2038).
+               *
+               * FIXME: find a way to test eventhough the selection API is not available in jsDOM.
+               */
+              return 0;
           }
           else {
               if (!current.value.hasChildNodes()) {
@@ -38665,7 +38685,8 @@
       getCellWidth(position) {
           const text = this.getCellText(position);
           const style = this.getters.getCellComputedStyle(position);
-          let contentWidth = this.getTextWidth(text, style);
+          const multiLineText = text.split(NEWLINE);
+          let contentWidth = Math.max(...multiLineText.map((line) => this.getTextWidth(line, style)));
           const icon = this.getters.getConditionalIcon(position);
           if (icon) {
               contentWidth += computeIconWidth(this.getters.getCellStyle(position));
@@ -43794,6 +43815,10 @@
       }
       checkViewportSize() {
           const { xRatio, yRatio } = this.env.model.getters.getFrozenSheetViewRatio(this.env.model.getters.getActiveSheetId());
+          if (!isFinite(xRatio) || !isFinite(yRatio)) {
+              // before mounting, the ratios can be NaN or Infinity if the viewport size is 0
+              return;
+          }
           if (yRatio > MAXIMAL_FREEZABLE_RATIO || xRatio > MAXIMAL_FREEZABLE_RATIO) {
               if (this.isViewportTooSmall) {
                   return;
@@ -47557,9 +47582,9 @@
   Object.defineProperty(exports, '__esModule', { value: true });
 
 
-  __info__.version = '16.3.0';
-  __info__.date = '2023-05-17T14:44:57.860Z';
-  __info__.hash = '33b37e4';
+  __info__.version = '16.3.1';
+  __info__.date = '2023-05-25T13:00:27.320Z';
+  __info__.hash = 'b3cfd2c';
 
 
 })(this.o_spreadsheet = this.o_spreadsheet || {}, owl);