@odoo/o-spreadsheet 17.4.27 → 17.4.28

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.27
- * @date 2025-03-19T08:27:27.414Z
- * @hash a87c1da
+ * @version 17.4.28
+ * @date 2025-03-26T12:48:33.387Z
+ * @hash a81a8f4
  */
 
 'use strict';
@@ -1009,7 +1009,10 @@ function rgbaStringToHex(color) {
     }
     else if (stringVals.length === 4) {
         const alpha = parseFloat(stringVals.pop() || "1");
-        alphaHex = Math.round((alpha || 1) * 255);
+        if (isNaN(alpha)) {
+            throw new Error("invalid alpha value");
+        }
+        alphaHex = Math.round(alpha * 255);
     }
     const vals = stringVals.map((val) => parseInt(val, 10));
     if (alphaHex !== 255) {
@@ -5965,7 +5968,7 @@ function isValidLocale(locale) {
  */
 function canonicalizeNumberContent(content, locale) {
     return content.startsWith("=")
-        ? canonicalizeFormula$1(content, locale)
+        ? canonicalizeFormula(content, locale)
         : canonicalizeNumberLiteral(content, locale);
 }
 /**
@@ -5980,7 +5983,7 @@ function canonicalizeNumberContent(content, locale) {
  */
 function canonicalizeContent(content, locale) {
     return content.startsWith("=")
-        ? canonicalizeFormula$1(content, locale)
+        ? canonicalizeFormula(content, locale)
         : canonicalizeLiteral(content, locale);
 }
 /**
@@ -5996,15 +5999,21 @@ function localizeContent(content, locale) {
         ? localizeFormula(content, locale)
         : localizeLiteral(content, locale);
 }
+/** Change a number string to its canonical form (en_US locale) */
+function canonicalizeNumberValue(content, locale) {
+    return content.startsWith("=")
+        ? canonicalizeFormula(content, locale)
+        : canonicalizeNumberLiteral(content, locale);
+}
 /** Change a formula to its canonical form (en_US locale) */
-function canonicalizeFormula$1(formula, locale) {
-    return _localizeFormula$1(formula, locale, DEFAULT_LOCALE);
+function canonicalizeFormula(formula, locale) {
+    return _localizeFormula(formula, locale, DEFAULT_LOCALE);
 }
 /** Change a formula from the canonical form to the given locale */
 function localizeFormula(formula, locale) {
-    return _localizeFormula$1(formula, DEFAULT_LOCALE, locale);
+    return _localizeFormula(formula, DEFAULT_LOCALE, locale);
 }
-function _localizeFormula$1(formula, fromLocale, toLocale) {
+function _localizeFormula(formula, fromLocale, toLocale) {
     if (fromLocale.formulaArgSeparator === toLocale.formulaArgSeparator &&
         fromLocale.decimalSeparator === toLocale.decimalSeparator) {
         return formula;
@@ -6159,37 +6168,6 @@ function getDateTimeFormat(locale) {
     return locale.dateFormat + " " + locale.timeFormat;
 }
 
-/** Change a number string to its canonical form (en_US locale) */
-function canonicalizeNumberValue(content, locale) {
-    return content.startsWith("=")
-        ? canonicalizeFormula(content, locale)
-        : canonicalizeNumberLiteral(content, locale);
-}
-/** Change a formula to its canonical form (en_US locale) */
-function canonicalizeFormula(formula, locale) {
-    return _localizeFormula(formula, locale, DEFAULT_LOCALE);
-}
-function _localizeFormula(formula, fromLocale, toLocale) {
-    if (fromLocale.formulaArgSeparator === toLocale.formulaArgSeparator &&
-        fromLocale.decimalSeparator === toLocale.decimalSeparator) {
-        return formula;
-    }
-    const tokens = tokenize(formula, fromLocale);
-    let localizedFormula = "";
-    for (const token of tokens) {
-        if (token.type === "NUMBER") {
-            localizedFormula += token.value.replace(fromLocale.decimalSeparator, toLocale.decimalSeparator);
-        }
-        else if (token.type === "ARG_SEPARATOR") {
-            localizedFormula += toLocale.formulaArgSeparator;
-        }
-        else {
-            localizedFormula += token.value;
-        }
-    }
-    return localizedFormula;
-}
-
 function boolAnd(args) {
     let foundBoolean = false;
     let acc = true;
@@ -12026,7 +12004,8 @@ function getChartLabelValues(getters, dataSets, labelRange) {
         }
     }
     else if (dataSets.length === 1) {
-        for (let i = 0; i < getData(getters, dataSets[0]).length; i++) {
+        const dataLength = getData(getters, dataSets[0]).length;
+        for (let i = 0; i < dataLength; i++) {
             labels.formattedValues.push("");
             labels.values.push("");
         }
@@ -43181,9 +43160,9 @@ const XLSX_CHART_TYPES = [
 /** In XLSX color format (no #)  */
 const AUTO_COLOR = "000000";
 const XLSX_ICONSET_MAP = {
-    arrow: "3Arrows",
+    arrows: "3Arrows",
     smiley: "3Symbols",
-    dot: "3TrafficLights1",
+    dots: "3TrafficLights1",
 };
 const NAMESPACE = {
     styleSheet: "http://schemas.openxmlformats.org/spreadsheetml/2006/main",
@@ -43599,6 +43578,7 @@ const ICON_SET_CONVERSION_MAP = {
 };
 /** Map between legend position in XLSX file and human readable position  */
 const DRAWING_LEGEND_POSITION_CONVERSION_MAP = {
+    none: "none",
     b: "bottom",
     t: "top",
     l: "left",
@@ -45848,7 +45828,7 @@ class XlsxChartExtractor extends XlsxBaseExtractor {
                     default: "ffffff",
                 }).asString(),
                 legendPosition: DRAWING_LEGEND_POSITION_CONVERSION_MAP[this.extractChildAttr(rootChartElement, "c:legendPos", "val", {
-                    default: "b",
+                    default: "none",
                 }).asString()],
                 stacked: barChartGrouping === "stacked",
                 fontColor: "000000",
@@ -45882,7 +45862,7 @@ class XlsxChartExtractor extends XlsxBaseExtractor {
                 default: "ffffff",
             }).asString(),
             legendPosition: DRAWING_LEGEND_POSITION_CONVERSION_MAP[this.extractChildAttr(chartElement, "c:legendPos", "val", {
-                default: "b",
+                default: "none",
             }).asString()],
             stacked: barChartGrouping === "stacked",
             fontColor: "000000",
@@ -64180,7 +64160,9 @@ css /* scss */ `
     margin-top: -1px;
     border: 1px solid;
 
-    .o-composer:empty:not(:focus):not(.active)::before {
+    /* In readonly we always show the fx icon if the composer is empty, not matter the focus */
+    .o-composer:empty:not(:focus):not(.active)::before,
+    &.o-topbar-composer-readonly .o-composer:empty::before {
       content: url("data:image/svg+xml,${encodeURIComponent(FX_SVG)}");
       position: relative;
       top: 20%;
@@ -67424,10 +67406,14 @@ function addIconSetRule(cf, rule) {
             continue;
         }
         const cfValueObjectNodes = cfValueObject.map((attrs) => escapeXml /*xml*/ `<cfvo ${formatAttributes(attrs)} />`);
+        const iconSetAttrs = [["iconSet", getIconSet(rule.icons)]];
+        if (isIconSetReversed(rule.icons)) {
+            iconSetAttrs.push(["reverse", "1"]);
+        }
         conditionalFormats.push(escapeXml /*xml*/ `
       <conditionalFormatting sqref="${range}">
         <cfRule ${formatAttributes(ruleAttributes)}>
-          <iconSet iconSet="${getIconSet(rule.icons)}">
+          <iconSet ${formatAttributes(iconSetAttrs)}>
             ${joinXmlNodes(cfValueObjectNodes)}
           </iconSet>
         </cfRule>
@@ -67445,9 +67431,21 @@ function commonCfAttributes(cf) {
         ["stopIfTrue", cf.stopIfTrue ? 1 : 0],
     ];
 }
+function isIconSetReversed(iconSet) {
+    const defaultIconSet = ICON_SETS[detectIconsType(iconSet)];
+    return iconSet.upper === defaultIconSet.bad && iconSet.lower === defaultIconSet.good;
+}
 function getIconSet(iconSet) {
-    return XLSX_ICONSET_MAP[Object.keys(XLSX_ICONSET_MAP).find((key) => iconSet.upper.toLowerCase().startsWith(key)) ||
-        "dots"];
+    return XLSX_ICONSET_MAP[detectIconsType(iconSet)];
+}
+/**
+ * Partial detection based on "upper" point only.
+ * We support any arbitrary icon in the set, while excel doesn't allow
+ * mixing icons from different types.
+ */
+function detectIconsType(iconSet) {
+    const type = Object.keys(ICON_SETS).find((type) => Object.values(ICON_SETS[type]).includes(iconSet.upper)) || "dots";
+    return type;
 }
 function thresholdAttributes(threshold, position) {
     const type = getExcelThresholdType(threshold.type, position);
@@ -69229,6 +69227,6 @@ exports.tokenColors = tokenColors;
 exports.tokenize = tokenize;
 
 
-__info__.version = "17.4.27";
-__info__.date = "2025-03-19T08:27:27.414Z";
-__info__.hash = "a87c1da";
+__info__.version = "17.4.28";
+__info__.date = "2025-03-26T12:48:33.387Z";
+__info__.hash = "a81a8f4";

package/dist/o-spreadsheet.esm.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.27
- * @date 2025-03-19T08:27:27.414Z
- * @hash a87c1da
+ * @version 17.4.28
+ * @date 2025-03-26T12:48:33.387Z
+ * @hash a81a8f4
  */
 
 import { useEnv, useSubEnv, onWillUnmount, useComponent, status, Component, useRef, onMounted, useEffect, useState, onPatched, onWillPatch, onWillUpdateProps, useExternalListener, onWillStart, xml, useChildSubEnv, markRaw, toRaw } from '@odoo/owl';
@@ -1007,7 +1007,10 @@ function rgbaStringToHex(color) {
     }
     else if (stringVals.length === 4) {
         const alpha = parseFloat(stringVals.pop() || "1");
-        alphaHex = Math.round((alpha || 1) * 255);
+        if (isNaN(alpha)) {
+            throw new Error("invalid alpha value");
+        }
+        alphaHex = Math.round(alpha * 255);
     }
     const vals = stringVals.map((val) => parseInt(val, 10));
     if (alphaHex !== 255) {
@@ -5963,7 +5966,7 @@ function isValidLocale(locale) {
  */
 function canonicalizeNumberContent(content, locale) {
     return content.startsWith("=")
-        ? canonicalizeFormula$1(content, locale)
+        ? canonicalizeFormula(content, locale)
         : canonicalizeNumberLiteral(content, locale);
 }
 /**
@@ -5978,7 +5981,7 @@ function canonicalizeNumberContent(content, locale) {
  */
 function canonicalizeContent(content, locale) {
     return content.startsWith("=")
-        ? canonicalizeFormula$1(content, locale)
+        ? canonicalizeFormula(content, locale)
         : canonicalizeLiteral(content, locale);
 }
 /**
@@ -5994,15 +5997,21 @@ function localizeContent(content, locale) {
         ? localizeFormula(content, locale)
         : localizeLiteral(content, locale);
 }
+/** Change a number string to its canonical form (en_US locale) */
+function canonicalizeNumberValue(content, locale) {
+    return content.startsWith("=")
+        ? canonicalizeFormula(content, locale)
+        : canonicalizeNumberLiteral(content, locale);
+}
 /** Change a formula to its canonical form (en_US locale) */
-function canonicalizeFormula$1(formula, locale) {
-    return _localizeFormula$1(formula, locale, DEFAULT_LOCALE);
+function canonicalizeFormula(formula, locale) {
+    return _localizeFormula(formula, locale, DEFAULT_LOCALE);
 }
 /** Change a formula from the canonical form to the given locale */
 function localizeFormula(formula, locale) {
-    return _localizeFormula$1(formula, DEFAULT_LOCALE, locale);
+    return _localizeFormula(formula, DEFAULT_LOCALE, locale);
 }
-function _localizeFormula$1(formula, fromLocale, toLocale) {
+function _localizeFormula(formula, fromLocale, toLocale) {
     if (fromLocale.formulaArgSeparator === toLocale.formulaArgSeparator &&
         fromLocale.decimalSeparator === toLocale.decimalSeparator) {
         return formula;
@@ -6157,37 +6166,6 @@ function getDateTimeFormat(locale) {
     return locale.dateFormat + " " + locale.timeFormat;
 }
 
-/** Change a number string to its canonical form (en_US locale) */
-function canonicalizeNumberValue(content, locale) {
-    return content.startsWith("=")
-        ? canonicalizeFormula(content, locale)
-        : canonicalizeNumberLiteral(content, locale);
-}
-/** Change a formula to its canonical form (en_US locale) */
-function canonicalizeFormula(formula, locale) {
-    return _localizeFormula(formula, locale, DEFAULT_LOCALE);
-}
-function _localizeFormula(formula, fromLocale, toLocale) {
-    if (fromLocale.formulaArgSeparator === toLocale.formulaArgSeparator &&
-        fromLocale.decimalSeparator === toLocale.decimalSeparator) {
-        return formula;
-    }
-    const tokens = tokenize(formula, fromLocale);
-    let localizedFormula = "";
-    for (const token of tokens) {
-        if (token.type === "NUMBER") {
-            localizedFormula += token.value.replace(fromLocale.decimalSeparator, toLocale.decimalSeparator);
-        }
-        else if (token.type === "ARG_SEPARATOR") {
-            localizedFormula += toLocale.formulaArgSeparator;
-        }
-        else {
-            localizedFormula += token.value;
-        }
-    }
-    return localizedFormula;
-}
-
 function boolAnd(args) {
     let foundBoolean = false;
     let acc = true;
@@ -12024,7 +12002,8 @@ function getChartLabelValues(getters, dataSets, labelRange) {
         }
     }
     else if (dataSets.length === 1) {
-        for (let i = 0; i < getData(getters, dataSets[0]).length; i++) {
+        const dataLength = getData(getters, dataSets[0]).length;
+        for (let i = 0; i < dataLength; i++) {
             labels.formattedValues.push("");
             labels.values.push("");
         }
@@ -43179,9 +43158,9 @@ const XLSX_CHART_TYPES = [
 /** In XLSX color format (no #)  */
 const AUTO_COLOR = "000000";
 const XLSX_ICONSET_MAP = {
-    arrow: "3Arrows",
+    arrows: "3Arrows",
     smiley: "3Symbols",
-    dot: "3TrafficLights1",
+    dots: "3TrafficLights1",
 };
 const NAMESPACE = {
     styleSheet: "http://schemas.openxmlformats.org/spreadsheetml/2006/main",
@@ -43597,6 +43576,7 @@ const ICON_SET_CONVERSION_MAP = {
 };
 /** Map between legend position in XLSX file and human readable position  */
 const DRAWING_LEGEND_POSITION_CONVERSION_MAP = {
+    none: "none",
     b: "bottom",
     t: "top",
     l: "left",
@@ -45846,7 +45826,7 @@ class XlsxChartExtractor extends XlsxBaseExtractor {
                     default: "ffffff",
                 }).asString(),
                 legendPosition: DRAWING_LEGEND_POSITION_CONVERSION_MAP[this.extractChildAttr(rootChartElement, "c:legendPos", "val", {
-                    default: "b",
+                    default: "none",
                 }).asString()],
                 stacked: barChartGrouping === "stacked",
                 fontColor: "000000",
@@ -45880,7 +45860,7 @@ class XlsxChartExtractor extends XlsxBaseExtractor {
                 default: "ffffff",
             }).asString(),
             legendPosition: DRAWING_LEGEND_POSITION_CONVERSION_MAP[this.extractChildAttr(chartElement, "c:legendPos", "val", {
-                default: "b",
+                default: "none",
             }).asString()],
             stacked: barChartGrouping === "stacked",
             fontColor: "000000",
@@ -64178,7 +64158,9 @@ css /* scss */ `
     margin-top: -1px;
     border: 1px solid;
 
-    .o-composer:empty:not(:focus):not(.active)::before {
+    /* In readonly we always show the fx icon if the composer is empty, not matter the focus */
+    .o-composer:empty:not(:focus):not(.active)::before,
+    &.o-topbar-composer-readonly .o-composer:empty::before {
       content: url("data:image/svg+xml,${encodeURIComponent(FX_SVG)}");
       position: relative;
       top: 20%;
@@ -67422,10 +67404,14 @@ function addIconSetRule(cf, rule) {
             continue;
         }
         const cfValueObjectNodes = cfValueObject.map((attrs) => escapeXml /*xml*/ `<cfvo ${formatAttributes(attrs)} />`);
+        const iconSetAttrs = [["iconSet", getIconSet(rule.icons)]];
+        if (isIconSetReversed(rule.icons)) {
+            iconSetAttrs.push(["reverse", "1"]);
+        }
         conditionalFormats.push(escapeXml /*xml*/ `
       <conditionalFormatting sqref="${range}">
         <cfRule ${formatAttributes(ruleAttributes)}>
-          <iconSet iconSet="${getIconSet(rule.icons)}">
+          <iconSet ${formatAttributes(iconSetAttrs)}>
             ${joinXmlNodes(cfValueObjectNodes)}
           </iconSet>
         </cfRule>
@@ -67443,9 +67429,21 @@ function commonCfAttributes(cf) {
         ["stopIfTrue", cf.stopIfTrue ? 1 : 0],
     ];
 }
+function isIconSetReversed(iconSet) {
+    const defaultIconSet = ICON_SETS[detectIconsType(iconSet)];
+    return iconSet.upper === defaultIconSet.bad && iconSet.lower === defaultIconSet.good;
+}
 function getIconSet(iconSet) {
-    return XLSX_ICONSET_MAP[Object.keys(XLSX_ICONSET_MAP).find((key) => iconSet.upper.toLowerCase().startsWith(key)) ||
-        "dots"];
+    return XLSX_ICONSET_MAP[detectIconsType(iconSet)];
+}
+/**
+ * Partial detection based on "upper" point only.
+ * We support any arbitrary icon in the set, while excel doesn't allow
+ * mixing icons from different types.
+ */
+function detectIconsType(iconSet) {
+    const type = Object.keys(ICON_SETS).find((type) => Object.values(ICON_SETS[type]).includes(iconSet.upper)) || "dots";
+    return type;
 }
 function thresholdAttributes(threshold, position) {
     const type = getExcelThresholdType(threshold.type, position);
@@ -69184,6 +69182,6 @@ const constants = {
 export { AbstractCellClipboardHandler, AbstractChart, AbstractFigureClipboardHandler, CellErrorType, CommandResult, CorePlugin, DispatchResult, EvaluationError, Model, PivotRuntimeDefinition, Registry, Revision, SPREADSHEET_DIMENSIONS, Spreadsheet, SpreadsheetPivotTable, UIPlugin, __info__, addFunction, addRenderingLayer, astToFormula, compile, compileTokens, components, constants, convertAstNodes, coreTypes, findCellInNewZone, functionCache, helpers, hooks, invalidateCFEvaluationCommands, invalidateDependenciesCommands, invalidateEvaluationCommands, iterateAstNodes, links, load, parse, parseTokens, readonlyAllowedCommands, registries, setDefaultSheetViewSize, setTranslationMethod, stores, tokenColors, tokenize };
 
 
-__info__.version = "17.4.27";
-__info__.date = "2025-03-19T08:27:27.414Z";
-__info__.hash = "a87c1da";
+__info__.version = "17.4.28";
+__info__.date = "2025-03-26T12:48:33.387Z";
+__info__.hash = "a81a8f4";

package/dist/o-spreadsheet.iife.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.27
- * @date 2025-03-19T08:27:27.414Z
- * @hash a87c1da
+ * @version 17.4.28
+ * @date 2025-03-26T12:48:33.387Z
+ * @hash a81a8f4
  */
 
 (function (exports, owl) {
@@ -1008,7 +1008,10 @@
       }
       else if (stringVals.length === 4) {
           const alpha = parseFloat(stringVals.pop() || "1");
-          alphaHex = Math.round((alpha || 1) * 255);
+          if (isNaN(alpha)) {
+              throw new Error("invalid alpha value");
+          }
+          alphaHex = Math.round(alpha * 255);
       }
       const vals = stringVals.map((val) => parseInt(val, 10));
       if (alphaHex !== 255) {
@@ -5964,7 +5967,7 @@
    */
   function canonicalizeNumberContent(content, locale) {
       return content.startsWith("=")
-          ? canonicalizeFormula$1(content, locale)
+          ? canonicalizeFormula(content, locale)
           : canonicalizeNumberLiteral(content, locale);
   }
   /**
@@ -5979,7 +5982,7 @@
    */
   function canonicalizeContent(content, locale) {
       return content.startsWith("=")
-          ? canonicalizeFormula$1(content, locale)
+          ? canonicalizeFormula(content, locale)
           : canonicalizeLiteral(content, locale);
   }
   /**
@@ -5995,15 +5998,21 @@
           ? localizeFormula(content, locale)
           : localizeLiteral(content, locale);
   }
+  /** Change a number string to its canonical form (en_US locale) */
+  function canonicalizeNumberValue(content, locale) {
+      return content.startsWith("=")
+          ? canonicalizeFormula(content, locale)
+          : canonicalizeNumberLiteral(content, locale);
+  }
   /** Change a formula to its canonical form (en_US locale) */
-  function canonicalizeFormula$1(formula, locale) {
-      return _localizeFormula$1(formula, locale, DEFAULT_LOCALE);
+  function canonicalizeFormula(formula, locale) {
+      return _localizeFormula(formula, locale, DEFAULT_LOCALE);
   }
   /** Change a formula from the canonical form to the given locale */
   function localizeFormula(formula, locale) {
-      return _localizeFormula$1(formula, DEFAULT_LOCALE, locale);
+      return _localizeFormula(formula, DEFAULT_LOCALE, locale);
   }
-  function _localizeFormula$1(formula, fromLocale, toLocale) {
+  function _localizeFormula(formula, fromLocale, toLocale) {
       if (fromLocale.formulaArgSeparator === toLocale.formulaArgSeparator &&
           fromLocale.decimalSeparator === toLocale.decimalSeparator) {
           return formula;
@@ -6158,37 +6167,6 @@
       return locale.dateFormat + " " + locale.timeFormat;
   }
 
-  /** Change a number string to its canonical form (en_US locale) */
-  function canonicalizeNumberValue(content, locale) {
-      return content.startsWith("=")
-          ? canonicalizeFormula(content, locale)
-          : canonicalizeNumberLiteral(content, locale);
-  }
-  /** Change a formula to its canonical form (en_US locale) */
-  function canonicalizeFormula(formula, locale) {
-      return _localizeFormula(formula, locale, DEFAULT_LOCALE);
-  }
-  function _localizeFormula(formula, fromLocale, toLocale) {
-      if (fromLocale.formulaArgSeparator === toLocale.formulaArgSeparator &&
-          fromLocale.decimalSeparator === toLocale.decimalSeparator) {
-          return formula;
-      }
-      const tokens = tokenize(formula, fromLocale);
-      let localizedFormula = "";
-      for (const token of tokens) {
-          if (token.type === "NUMBER") {
-              localizedFormula += token.value.replace(fromLocale.decimalSeparator, toLocale.decimalSeparator);
-          }
-          else if (token.type === "ARG_SEPARATOR") {
-              localizedFormula += toLocale.formulaArgSeparator;
-          }
-          else {
-              localizedFormula += token.value;
-          }
-      }
-      return localizedFormula;
-  }
-
   function boolAnd(args) {
       let foundBoolean = false;
       let acc = true;
@@ -12025,7 +12003,8 @@ stores.inject(MyMetaStore, storeInstance);
           }
       }
       else if (dataSets.length === 1) {
-          for (let i = 0; i < getData(getters, dataSets[0]).length; i++) {
+          const dataLength = getData(getters, dataSets[0]).length;
+          for (let i = 0; i < dataLength; i++) {
               labels.formattedValues.push("");
               labels.values.push("");
           }
@@ -43180,9 +43159,9 @@ stores.inject(MyMetaStore, storeInstance);
   /** In XLSX color format (no #)  */
   const AUTO_COLOR = "000000";
   const XLSX_ICONSET_MAP = {
-      arrow: "3Arrows",
+      arrows: "3Arrows",
       smiley: "3Symbols",
-      dot: "3TrafficLights1",
+      dots: "3TrafficLights1",
   };
   const NAMESPACE = {
       styleSheet: "http://schemas.openxmlformats.org/spreadsheetml/2006/main",
@@ -43598,6 +43577,7 @@ stores.inject(MyMetaStore, storeInstance);
   };
   /** Map between legend position in XLSX file and human readable position  */
   const DRAWING_LEGEND_POSITION_CONVERSION_MAP = {
+      none: "none",
       b: "bottom",
       t: "top",
       l: "left",
@@ -45847,7 +45827,7 @@ stores.inject(MyMetaStore, storeInstance);
                       default: "ffffff",
                   }).asString(),
                   legendPosition: DRAWING_LEGEND_POSITION_CONVERSION_MAP[this.extractChildAttr(rootChartElement, "c:legendPos", "val", {
-                      default: "b",
+                      default: "none",
                   }).asString()],
                   stacked: barChartGrouping === "stacked",
                   fontColor: "000000",
@@ -45881,7 +45861,7 @@ stores.inject(MyMetaStore, storeInstance);
                   default: "ffffff",
               }).asString(),
               legendPosition: DRAWING_LEGEND_POSITION_CONVERSION_MAP[this.extractChildAttr(chartElement, "c:legendPos", "val", {
-                  default: "b",
+                  default: "none",
               }).asString()],
               stacked: barChartGrouping === "stacked",
               fontColor: "000000",
@@ -64179,7 +64159,9 @@ stores.inject(MyMetaStore, storeInstance);
     margin-top: -1px;
     border: 1px solid;
 
-    .o-composer:empty:not(:focus):not(.active)::before {
+    /* In readonly we always show the fx icon if the composer is empty, not matter the focus */
+    .o-composer:empty:not(:focus):not(.active)::before,
+    &.o-topbar-composer-readonly .o-composer:empty::before {
       content: url("data:image/svg+xml,${encodeURIComponent(FX_SVG)}");
       position: relative;
       top: 20%;
@@ -67423,10 +67405,14 @@ stores.inject(MyMetaStore, storeInstance);
               continue;
           }
           const cfValueObjectNodes = cfValueObject.map((attrs) => escapeXml /*xml*/ `<cfvo ${formatAttributes(attrs)} />`);
+          const iconSetAttrs = [["iconSet", getIconSet(rule.icons)]];
+          if (isIconSetReversed(rule.icons)) {
+              iconSetAttrs.push(["reverse", "1"]);
+          }
           conditionalFormats.push(escapeXml /*xml*/ `
       <conditionalFormatting sqref="${range}">
         <cfRule ${formatAttributes(ruleAttributes)}>
-          <iconSet iconSet="${getIconSet(rule.icons)}">
+          <iconSet ${formatAttributes(iconSetAttrs)}>
             ${joinXmlNodes(cfValueObjectNodes)}
           </iconSet>
         </cfRule>
@@ -67444,9 +67430,21 @@ stores.inject(MyMetaStore, storeInstance);
           ["stopIfTrue", cf.stopIfTrue ? 1 : 0],
       ];
   }
+  function isIconSetReversed(iconSet) {
+      const defaultIconSet = ICON_SETS[detectIconsType(iconSet)];
+      return iconSet.upper === defaultIconSet.bad && iconSet.lower === defaultIconSet.good;
+  }
   function getIconSet(iconSet) {
-      return XLSX_ICONSET_MAP[Object.keys(XLSX_ICONSET_MAP).find((key) => iconSet.upper.toLowerCase().startsWith(key)) ||
-          "dots"];
+      return XLSX_ICONSET_MAP[detectIconsType(iconSet)];
+  }
+  /**
+   * Partial detection based on "upper" point only.
+   * We support any arbitrary icon in the set, while excel doesn't allow
+   * mixing icons from different types.
+   */
+  function detectIconsType(iconSet) {
+      const type = Object.keys(ICON_SETS).find((type) => Object.values(ICON_SETS[type]).includes(iconSet.upper)) || "dots";
+      return type;
   }
   function thresholdAttributes(threshold, position) {
       const type = getExcelThresholdType(threshold.type, position);
@@ -69228,9 +69226,9 @@ stores.inject(MyMetaStore, storeInstance);
   exports.tokenize = tokenize;
 
 
-  __info__.version = "17.4.27";
-  __info__.date = "2025-03-19T08:27:27.414Z";
-  __info__.hash = "a87c1da";
+  __info__.version = "17.4.28";
+  __info__.date = "2025-03-26T12:48:33.387Z";
+  __info__.hash = "a81a8f4";
 
 
 })(this.o_spreadsheet = this.o_spreadsheet || {}, owl);