@useinsider/guido 3.21.0-beta.7e166b5 → 3.21.0-beta.ab76470

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

@@ -162,7 +162,7 @@ function G(t, r) {
 function V(t) {
   const r = t.getAttribute("style") || "", e = y(r, "width", 600), o = y(r, "padding", 0) * 2, c = Math.max(0, e - o);
   c !== 0 && t.querySelectorAll("img.adapt-img").forEach((i) => {
-    if (i.hasAttribute("width"))
+    if (i.hasAttribute("width") || i.hasAttribute("height"))
       return;
     const n = G(i, t);
     if (!n)

package/dist/extensions/Blocks/Recommendation/block.js

@@ -1,46 +1,43 @@
-var A = Object.defineProperty;
-var b = (c, l, e) => l in c ? A(c, l, { enumerable: !0, configurable: !0, writable: !0, value: e }) : c[l] = e;
-var m = (c, l, e) => b(c, typeof l != "symbol" ? l + "" : l, e);
-import { useConfig as d } from "../../../composables/useConfig.js";
-import { BlockId as E } from "../../../enums/block.js";
-import { getMigrationBannerHtml as C } from "../../../utils/migrationBannerHtml.js";
-import { Block as S, BlockCompositionType as N, ModificationDescription as p } from "../../../node_modules/@stripoinc/ui-editor-extensions/dist/esm/index.js";
-import { CURRENT_CONFIG_VERSION as v, PRE_RRS_CONFIG_VERSION as w } from "./constants/defaultConfig.js";
-import { RECOMMENDATION_ID_ATTR as h } from "./constants/selectors.js";
+var B = Object.defineProperty;
+var b = (a, l, t) => l in a ? B(a, l, { enumerable: !0, configurable: !0, writable: !0, value: t }) : a[l] = t;
+var d = (a, l, t) => b(a, typeof l != "symbol" ? l + "" : l, t);
+import { useConfig as g } from "../../../composables/useConfig.js";
+import { BlockId as C } from "../../../enums/block.js";
+import { getMigrationBannerHtml as A } from "../../../utils/migrationBannerHtml.js";
+import { Block as E, BlockCompositionType as S, ModificationDescription as p } from "../../../node_modules/@stripoinc/ui-editor-extensions/dist/esm/index.js";
+import { CURRENT_CONFIG_VERSION as N, PRE_RRS_CONFIG_VERSION as w } from "./constants/defaultConfig.js";
+import { RECOMMENDATION_ID_ATTR as m } from "./constants/selectors.js";
 import { regenerateMobileProductRows as M } from "./controls/main/utils.js";
 import { ensureMobileCssRulesExist as I, setMobileLayoutOptOut as R, hasMobileLayoutOptOut as O } from "./controls/mobileLayout/cssRules.js";
-import { RecommendationConfigService as o } from "./services/configService.js";
-import { useRecommendationExtensionStore as f } from "./store/recommendation.js";
-import { getDefaultTemplate as T } from "./templates/grid/template.js";
-import { useRecommendationBlockWarning as x } from "./useRecommendationBlockWarning.js";
-import { registerNodeConfigWriter as k } from "./utils/nodeConfigWriter.js";
-import { refreshArrivedStrategyBlocks as $, refreshDuplicatedStrategyBlocks as H } from "./utils/strategyCommit.js";
-const y = E.Recommendation, u = "recommendation-block-v2", B = "hide-price";
-let D = !1;
-class Z extends S {
+import { RecommendationConfigService as s } from "./services/configService.js";
+import { useRecommendationExtensionStore as h } from "./store/recommendation.js";
+import { getDefaultTemplate as x } from "./templates/grid/template.js";
+import { useRecommendationBlockWarning as T } from "./useRecommendationBlockWarning.js";
+import { registerNodeConfigWriter as H } from "./utils/nodeConfigWriter.js";
+import { refreshDuplicatedStrategyBlocks as $ } from "./utils/strategyCommit.js";
+const k = C.Recommendation, u = "recommendation-block-v2", D = "hide-price";
+let y = !1;
+class Y extends E {
   constructor() {
     super();
     /**
      * Stores the ID generated in getTemplate() so onCreated() can reuse it.
      * This avoids generating a new (different) ID in onCreated().
      */
-    m(this, "_pendingBlockId", null);
+    d(this, "_pendingBlockId", null);
     /** Ids already reset under their new name, so the deferred pass skips their stale snapshot. */
-    m(this, "_resetSourceIds", /* @__PURE__ */ new Set());
+    d(this, "_resetSourceIds", /* @__PURE__ */ new Set());
     /** Ids handed to duplicates the document root does not show yet: a copied module renames all its blocks at once. */
-    m(this, "_unsettledDuplicateIds", /* @__PURE__ */ new Set());
-    m(this, "_arrivedNodes", []);
-    /** Ids whose arrival or duplicate refresh is in flight, so a second refresh of the same block does not race it. */
-    m(this, "_refreshingIds", /* @__PURE__ */ new Set());
+    d(this, "_unsettledDuplicateIds", /* @__PURE__ */ new Set());
   }
   getId() {
-    return y;
+    return k;
   }
   getIcon() {
     return "recommendation-icon";
   }
   getBlockCompositionType() {
-    return N.CONTAINER;
+    return S.CONTAINER;
   }
   getName() {
     return this.api.translate("Recommendation Block");
@@ -51,8 +48,8 @@ class Z extends S {
     );
   }
   getSettingsPanelTitleHtml() {
-    return C(
-      y,
+    return A(
+      k,
       this.api.translate("Recommendation Block"),
       this.api.translate("This block is switched from the Old Version to the New Version. We recommend you check the Recommendation block and test your message to ensure it works properly.")
     );
@@ -74,8 +71,8 @@ class Z extends S {
    * attribute and overwrites dynamically added classes via setAttribute.
    */
   getTemplate() {
-    const e = this._generateNextId();
-    return this._pendingBlockId = e, T(e);
+    const t = this._generateNextId();
+    return this._pendingBlockId = t, x(t);
   }
   /**
    * Called when a new block is dropped into the template
@@ -84,58 +81,42 @@ class Z extends S {
    * and assigns the recommendation-id attribute + initializes configuration.
    * @param node - The newly created block node
    */
-  onCreated(e) {
-    const t = this._getRecommendationId(e);
-    if (t !== null && t > 0) {
-      if (this._detectDuplicate(t)) {
-        this._handleDuplicate(e, t), this._scheduleArrivalReset(e);
+  onCreated(t) {
+    const e = this._getRecommendationId(t);
+    if (e !== null && e > 0) {
+      if (this._detectDuplicate(e)) {
+        this._handleDuplicate(t, e), this._scheduleArrivalReset(t);
         return;
       }
-      const g = this._pendingBlockId === null;
-      setTimeout(() => {
-        if (this._getBlockElement(e)) {
-          if (g && this._detectDuplicate(t)) {
-            this._handleDuplicate(e, t);
-            return;
-          }
-          this._queueArrivalRefresh(e);
-        }
-      }, 0), this._scheduleArrivalReset(e);
+      this._pendingBlockId === null && setTimeout(() => {
+        this._getBlockElement(t) && this._detectDuplicate(e) && this._handleDuplicate(t, e);
+      }, 0), this._scheduleArrivalReset(t);
       return;
     }
     const i = this._pendingBlockId ?? this._generateNextId();
-    this._pendingBlockId = null, this._assignRecommendationId(e, i);
-    const r = d().isFeatureEnabled("reusableRecommendationStrategy"), s = r && !o.hasConfig(e), { config: n, wasFreshDrop: a } = o.initializeConfig(
+    this._pendingBlockId = null, this._assignRecommendationId(t, i);
+    const n = g().isFeatureEnabled("reusableRecommendationStrategy"), o = n && !s.hasConfig(t), { config: r, wasFreshDrop: c } = s.initializeConfig(
       this.api,
-      e,
+      t,
       {
         recommendationId: i,
-        configVersion: r ? v : w,
-        ...s ? { partnerName: d().partnerName.value } : {},
+        configVersion: n ? N : w,
+        ...o ? { partnerName: g().partnerName.value } : {},
         // An RRS block picks a strategy, not an algorithm, and its algorithm control is
         // hidden. Keeping the `mostPopular` default would bake a most-popular feed URL
         // at save for a block the user never configured.
-        ...s ? { strategy: "" } : {}
+        ...o ? { strategy: "" } : {}
       }
-    ), _ = f();
-    if (_.setCurrentBlock(i), this._stampHidePrice(e, n.priceHideIfSameAsDiscounted), a) {
+    ), f = h();
+    if (f.setCurrentBlock(i), this._stampHidePrice(t, r.priceHideIfSameAsDiscounted), c) {
       I(this.api);
-      const g = this._getBlockElement(e);
-      g && (R(this.api, g, !0), M({
-        currentNode: e,
+      const _ = this._getBlockElement(t);
+      _ && (R(this.api, _, !0), M({
+        currentNode: t,
         documentModifier: this.api.getDocumentModifier()
       }));
     }
-    _.patchCurrentBlockConfig({ language: n.language }, { triggerRefetch: !1 }), this._warnIfMultipleBlocks();
-  }
-  /** Refreshes every block of an opened template: Stripo restores a stored document without `onCreated`. */
-  onDocumentInit() {
-    try {
-      const e = this.api.getDocumentRoot();
-      e && "querySelectorAll" in e && e.querySelectorAll(`.${u}`).forEach((t) => this._queueArrivalRefresh(t));
-    } catch (e) {
-      console.error("[recommendation] refreshing an opened strategy failed", e);
-    }
+    f.patchCurrentBlockConfig({ language: r.language }, { triggerRefetch: !1 }), this._warnIfMultipleBlocks();
   }
   /**
    * Called when the document changes or template is loaded
@@ -144,24 +125,24 @@ class Z extends S {
    * and assigns recommendation-id to legacy blocks that lack one.
    * @param node - The block node that may need migration
    */
-  onDocumentChanged(e) {
-    if (!(!e || !("getNodeConfig" in e))) {
-      if (!this._getRecommendationId(e)) {
-        const t = this._generateNextId();
-        this._assignRecommendationId(e, t), o.hasConfig(e) && o.updateConfig(
+  onDocumentChanged(t) {
+    if (!(!t || !("getNodeConfig" in t))) {
+      if (!this._getRecommendationId(t)) {
+        const e = this._generateNextId();
+        this._assignRecommendationId(t, e), s.hasConfig(t) && s.updateConfig(
           this.api,
-          e,
-          { recommendationId: t },
+          t,
+          { recommendationId: e },
           "Assign recommendation ID to legacy block"
         );
       }
-      this._healLingeringDuplicate(e), o.needsMigration(e) && this._migrateFromLegacy(e), this._healHidePriceAttr(e);
+      this._healLingeringDuplicate(t), s.needsMigration(t) && this._migrateFromLegacy(t), this._healHidePriceAttr(t);
       try {
-        D || (I(this.api), D = !0);
-        const t = o.getConfig(e), i = this._getBlockElement(e);
+        y || (I(this.api), y = !0);
+        const e = s.getConfig(t), i = this._getBlockElement(t);
         if (i) {
-          const r = !t.mobileLayoutEnabled;
-          O(i) !== r && R(this.api, i, r);
+          const n = !e.mobileLayoutEnabled;
+          O(i) !== n && R(this.api, i, n);
         }
       } catch {
       }
@@ -173,9 +154,9 @@ class Z extends S {
    * Cleans up the per-block state from the Pinia store.
    * @param node - The block node being deleted
    */
-  onDelete(e) {
-    const t = this._getRecommendationId(e);
-    t && f().removeBlockState(t);
+  onDelete(t) {
+    const e = this._getRecommendationId(t);
+    e && h().removeBlockState(e);
   }
   /**
    * Warns (dark advisory toaster) when the design holds more than one live
@@ -191,7 +172,7 @@ class Z extends S {
    */
   _warnIfMultipleBlocks() {
     try {
-      x().warnIfMultipleBlocks(this.api.getDocumentRootHtmlNode());
+      T().warnIfMultipleBlocks(this.api.getDocumentRootHtmlNode());
     } catch {
     }
   }
@@ -200,21 +181,21 @@ class Z extends S {
    * in the document and finding the maximum existing ID + 1.
    */
   _generateNextId() {
-    let e = 0;
-    const t = /* @__PURE__ */ new Set();
+    let t = 0;
+    const e = /* @__PURE__ */ new Set();
     try {
       const i = this.api.getDocumentRoot();
-      i && "querySelectorAll" in i && i.querySelectorAll(`.${u}`).forEach((s) => {
-        if ("getAttribute" in s) {
-          const n = s.getAttribute(h), a = n ? parseInt(n) : 0;
-          t.add(a), a > e && (e = a);
+      i && "querySelectorAll" in i && i.querySelectorAll(`.${u}`).forEach((o) => {
+        if ("getAttribute" in o) {
+          const r = o.getAttribute(m), c = r ? parseInt(r) : 0;
+          e.add(c), c > t && (t = c);
         }
       });
     } catch {
     }
     return this._unsettledDuplicateIds.forEach((i) => {
-      t.has(i) && this._unsettledDuplicateIds.delete(i);
-    }), Math.max(e, ...this._unsettledDuplicateIds) + 1;
+      e.has(i) && this._unsettledDuplicateIds.delete(i);
+    }), Math.max(t, ...this._unsettledDuplicateIds) + 1;
   }
   /**
    * Drops the "already fetched" claims of a block that arrived holding a baked id
@@ -223,34 +204,12 @@ class Z extends S {
    * Deferred a tick: the collision handlers rename a sibling first, and either sibling can be
    * the renamed one depending on where the module was dropped (SD-149916).
    */
-  _scheduleArrivalReset(e) {
-    d().isFeatureEnabled("reusableRecommendationStrategy") && setTimeout(() => {
-      const t = this._getRecommendationId(e);
-      t !== null && t > 0 && !this._resetSourceIds.delete(t) && f().resetArrivedBlock(t);
+  _scheduleArrivalReset(t) {
+    g().isFeatureEnabled("reusableRecommendationStrategy") && setTimeout(() => {
+      const e = this._getRecommendationId(t);
+      e !== null && e > 0 && !this._resetSourceIds.delete(e) && h().resetArrivedBlock(e);
     }, 0);
   }
-  /**
-   * Re-reads the strategy of a block that arrived with saved cards, without waiting for a click.
-   * Blocks of one opened template or dropped module arrive in one tick and refresh together.
-   */
-  _queueArrivalRefresh(e) {
-    d().isFeatureEnabled("reusableRecommendationStrategy") && (this._arrivedNodes.push(e) > 1 || setTimeout(() => {
-      const t = this._arrivedNodes;
-      this._arrivedNodes = [];
-      try {
-        this._refreshArrivedBlocks(t);
-      } catch (i) {
-        console.error("[recommendation] refreshing an arrived strategy failed", i);
-      }
-    }, 0));
-  }
-  _refreshArrivedBlocks(e) {
-    const t = e.flatMap((i) => {
-      const r = this._getRecommendationId(i), s = this._getBlockElement(i);
-      return !r || !s || this._refreshingIds.has(r) ? [] : (this._refreshingIds.add(r), [{ id: r, config: o.getConfig(s) }]);
-    });
-    t.length && (k(this.api), $(t).catch((i) => console.error("[recommendation] refreshing an arrived strategy failed", i)).finally(() => t.forEach(({ id: i }) => this._refreshingIds.delete(i))));
-  }
   /**
    * Heals a duplicate that slipped past the create-time `_detectDuplicate`
    * scan (or that was persisted by an older template before SD-142352).
@@ -263,54 +222,49 @@ class Z extends S {
    * yanking the user's current selection mid-edit when the heal fires from
    * `onDocumentChanged`.
    */
-  _healLingeringDuplicate(e) {
-    const t = this._getRecommendationId(e);
-    if (!(t === null || t <= 0))
+  _healLingeringDuplicate(t) {
+    const e = this._getRecommendationId(t);
+    if (!(e === null || e <= 0))
       try {
         const i = this.api.getDocumentRoot();
         if (!i || !("querySelectorAll" in i))
           return;
-        const r = i.querySelectorAll(
-          `.${u}[${h}="${t}"]`
+        const n = i.querySelectorAll(
+          `.${u}[${m}="${e}"]`
         );
-        if (r.length <= 1)
+        if (n.length <= 1)
           return;
-        const s = this._getBlockElement(e);
-        if (!s)
+        const o = this._getBlockElement(t);
+        if (!o)
           return;
-        let n = -1;
-        for (let a = 0; a < r.length; a++)
-          if (r[a] === s) {
-            n = a;
+        let r = -1;
+        for (let c = 0; c < n.length; c++)
+          if (n[c] === o) {
+            r = c;
             break;
           }
-        if (n <= 0)
+        if (r <= 0)
           return;
-        this._reassignDuplicateId(e, t);
+        this._reassignDuplicateId(t, e);
       } catch {
       }
   }
   /** True when another block already holds the same id — Stripo's duplication signal. */
-  _detectDuplicate(e) {
+  _detectDuplicate(t) {
     try {
-      const t = this.api.getDocumentRoot();
-      if (t && "querySelectorAll" in t)
-        return t.querySelectorAll(
-          `.${u}[${h}="${e}"]`
+      const e = this.api.getDocumentRoot();
+      if (e && "querySelectorAll" in e)
+        return e.querySelectorAll(
+          `.${u}[${m}="${t}"]`
         ).length > 1;
     } catch {
     }
     return !1;
   }
   /** Assigns a fresh id to a duplicated block and syncs DOM, node config and store. */
-  _handleDuplicate(e, t) {
-    const i = this._reassignDuplicateId(e, t), r = f();
-    if (r.cloneBlockState(t, i), r.setCurrentBlock(i), d().isFeatureEnabled("reusableRecommendationStrategy")) {
-      this._resetSourceIds.add(t), k(this.api);
-      const s = [t, i].filter((n) => !this._refreshingIds.has(n));
-      s.forEach((n) => this._refreshingIds.add(n)), H(t, i, o.getConfig(e)).catch((n) => console.error("[recommendation] refreshing a duplicated strategy failed", n)).finally(() => s.forEach((n) => this._refreshingIds.delete(n)));
-    }
-    this._warnIfMultipleBlocks();
+  _handleDuplicate(t, e) {
+    const i = this._reassignDuplicateId(t, e), n = h();
+    n.cloneBlockState(e, i), n.setCurrentBlock(i), g().isFeatureEnabled("reusableRecommendationStrategy") && (this._resetSourceIds.add(e), H(this.api), $(e, i, s.getConfig(t)).catch((o) => console.error("[recommendation] refreshing a duplicated strategy failed", o))), this._warnIfMultipleBlocks();
   }
   /**
    * Core id-reassignment for a duplicated block: rewrites the DOM
@@ -323,13 +277,13 @@ class Z extends S {
    * effects so older bug-affected templates can self-repair without
    * disturbing the user's current selection).
    */
-  _reassignDuplicateId(e, t) {
-    const i = this._generateNextId(), r = this._getBlockElement(e);
-    return this._unsettledDuplicateIds.add(i), this._assignRecommendationId(e, i), r && this._reassignInstanceClass(r, t, i), o.hasConfig(e) && o.updateConfig(
+  _reassignDuplicateId(t, e) {
+    const i = this._generateNextId(), n = this._getBlockElement(t);
+    return this._unsettledDuplicateIds.add(i), this._assignRecommendationId(t, i), n && this._reassignInstanceClass(n, e, i), s.hasConfig(t) && s.updateConfig(
       this.api,
-      e,
+      t,
       { recommendationId: i },
-      `Reassign recommendation ID on duplicate (was ${t})`
+      `Reassign recommendation ID on duplicate (was ${e})`
     ), i;
   }
   /**
@@ -339,12 +293,12 @@ class Z extends S {
    * the source's class would both leak stylistic edits across blocks and
    * cause the mail render to apply the source block's config to the clone.
    */
-  _reassignInstanceClass(e, t, i) {
-    if (!("getAttribute" in e))
+  _reassignInstanceClass(t, e, i) {
+    if (!("getAttribute" in t))
       return;
-    const r = `ins-recommendation-v3-block-${t}`, s = `ins-recommendation-v3-block-${i}`, n = this.api.getDocumentModifier();
-    n.modifyHtml(e).removeClass(r).setClass(s), n.apply(new p(
-      `Reassign recommendation instance class ${r} -> ${s}`
+    const n = `ins-recommendation-v3-block-${e}`, o = `ins-recommendation-v3-block-${i}`, r = this.api.getDocumentModifier();
+    r.modifyHtml(t).removeClass(n).setClass(o), r.apply(new p(
+      `Reassign recommendation instance class ${n} -> ${o}`
     ));
   }
   /**
@@ -353,64 +307,64 @@ class Z extends S {
    * HTML by getTemplate(), since Stripo's Angular rendering overwrites dynamically
    * added classes via setAttribute.
    */
-  _assignRecommendationId(e, t) {
-    const i = this._getBlockElement(e);
+  _assignRecommendationId(t, e) {
+    const i = this._getBlockElement(t);
     if (!i)
       return;
-    const r = this.api.getDocumentModifier();
-    r.modifyHtml(i).setAttribute(h, t.toString()), r.apply(new p(`Assign recommendation ID ${t}`));
+    const n = this.api.getDocumentModifier();
+    n.modifyHtml(i).setAttribute(m, e.toString()), n.apply(new p(`Assign recommendation ID ${e}`));
   }
   /**
    * Stamps the `hide-price` attribute (read by email-service at send time) on
    * the block element from the config flag.
    */
-  _stampHidePrice(e, t) {
-    const i = this._getBlockElement(e);
-    i && this.api.getDocumentModifier().modifyHtml(i).setAttribute(B, String(t)).apply(new p("Stamp hide-price"));
+  _stampHidePrice(t, e) {
+    const i = this._getBlockElement(t);
+    i && this.api.getDocumentModifier().modifyHtml(i).setAttribute(D, String(e)).apply(new p("Stamp hide-price"));
   }
   /**
    * Stamps `hide-price` from config only when the block lacks the attribute —
    * heals templates saved before this feature without dirtying current ones.
    */
-  _healHidePriceAttr(e) {
-    const t = this._getBlockElement(e);
-    if (!t || !("getAttribute" in t) || t.getAttribute(B) !== null)
+  _healHidePriceAttr(t) {
+    const e = this._getBlockElement(t);
+    if (!e || !("getAttribute" in e) || e.getAttribute(D) !== null)
       return;
-    const { priceHideIfSameAsDiscounted: i } = o.getConfig(e);
-    this._stampHidePrice(e, i);
+    const { priceHideIfSameAsDiscounted: i } = s.getConfig(t);
+    this._stampHidePrice(t, i);
   }
   /**
    * Gets the recommendation-id from a block node
    */
-  _getRecommendationId(e) {
-    const t = this._getBlockElement(e);
-    if (!t || !("getAttribute" in t))
+  _getRecommendationId(t) {
+    const e = this._getBlockElement(t);
+    if (!e || !("getAttribute" in e))
       return null;
-    const i = t.getAttribute(h);
+    const i = e.getAttribute(m);
     if (!i)
       return null;
-    const r = parseInt(i);
-    return Number.isNaN(r) ? null : r;
+    const n = parseInt(i);
+    return Number.isNaN(n) ? null : n;
   }
   /**
    * Gets the block element (the element with BLOCK_CLASS)
    */
-  _getBlockElement(e) {
-    if ("getAttribute" in e) {
-      const t = e.getAttribute("class");
-      if (t && t.includes(u))
-        return e;
+  _getBlockElement(t) {
+    if ("getAttribute" in t) {
+      const e = t.getAttribute("class");
+      if (e && e.includes(u))
+        return t;
     }
-    return "querySelector" in e ? e.querySelector(`.${u}`) ?? null : null;
+    return "querySelector" in t ? t.querySelector(`.${u}`) ?? null : null;
   }
   /**
    * Migrate configuration from legacy format
    */
-  _migrateFromLegacy(e) {
-    o.migrateFromDataAttributes(this.api, e);
+  _migrateFromLegacy(t) {
+    s.migrateFromDataAttributes(this.api, t);
   }
 }
 export {
-  y as BLOCK_ID,
-  Z as RecommendationBlock
+  k as BLOCK_ID,
+  Y as RecommendationBlock
 };

package/dist/extensions/Blocks/Recommendation/controls/image/index.js

@@ -1,19 +1,36 @@
-import { createImageMarginsControl as m, createImageSizeControl as n } from "../../../controlFactories.js";
-import { RecommendationBlockId as o } from "../../constants/blockIds.js";
-import { RecommendationControlId as r } from "../../constants/controlIds.js";
-import { BLOCK_ROOT_SELECTOR as t } from "../../constants/selectors.js";
-const e = n(
-  r.IMAGE_SIZE,
-  o.IMAGE,
-  t
-), i = m(
-  r.IMAGE_MARGINS,
-  o.IMAGE,
-  t
-), C = {
-  size: e,
-  margins: i
+import { createImageMarginsControl as I, createImageSizeControl as d } from "../../../controlFactories.js";
+import { RecommendationBlockId as n } from "../../constants/blockIds.js";
+import { RecommendationControlId as m } from "../../constants/controlIds.js";
+import { BLOCK_ROOT_SELECTOR as a } from "../../constants/selectors.js";
+import { hasGetStyle as f } from "../../utils/tagName.js";
+const s = ["width", "height"];
+function g(t) {
+  return f(t) && s.some((e) => !!t.getStyle(e));
+}
+class S extends d(
+  m.IMAGE_SIZE,
+  n.IMAGE,
+  a
+) {
+  getAdditionalModifications(e) {
+    const r = this.getTargetNodes(e).map((o) => o.querySelector("img")).filter((o) => !!o && g(o));
+    if (!r.length)
+      return;
+    const i = this.api.getDocumentModifier();
+    return r.forEach((o) => {
+      const l = i.modifyHtml(o);
+      s.forEach((c) => l.removeStyle(c));
+    }), i;
+  }
+}
+const h = I(
+  m.IMAGE_MARGINS,
+  n.IMAGE,
+  a
+), A = {
+  size: S,
+  margins: h
 };
 export {
-  C as ImageControls
+  A as ImageControls
 };

package/dist/extensions/Blocks/Recommendation/utils/strategyCommit.js

@@ -1,60 +1,43 @@
-import { useRecommendationExtensionStore as c } from "../store/recommendation.js";
-import { refreshBlockOutsidePanel as l } from "./blockRefresh.js";
-import { toBlockUrlConfig as i } from "./blockUrlConfig.js";
-import { commitBlockChangeOutsidePanel as y, readBlockNodeConfigs as d } from "./nodeConfigWriter.js";
-function S(o, t) {
-  const r = JSON.stringify(i(t));
-  return d(o).find((n) => JSON.stringify(i(n)) !== r) ?? t;
+import { useRecommendationExtensionStore as g } from "../store/recommendation.js";
+import { refreshBlockOutsidePanel as y } from "./blockRefresh.js";
+import { toBlockUrlConfig as a } from "./blockUrlConfig.js";
+import { commitBlockChangeOutsidePanel as l, readBlockNodeConfigs as S } from "./nodeConfigWriter.js";
+function d(e, t) {
+  const r = JSON.stringify(a(t));
+  return S(e).find((o) => JSON.stringify(a(o)) !== r) ?? t;
 }
-async function m(o, t, r = () => !0) {
-  const n = c();
-  await Promise.all(o.map(async (e) => {
-    if (!await n.refreshStrategyBlock(e))
+async function p(e, t, r) {
+  const o = g(), s = [e, t];
+  (/* @__PURE__ */ new Map([[e, d(e, r)], [t, r]])).forEach((n, i) => {
+    var m;
+    (!!n.strategyId || i === t) && ((m = o.blockStates[i]) == null ? void 0 : m.recommendationConfigs.strategyId) !== n.strategyId && o.seedBlockUrlConfig(i, a(n));
+  }), o.resetArrivedBlock(t), await Promise.all(s.map(async (n) => {
+    if (!await o.refreshStrategyBlock(n))
       return;
-    const { recommendationConfigs: s, recommendationProducts: a } = n.blockStates[e];
-    r(e, s.size) && y(
-      e,
-      { size: s.size },
-      a,
-      t
+    const { recommendationConfigs: i, recommendationProducts: f } = o.blockStates[n];
+    l(
+      n,
+      { size: i.size },
+      f,
+      "Refresh duplicated recommendation strategy"
     );
   }));
 }
-async function C(o, t, r) {
-  const n = c(), e = [o, t];
-  (/* @__PURE__ */ new Map([[o, S(o, r)], [t, r]])).forEach((a, f) => {
-    var g;
-    (!!a.strategyId || f === t) && ((g = n.blockStates[f]) == null ? void 0 : g.recommendationConfigs.strategyId) !== a.strategyId && n.seedBlockUrlConfig(f, i(a));
-  }), n.resetArrivedBlock(t), await m(e, "Refresh duplicated recommendation strategy");
-}
-async function k(o) {
-  const t = c(), r = o.filter(({ config: e }) => !!e.strategyId);
-  if (!r.length)
-    return;
-  r.forEach(({ id: e, config: s }) => t.seedBlockUrlConfig(e, i(s))), t.invalidateStrategies();
-  const n = new Map(r.map(({ id: e, config: s }) => [e, s.size]));
-  await m(
-    r.map(({ id: e }) => e),
-    "Refresh recommendation strategy",
-    (e, s) => s !== n.get(e)
-  );
-}
-async function O(o, t) {
-  if (!c().blockStates[o])
+async function k(e, t) {
+  if (!g().blockStates[e])
     return !1;
-  const r = await l(o, {
-    pickUpdates: ({ strategyId: n, strategy: e, size: s, nodeConfigVersion: a }) => ({
-      strategyId: n,
-      strategy: e,
-      size: s,
-      configVersion: a
+  const r = await y(e, {
+    pickUpdates: ({ strategyId: o, strategy: s, size: c, nodeConfigVersion: n }) => ({
+      strategyId: o,
+      strategy: s,
+      size: c,
+      configVersion: n
     }),
     description: t
   });
-  return r || console.error("[recommendation] strategy change did not reach a node", { blockId: o }), r;
+  return r || console.error("[recommendation] strategy change did not reach a node", { blockId: e }), r;
 }
 export {
-  O as commitStrategyChange,
-  k as refreshArrivedStrategyBlocks,
-  C as refreshDuplicatedStrategyBlocks
+  k as commitStrategyChange,
+  p as refreshDuplicatedStrategyBlocks
 };

package/dist/src/extensions/Blocks/Recommendation/block.d.ts

@@ -19,9 +19,6 @@ export declare class RecommendationBlock extends Block {
     private _resetSourceIds;
     /** Ids handed to duplicates the document root does not show yet: a copied module renames all its blocks at once. */
     private _unsettledDuplicateIds;
-    private _arrivedNodes;
-    /** Ids whose arrival or duplicate refresh is in flight, so a second refresh of the same block does not race it. */
-    private _refreshingIds;
     constructor();
     getId(): string;
     getIcon(): string;
@@ -47,8 +44,6 @@ export declare class RecommendationBlock extends Block {
      * @param node - The newly created block node
      */
     onCreated(node: ImmutableHtmlNode): void;
-    /** Refreshes every block of an opened template: Stripo restores a stored document without `onCreated`. */
-    onDocumentInit(): void;
     /**
      * Called when the document changes or template is loaded
      *
@@ -90,12 +85,6 @@ export declare class RecommendationBlock extends Block {
      * the renamed one depending on where the module was dropped (SD-149916).
      */
     private _scheduleArrivalReset;
-    /**
-     * Re-reads the strategy of a block that arrived with saved cards, without waiting for a click.
-     * Blocks of one opened template or dropped module arrive in one tick and refresh together.
-     */
-    private _queueArrivalRefresh;
-    private _refreshArrivedBlocks;
     /**
      * Heals a duplicate that slipped past the create-time `_detectDuplicate`
      * scan (or that was persisted by an older template before SD-142352).

package/dist/src/extensions/Blocks/Recommendation/controls/image/index.d.ts

@@ -3,35 +3,41 @@
  *
  * Controls for styling the product image element in recommendation cards.
  */
+import type { ImmutableHtmlNode } from '@stripoinc/ui-editor-extensions';
+declare const SizeControl_base: {
+    new (): {
+        getId(): string;
+        getTargetNodes(root: ImmutableHtmlNode): ImmutableHtmlNode[];
+        getParentControlId(): string;
+        api: import("@stripoinc/ui-editor-extensions").ControlApi;
+        getLabels(): import("@stripoinc/ui-editor-extensions").ControlLabels | undefined;
+        getModificationDescription(): import("@stripoinc/ui-editor-extensions").ModificationDescription | undefined;
+        getAdditionalModifications(_root: ImmutableHtmlNode): import("@stripoinc/ui-editor-extensions").TemplateModifier<import("@stripoinc/ui-editor-extensions").HtmlNodeModifier, import("@stripoinc/ui-editor-extensions").CssNodeModifier> | undefined;
+        isVisible(_node: ImmutableHtmlNode): boolean;
+        getElementLockCategory(): import("@stripoinc/ui-editor-extensions").ElementLockCategory | undefined;
+    };
+};
+declare class SizeControl extends SizeControl_base {
+    getAdditionalModifications(root: ImmutableHtmlNode): import("@stripoinc/ui-editor-extensions").TemplateModifier<import("@stripoinc/ui-editor-extensions").HtmlNodeModifier, import("@stripoinc/ui-editor-extensions").CssNodeModifier> | undefined;
+}
 /**
  * Grouped Image Controls
  * Use this object for cleaner imports in extension.ts
  */
 export declare const ImageControls: {
-    readonly size: {
-        new (): {
-            getId(): string;
-            getTargetNodes(root: import("@stripoinc/ui-editor-extensions").ImmutableHtmlNode): import("@stripoinc/ui-editor-extensions").ImmutableHtmlNode[];
-            getParentControlId(): string;
-            api: import("@stripoinc/ui-editor-extensions").ControlApi;
-            getLabels(): import("@stripoinc/ui-editor-extensions").ControlLabels | undefined;
-            getModificationDescription(): import("@stripoinc/ui-editor-extensions").ModificationDescription | undefined;
-            getAdditionalModifications(_root: import("@stripoinc/ui-editor-extensions").ImmutableHtmlNode): import("@stripoinc/ui-editor-extensions").TemplateModifier<import("@stripoinc/ui-editor-extensions").HtmlNodeModifier, import("@stripoinc/ui-editor-extensions").CssNodeModifier> | undefined;
-            isVisible(_node: import("@stripoinc/ui-editor-extensions").ImmutableHtmlNode): boolean;
-            getElementLockCategory(): import("@stripoinc/ui-editor-extensions").ElementLockCategory | undefined;
-        };
-    };
+    readonly size: typeof SizeControl;
     readonly margins: {
         new (): {
             getId(): string;
-            getTargetNodes(root: import("@stripoinc/ui-editor-extensions").ImmutableHtmlNode): import("@stripoinc/ui-editor-extensions").ImmutableHtmlNode[];
+            getTargetNodes(root: ImmutableHtmlNode): ImmutableHtmlNode[];
             getParentControlId(): string;
             api: import("@stripoinc/ui-editor-extensions").ControlApi;
             getLabels(): import("@stripoinc/ui-editor-extensions").ControlLabels | undefined;
             getModificationDescription(): import("@stripoinc/ui-editor-extensions").ModificationDescription | undefined;
-            getAdditionalModifications(_root: import("@stripoinc/ui-editor-extensions").ImmutableHtmlNode): import("@stripoinc/ui-editor-extensions").TemplateModifier<import("@stripoinc/ui-editor-extensions").HtmlNodeModifier, import("@stripoinc/ui-editor-extensions").CssNodeModifier> | undefined;
-            isVisible(_node: import("@stripoinc/ui-editor-extensions").ImmutableHtmlNode): boolean;
+            getAdditionalModifications(_root: ImmutableHtmlNode): import("@stripoinc/ui-editor-extensions").TemplateModifier<import("@stripoinc/ui-editor-extensions").HtmlNodeModifier, import("@stripoinc/ui-editor-extensions").CssNodeModifier> | undefined;
+            isVisible(_node: ImmutableHtmlNode): boolean;
             getElementLockCategory(): import("@stripoinc/ui-editor-extensions").ElementLockCategory | undefined;
         };
     };
 };
+export {};

package/dist/src/extensions/Blocks/Recommendation/utils/strategyCommit.d.ts

@@ -5,15 +5,6 @@ import type { RecommendationNodeConfig } from '../types/nodeConfig';
  * @param config - the arriving block's persisted config; seeds either block its store entry does not match
  */
 export declare function refreshDuplicatedStrategyBlocks(sourceId: number, duplicateId: number, config: RecommendationNodeConfig): Promise<void>;
-/**
- * Re-reads the strategy of blocks that arrived from a saved template or module, so the panel and
- * preview show a strategy edited since the design was saved without selecting the block.
- * @param blocks - each block's id with its persisted config, the truth its store entry is seeded from
- */
-export declare function refreshArrivedStrategyBlocks(blocks: {
-    id: number;
-    config: RecommendationNodeConfig;
-}[]): Promise<void>;
 /**
  * Writes a strategy change and its products straight to the document, without the settings panel.
  *

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@useinsider/guido",
-  "version": "3.21.0-beta.7e166b5",
+  "version": "3.21.0-beta.ab76470",
   "description": "Guido is a Vue + TypeScript wrapper for Email Plugin. Easily embed the email editor in your Vue applications.",
   "main": "./dist/guido.umd.cjs",
   "module": "./dist/library.js",