@useinsider/guido 3.20.1-beta.8a05319 → 3.20.1-beta.991508

package/dist/composables/useLegacyStrategyConversion.js

@@ -1,34 +1,34 @@
-import { useRecommendation as C } from "./useRecommendation.js";
+import { useRecommendation as S } from "./useRecommendation.js";
 import { useToaster as I } from "./useToaster.js";
 import { useTranslations as v } from "./useTranslations.js";
 import { ToasterTypeOptions as g } from "../enums/toaster.js";
 import "../extensions/Blocks/Recommendation/constants/selectors.js";
 import { CURRENT_CONFIG_VERSION as L } from "../extensions/Blocks/Recommendation/constants/defaultConfig.js";
-import { useRecommendationExtensionStore as b } from "../extensions/Blocks/Recommendation/store/recommendation.js";
-import { resolveMigrationTarget as k, buildLegacyMigrationPayload as R } from "../extensions/Blocks/Recommendation/utils/legacyStrategyMigration.js";
+import { useRecommendationExtensionStore as R } from "../extensions/Blocks/Recommendation/store/recommendation.js";
+import { resolveMigrationTarget as b, buildLegacyMigrationPayload as k } from "../extensions/Blocks/Recommendation/utils/legacyStrategyMigration.js";
 import { commitBlockChangeOutsidePanel as T } from "../extensions/Blocks/Recommendation/utils/nodeConfigWriter.js";
 import { useConfigStore as h } from "../stores/config.js";
 const F = () => {
-  const e = b(), r = h(), { buildCampaignUrl: m } = C(), { showToaster: c } = I(), a = v();
+  const e = R(), o = h(), { buildCampaignUrl: m } = S(), { showToaster: c } = I(), a = v();
   return { convertLegacyStrategy: async () => {
     const t = e.currentRecommendationId;
     if (t === null || !e.isLegacyStrategyBlock || e.legacyConversionLoading)
       return "";
-    const { strategy: d, productIds: y, size: l, strategyId: p, nodeConfigVersion: u } = e.recommendationConfigs, f = { strategyId: p, size: l, nodeConfigVersion: u }, s = k({
-      campaignId: r.campaignId,
-      templateLibraryId: r.templateLibraryId
-    }), S = R({
+    const { strategy: d, productIds: y, size: l, strategyId: p, nodeConfigVersion: u } = e.recommendationConfigs, f = { strategyId: p, size: l, nodeConfigVersion: u }, s = b({
+      campaignId: o.campaignId,
+      templateLibraryId: o.templateLibraryId
+    }), C = k({
       blockId: t,
       url: m(String(t)),
-      campaignId: r.campaignId,
-      variationId: r.variationId,
+      campaignId: o.campaignId,
+      variationId: o.variationId,
       strategy: d,
       productIds: y,
       target: s
     });
-    let o;
+    let r;
     try {
-      o = await e.convertLegacyStrategy(S, s);
+      r = await e.convertLegacyStrategy(C, s);
     } catch (i) {
       return console.error("[useLegacyStrategyConversion] migration failed", i), c({
         type: g.Error,
@@ -40,7 +40,7 @@ const F = () => {
       }), "";
     }
     try {
-      await e.fetchStrategyProductsForBlock(t);
+      await e.fetchRecommendationProductsForBlock(t);
     } catch (i) {
       console.error("[useLegacyStrategyConversion] product refetch failed", i);
     }
@@ -48,16 +48,16 @@ const F = () => {
     return n ? T(
       t,
       {
-        strategyId: o,
+        strategyId: r,
         size: n.recommendationConfigs.size,
         configVersion: L
       },
       n.recommendationProducts,
       "Converted legacy recommendation strategy"
-    ) ? (e.currentRecommendationId === t && e.openLegacyStrategyUpdatedModal(), o) : (console.error("[useLegacyStrategyConversion] node write failed", t), e.revertLegacyConversion(t, f), c({
+    ) ? (e.currentRecommendationId === t && e.openLegacyStrategyUpdatedModal(), r) : (console.error("[useLegacyStrategyConversion] node write failed", t), e.revertLegacyConversion(t, f), c({
       type: g.Error,
       message: a("email-editor.legacy-strategy-update-failed")
-    }), "") : o;
+    }), "") : r;
   } };
 };
 export {

package/dist/config/compiler/itemsCompilerRules.js

@@ -1,6 +1,6 @@
 import { pairProductVariables as o } from "../../utils/pairProductVariables.js";
-import { normalizeItemsBlockWidths as r, wrapHiddenPriceTdsForOutlook as e, MSO_ITEMS_HIDE_START as t, MSO_ITEMS_HIDE_END as s } from "./utils/itemsCompilerUtils.js";
-const d = [
+import { normalizeItemsBlockWidths as r, wrapHiddenPriceTdsForOutlook as t, lockHorizontalItemImageWidths as e, MSO_ITEMS_HIDE_START as s, MSO_ITEMS_HIDE_END as p } from "./utils/itemsCompilerUtils.js";
+const c = [
   {
     id: "normalize-items-block-widths",
     // Runs before the other items rules so they see non-zero widths; not AMP-safe.
@@ -25,20 +25,27 @@ const d = [
     // with safe placeholder comments and collapse them inline for web-engine Outlook.
     description: "Mark hidden Items price tds for Outlook conditional hiding",
     type: "custom",
-    processor: e,
+    processor: t,
     priority: 65
     // After pair-product-variables (60), before outlook rules (70+)
   },
+  {
+    id: "lock-horizontal-item-image-widths",
+    description: "Pin horizontal Items thumbnails to their width so narrow rows cannot collapse them",
+    type: "custom",
+    processor: e,
+    priority: 66
+  },
   {
     id: "swap-items-mso-markers",
     // Injecting the delicate `<!-- -->` ghost token LAST (after reco minify P99
     // and reco swap P100) keeps it out of every whitespace/comment-mangling pass.
     description: "Swap Items MSO markers into [if !mso] conditional comments",
     type: "custom",
-    processor: (i) => i.replaceAll(`<!--${t}-->`, "<!--[if !mso]><!-- -->").replaceAll(`<!--${s}-->`, "<!--<![endif]-->"),
+    processor: (i) => i.replaceAll(`<!--${s}-->`, "<!--[if !mso]><!-- -->").replaceAll(`<!--${p}-->`, "<!--<![endif]-->"),
     priority: 101
   }
 ];
 export {
-  d as itemsCompilerRules
+  c as itemsCompilerRules
 };

package/dist/config/compiler/utils/itemsCompilerUtils.js

@@ -1,55 +1,75 @@
-import { PRODUCT_WRAPPER_SELECTOR as A, hasZeroWidthDeclaration as E, isZeroWidthValue as u, withWidthPin as h, PRODUCT_TD_SELECTOR as S, withoutZeroWidth as _ } from "../../../extensions/Blocks/Items/constants/cardWidth.js";
-import { protectLiquid as m, restoreLiquid as y } from "../../../utils/liquidProtection.js";
-const f = "MSO_ITEMS_HIDE_START", D = "MSO_ITEMS_HIDE_END", T = "mso-hide:all; max-height:0; overflow:hidden; font-size:0; line-height:0;", M = ".ins-product-cart td.horizontal-price, .ins-product-cart td.vertical-price";
-function g(e) {
-  if (!e.includes("vertical-price") || !e.includes("ins-product-cart"))
-    return e;
-  const n = e.replaceAll("{%", "<!--{%").replaceAll("%}", "%}-->"), r = new DOMParser().parseFromString(n, "text/html");
-  let s = !1;
-  if (r.querySelectorAll(M).forEach((t) => {
-    var d, a;
-    const o = t.getAttribute("style") || "";
-    if (!/display\s*:\s*none/i.test(o))
+import { PRODUCT_WRAPPER_SELECTOR as h, hasZeroWidthDeclaration as E, isZeroWidthValue as f, withWidthPin as T, PRODUCT_TD_SELECTOR as _, withoutZeroWidth as y } from "../../../extensions/Blocks/Items/constants/cardWidth.js";
+import { DefaultConfigValues as D } from "../../../extensions/Blocks/Items/enums/settingsEnums.js";
+import { protectLiquid as p, restoreLiquid as A } from "../../../utils/liquidProtection.js";
+const m = "MSO_ITEMS_HIDE_START", S = "MSO_ITEMS_HIDE_END", M = "mso-hide:all; max-height:0; overflow:hidden; font-size:0; line-height:0;", I = ".ins-product-cart td.horizontal-price, .ins-product-cart td.vertical-price", O = ".ins-product-td td.product-image-class img[width]", g = Number(D.productImageWidth);
+function C(t) {
+  if (!t.includes("vertical-price") || !t.includes("ins-product-cart"))
+    return t;
+  const s = t.replaceAll("{%", "<!--{%").replaceAll("%}", "%}-->"), o = new DOMParser().parseFromString(s, "text/html");
+  let n = !1;
+  if (o.querySelectorAll(I).forEach((e) => {
+    var d, u;
+    const r = e.getAttribute("style") || "";
+    if (!/display\s*:\s*none/i.test(r))
       return;
-    const l = t.previousSibling;
-    if (((l == null ? void 0 : l.nodeType) !== Node.COMMENT_NODE || l.data !== f) && ((d = t.parentNode) == null || d.insertBefore(r.createComment(f), t), (a = t.parentNode) == null || a.insertBefore(r.createComment(D), t.nextSibling)), !/mso-hide/i.test(o)) {
-      const p = o.trim().endsWith(";") ? " " : "; ";
-      t.setAttribute("style", `${o}${p}${T}`);
+    const l = e.previousSibling;
+    if (((l == null ? void 0 : l.nodeType) !== Node.COMMENT_NODE || l.data !== m) && ((d = e.parentNode) == null || d.insertBefore(o.createComment(m), e), (u = e.parentNode) == null || u.insertBefore(o.createComment(S), e.nextSibling)), !/mso-hide/i.test(r)) {
+      const a = r.trim().endsWith(";") ? " " : "; ";
+      e.setAttribute("style", `${r}${a}${M}`);
     }
-    s = !0;
-  }), !s)
-    return e;
-  const c = n.match(/<!DOCTYPE[^>]*>/i);
+    n = !0;
+  }), !n)
+    return t;
+  const c = s.match(/<!DOCTYPE[^>]*>/i);
   return ((c ? `${c[0]}
-` : "") + r.documentElement.outerHTML).replaceAll("<!--{%", "{%").replaceAll("%}-->", "%}").replaceAll("&lt;!--{%", "{%").replaceAll("%}--&gt;", "%}");
+` : "") + o.documentElement.outerHTML).replaceAll("<!--{%", "{%").replaceAll("%}-->", "%}").replaceAll("&lt;!--{%", "{%").replaceAll("%}--&gt;", "%}");
 }
-function I(e) {
-  if (!e.includes("ins-product-wrapper") && !e.includes("ins-product-td"))
-    return e;
-  const n = new DOMParser().parseFromString(m(e), "text/html");
-  let r = !1;
-  if (n.querySelectorAll(A).forEach((i) => {
-    const t = i.getAttribute("style") || "";
-    if (!E(t) && !u(i.getAttribute("width")))
+function N(t) {
+  if (!t.includes("ins-product-wrapper") && !t.includes("ins-product-td"))
+    return t;
+  const s = new DOMParser().parseFromString(p(t), "text/html");
+  let o = !1;
+  if (s.querySelectorAll(h).forEach((i) => {
+    const e = i.getAttribute("style") || "";
+    if (!E(e) && !f(i.getAttribute("width")))
       return;
-    const o = h(t);
-    o !== t && (i.setAttribute("style", o), r = !0), u(i.getAttribute("width")) && (i.setAttribute("width", "100%"), r = !0);
-  }), n.querySelectorAll(S).forEach((i) => {
-    const t = i.getAttribute("style") || "";
-    if (E(t)) {
-      const o = _(t);
-      o ? i.setAttribute("style", o) : i.removeAttribute("style"), r = !0;
+    const r = T(e);
+    r !== e && (i.setAttribute("style", r), o = !0), f(i.getAttribute("width")) && (i.setAttribute("width", "100%"), o = !0);
+  }), s.querySelectorAll(_).forEach((i) => {
+    const e = i.getAttribute("style") || "";
+    if (E(e)) {
+      const r = y(e);
+      r ? i.setAttribute("style", r) : i.removeAttribute("style"), o = !0;
     }
-    u(i.getAttribute("width")) && (i.removeAttribute("width"), r = !0);
-  }), !r)
-    return e;
-  const s = e.match(/<!DOCTYPE[^>]*>/i), c = s ? `${s[0]}
+    f(i.getAttribute("width")) && (i.removeAttribute("width"), o = !0);
+  }), !o)
+    return t;
+  const n = t.match(/<!DOCTYPE[^>]*>/i), c = n ? `${n[0]}
 ` : "";
-  return y(c + n.documentElement.outerHTML);
+  return A(c + s.documentElement.outerHTML);
+}
+function L(t) {
+  if (!t.includes("product-image-class"))
+    return t;
+  const s = new DOMParser().parseFromString(p(t), "text/html");
+  let o = !1;
+  if (s.querySelectorAll(O).forEach((i) => {
+    var u, a;
+    const e = ((u = i.getAttribute("width")) == null ? void 0 : u.trim()) ?? "", r = ((a = i.getAttribute("style")) == null ? void 0 : a.trim()) ?? "";
+    if (!/^\d+$/.test(e) || /(^|;)\s*min-width\s*:/i.test(r))
+      return;
+    const l = r && !r.endsWith(";") ? ";" : "", d = Math.min(Number(e), g);
+    i.setAttribute("style", `${r}${l}min-width:${d}px;max-width:${e}px`), o = !0;
+  }), !o)
+    return t;
+  const n = t.match(/<!DOCTYPE[^>]*>/i), c = n ? `${n[0]}
+` : "";
+  return A(c + s.documentElement.outerHTML);
 }
 export {
-  D as MSO_ITEMS_HIDE_END,
-  f as MSO_ITEMS_HIDE_START,
-  I as normalizeItemsBlockWidths,
-  g as wrapHiddenPriceTdsForOutlook
+  S as MSO_ITEMS_HIDE_END,
+  m as MSO_ITEMS_HIDE_START,
+  L as lockHorizontalItemImageWidths,
+  N as normalizeItemsBlockWidths,
+  C as wrapHiddenPriceTdsForOutlook
 };