@odoo/o-spreadsheet 15.2.3 → 15.2.5

package/dist/o-spreadsheet.cjs.js

@@ -372,7 +372,7 @@ function formatJSTime(jsDate, format) {
  * - it does not accept "," as thousand separator, because when we tokenize a
  *   formula, commas are used to separate arguments
  */
-const formulaNumberRegexp = /^-?\d+(\.?\d*(e\d+)?)?(\s*%)?|^-?\.\d+(\s*%)?/;
+const formulaNumberRegexp = /(^-?\d+(\.?\d*(e\d+)?)?(\s*%)?|^-?\.\d+(\s*%)?)(?!\w|!)/;
 const numberRegexp = /^-?\d+(,\d+)*(\.?\d*(e\d+)?)?(\s*%)?$|^-?\.\d+(\s*%)?$/;
 /**
  * Return true if the argument is a "number string".
@@ -26031,7 +26031,7 @@ const { Portal } = owl.misc;
 const { xml: xml$l } = owl.tags;
 const TEMPLATE$j = xml$l /* xml */ `
   <Portal target="'.o-spreadsheet'">
-    <div t-att-style="style">
+    <div class="o-popover" t-att-style="style">
       <t t-slot="default"/>
     </div>
   </Portal>
@@ -27022,14 +27022,31 @@ class ContentEditableHelper {
      * */
     getCurrentSelection() {
         let { startElement, endElement, startSelectionOffset, endSelectionOffset } = this.getStartAndEndSelection();
-        let startSizeBefore = this.findSizeBeforeElement(startElement);
-        let endSizeBefore = this.findSizeBeforeElement(endElement);
+        let startSizeBefore = this.findSelectionIndex(startElement, startSelectionOffset);
+        let endSizeBefore = this.findSelectionIndex(endElement, endSelectionOffset);
         return {
-            start: startSizeBefore + startSelectionOffset,
-            end: endSizeBefore + endSelectionOffset,
+            start: startSizeBefore,
+            end: endSizeBefore,
         };
     }
-    findSizeBeforeElement(nodeToFind) {
+    /**
+     * Computes the text 'index' inside this.el based on the currently selected node and its offset.
+     * The selected node is either a Text node or an Element node.
+     *
+     * case 1 -Text node:
+     * the offset is the number of characters from the start of the node. We have to add this offset to the
+     * content length of all previous nodes.
+     *
+     * case 2 - Element node:
+     * the offset is the number of child nodes before the selected node. We have to add the content length of
+     * all the bnodes prior to the selected node as well as the content of the child node before the offset.
+     *
+     * See the MDN documentation for more details.
+     * https://developer.mozilla.org/en-US/docs/Web/API/Range/startOffset
+     * https://developer.mozilla.org/en-US/docs/Web/API/Range/endOffset
+     *
+     */
+    findSelectionIndex(nodeToFind, nodeOffset) {
         let it = this.iterateChildren(this.el);
         let usedCharacters = 0;
         let current = it.next();
@@ -27041,6 +27058,19 @@ class ContentEditableHelper {
             }
             current = it.next();
         }
+        if (current.value === nodeToFind) {
+            if (!current.value.hasChildNodes()) {
+                usedCharacters += nodeOffset;
+            }
+            else {
+                const children = [...current.value.childNodes].slice(0, nodeOffset);
+                for (const child of children) {
+                    if (!child.textContent)
+                        continue;
+                    usedCharacters += child.textContent.length;
+                }
+            }
+        }
         return usedCharacters;
     }
     getStartAndEndSelection() {
@@ -27265,7 +27295,7 @@ const CSS$b = css$d /* scss */ `
     padding: 0;
     margin: 0;
     border: 0;
-    z-index: 5;
+    z-index: 6;
     flex-grow: 1;
     max-height: inherit;
     .o-composer {
@@ -27834,6 +27864,10 @@ const CSS$9 = css$b /* scss */ `
     border-left: 3px solid red;
     padding: 10px;
   }
+
+  .o-popover:has(.o-error-tooltip) {
+    z-index: 4 !important;
+  }
 `;
 class ErrorToolTip extends owl.Component {
 }
@@ -31298,6 +31332,6 @@ exports.registries = registries;
 exports.setTranslationMethod = setTranslationMethod;
 
 
-__info__.version = '15.2.3';
-__info__.date = '2023-03-23T11:32:58.083Z';
-__info__.hash = '0541ed5';
+__info__.version = '15.2.5';
+__info__.date = '2023-04-21T07:27:24.954Z';
+__info__.hash = '0896859';

package/dist/o-spreadsheet.d.ts

@@ -4288,7 +4288,24 @@ declare class ContentEditableHelper {
         start: number;
         end: number;
     };
-    private findSizeBeforeElement;
+    /**
+     * Computes the text 'index' inside this.el based on the currently selected node and its offset.
+     * The selected node is either a Text node or an Element node.
+     *
+     * case 1 -Text node:
+     * the offset is the number of characters from the start of the node. We have to add this offset to the
+     * content length of all previous nodes.
+     *
+     * case 2 - Element node:
+     * the offset is the number of child nodes before the selected node. We have to add the content length of
+     * all the bnodes prior to the selected node as well as the content of the child node before the offset.
+     *
+     * See the MDN documentation for more details.
+     * https://developer.mozilla.org/en-US/docs/Web/API/Range/startOffset
+     * https://developer.mozilla.org/en-US/docs/Web/API/Range/endOffset
+     *
+     */
+    private findSelectionIndex;
     private getStartAndEndSelection;
 }
 

package/dist/o-spreadsheet.esm.js

@@ -349,7 +349,7 @@ function formatJSTime(jsDate, format) {
  * - it does not accept "," as thousand separator, because when we tokenize a
  *   formula, commas are used to separate arguments
  */
-const formulaNumberRegexp = /^-?\d+(\.?\d*(e\d+)?)?(\s*%)?|^-?\.\d+(\s*%)?/;
+const formulaNumberRegexp = /(^-?\d+(\.?\d*(e\d+)?)?(\s*%)?|^-?\.\d+(\s*%)?)(?!\w|!)/;
 const numberRegexp = /^-?\d+(,\d+)*(\.?\d*(e\d+)?)?(\s*%)?$|^-?\.\d+(\s*%)?$/;
 /**
  * Return true if the argument is a "number string".
@@ -26008,7 +26008,7 @@ const { Portal } = misc$1;
 const { xml: xml$l } = tags;
 const TEMPLATE$j = xml$l /* xml */ `
   <Portal target="'.o-spreadsheet'">
-    <div t-att-style="style">
+    <div class="o-popover" t-att-style="style">
       <t t-slot="default"/>
     </div>
   </Portal>
@@ -26999,14 +26999,31 @@ class ContentEditableHelper {
      * */
     getCurrentSelection() {
         let { startElement, endElement, startSelectionOffset, endSelectionOffset } = this.getStartAndEndSelection();
-        let startSizeBefore = this.findSizeBeforeElement(startElement);
-        let endSizeBefore = this.findSizeBeforeElement(endElement);
+        let startSizeBefore = this.findSelectionIndex(startElement, startSelectionOffset);
+        let endSizeBefore = this.findSelectionIndex(endElement, endSelectionOffset);
         return {
-            start: startSizeBefore + startSelectionOffset,
-            end: endSizeBefore + endSelectionOffset,
+            start: startSizeBefore,
+            end: endSizeBefore,
         };
     }
-    findSizeBeforeElement(nodeToFind) {
+    /**
+     * Computes the text 'index' inside this.el based on the currently selected node and its offset.
+     * The selected node is either a Text node or an Element node.
+     *
+     * case 1 -Text node:
+     * the offset is the number of characters from the start of the node. We have to add this offset to the
+     * content length of all previous nodes.
+     *
+     * case 2 - Element node:
+     * the offset is the number of child nodes before the selected node. We have to add the content length of
+     * all the bnodes prior to the selected node as well as the content of the child node before the offset.
+     *
+     * See the MDN documentation for more details.
+     * https://developer.mozilla.org/en-US/docs/Web/API/Range/startOffset
+     * https://developer.mozilla.org/en-US/docs/Web/API/Range/endOffset
+     *
+     */
+    findSelectionIndex(nodeToFind, nodeOffset) {
         let it = this.iterateChildren(this.el);
         let usedCharacters = 0;
         let current = it.next();
@@ -27018,6 +27035,19 @@ class ContentEditableHelper {
             }
             current = it.next();
         }
+        if (current.value === nodeToFind) {
+            if (!current.value.hasChildNodes()) {
+                usedCharacters += nodeOffset;
+            }
+            else {
+                const children = [...current.value.childNodes].slice(0, nodeOffset);
+                for (const child of children) {
+                    if (!child.textContent)
+                        continue;
+                    usedCharacters += child.textContent.length;
+                }
+            }
+        }
         return usedCharacters;
     }
     getStartAndEndSelection() {
@@ -27242,7 +27272,7 @@ const CSS$b = css$d /* scss */ `
     padding: 0;
     margin: 0;
     border: 0;
-    z-index: 5;
+    z-index: 6;
     flex-grow: 1;
     max-height: inherit;
     .o-composer {
@@ -27811,6 +27841,10 @@ const CSS$9 = css$b /* scss */ `
     border-left: 3px solid red;
     padding: 10px;
   }
+
+  .o-popover:has(.o-error-tooltip) {
+    z-index: 4 !important;
+  }
 `;
 class ErrorToolTip extends Component {
 }
@@ -31252,6 +31286,6 @@ const helpers = {
 export { CommandResult, CorePlugin, DATETIME_FORMAT, DataSource, DispatchResult, Model, Registry, Revision, SPREADSHEET_DIMENSIONS, Spreadsheet, UIPlugin, DEBUG as __DEBUG__, __info__, astToFormula, cellTypes, compile, convertAstNodes, coreTypes, functionCache, helpers, invalidateEvaluationCommands, parse, readonlyAllowedCommands, registries, setTranslationMethod };
 
 
-__info__.version = '15.2.3';
-__info__.date = '2023-03-23T11:32:58.083Z';
-__info__.hash = '0541ed5';
+__info__.version = '15.2.5';
+__info__.date = '2023-04-21T07:27:24.954Z';
+__info__.hash = '0896859';

package/dist/o-spreadsheet.iife.js

@@ -369,7 +369,7 @@
    * - it does not accept "," as thousand separator, because when we tokenize a
    *   formula, commas are used to separate arguments
    */
-  const formulaNumberRegexp = /^-?\d+(\.?\d*(e\d+)?)?(\s*%)?|^-?\.\d+(\s*%)?/;
+  const formulaNumberRegexp = /(^-?\d+(\.?\d*(e\d+)?)?(\s*%)?|^-?\.\d+(\s*%)?)(?!\w|!)/;
   const numberRegexp = /^-?\d+(,\d+)*(\.?\d*(e\d+)?)?(\s*%)?$|^-?\.\d+(\s*%)?$/;
   /**
    * Return true if the argument is a "number string".
@@ -26028,7 +26028,7 @@
   const { xml: xml$l } = owl.tags;
   const TEMPLATE$j = xml$l /* xml */ `
   <Portal target="'.o-spreadsheet'">
-    <div t-att-style="style">
+    <div class="o-popover" t-att-style="style">
       <t t-slot="default"/>
     </div>
   </Portal>
@@ -27019,14 +27019,31 @@
        * */
       getCurrentSelection() {
           let { startElement, endElement, startSelectionOffset, endSelectionOffset } = this.getStartAndEndSelection();
-          let startSizeBefore = this.findSizeBeforeElement(startElement);
-          let endSizeBefore = this.findSizeBeforeElement(endElement);
+          let startSizeBefore = this.findSelectionIndex(startElement, startSelectionOffset);
+          let endSizeBefore = this.findSelectionIndex(endElement, endSelectionOffset);
           return {
-              start: startSizeBefore + startSelectionOffset,
-              end: endSizeBefore + endSelectionOffset,
+              start: startSizeBefore,
+              end: endSizeBefore,
           };
       }
-      findSizeBeforeElement(nodeToFind) {
+      /**
+       * Computes the text 'index' inside this.el based on the currently selected node and its offset.
+       * The selected node is either a Text node or an Element node.
+       *
+       * case 1 -Text node:
+       * the offset is the number of characters from the start of the node. We have to add this offset to the
+       * content length of all previous nodes.
+       *
+       * case 2 - Element node:
+       * the offset is the number of child nodes before the selected node. We have to add the content length of
+       * all the bnodes prior to the selected node as well as the content of the child node before the offset.
+       *
+       * See the MDN documentation for more details.
+       * https://developer.mozilla.org/en-US/docs/Web/API/Range/startOffset
+       * https://developer.mozilla.org/en-US/docs/Web/API/Range/endOffset
+       *
+       */
+      findSelectionIndex(nodeToFind, nodeOffset) {
           let it = this.iterateChildren(this.el);
           let usedCharacters = 0;
           let current = it.next();
@@ -27038,6 +27055,19 @@
               }
               current = it.next();
           }
+          if (current.value === nodeToFind) {
+              if (!current.value.hasChildNodes()) {
+                  usedCharacters += nodeOffset;
+              }
+              else {
+                  const children = [...current.value.childNodes].slice(0, nodeOffset);
+                  for (const child of children) {
+                      if (!child.textContent)
+                          continue;
+                      usedCharacters += child.textContent.length;
+                  }
+              }
+          }
           return usedCharacters;
       }
       getStartAndEndSelection() {
@@ -27262,7 +27292,7 @@
     padding: 0;
     margin: 0;
     border: 0;
-    z-index: 5;
+    z-index: 6;
     flex-grow: 1;
     max-height: inherit;
     .o-composer {
@@ -27831,6 +27861,10 @@
     border-left: 3px solid red;
     padding: 10px;
   }
+
+  .o-popover:has(.o-error-tooltip) {
+    z-index: 4 !important;
+  }
 `;
   class ErrorToolTip extends owl.Component {
   }
@@ -31297,9 +31331,9 @@
   Object.defineProperty(exports, '__esModule', { value: true });
 
 
-  __info__.version = '15.2.3';
-  __info__.date = '2023-03-23T11:32:58.083Z';
-  __info__.hash = '0541ed5';
+  __info__.version = '15.2.5';
+  __info__.date = '2023-04-21T07:27:24.954Z';
+  __info__.hash = '0896859';
 
 
 })(this.o_spreadsheet = this.o_spreadsheet || {}, owl);