@magicfeedback/popup-sdk 1.7.2 → 1.8.1

package/dist/react-native.d.mts

@@ -538,7 +538,27 @@ declare class DeepdotsPopups {
     private showDefinition;
     private shouldShow;
     private matchesSegmentsLang;
+    /** `segments.path`: rutas donde el popup PUEDE mostrarse. Sin lista, en todas. */
     private matchesSegmentsPath;
+    /**
+     * `segments.excludedPaths`: rutas donde el popup NO debe mostrarse. Gana sobre
+     * `segments.path` (`path: ['/']` + `excludedPaths: ['/cart']` = en todas menos el
+     * carrito) y se aplica también cuando no hay `path` (= en todas menos las excluidas).
+     */
+    private isPathExcluded;
+    /**
+     * Compara UN candidato de `segments.path`/`segments.excludedPaths` con la ruta
+     * actual. Mismas reglas para incluir y para excluir:
+     *  - URL absoluta  → href completo exacto
+     *  - empieza por / → subcadena del href (así entran las rutas con hash: `/#/home`)
+     *  - resto         → pathname exacto
+     * Ignora entradas que no sean string: el backend puede colar un `null` en la lista
+     * y no puede tumbar la evaluación del trigger entero (mismo fallo que hubo con
+     * `segments.lang`).
+     */
+    private matchesPathCandidate;
+    /** href + pathname normalizados de la ruta a evaluar, o null si no hay `window.location`. */
+    private resolveCurrentLocation;
     private evaluateCooldown;
     private evaluateLegacyCondition;
     private hasCooldownElapsed;

package/dist/react-native.d.ts

@@ -538,7 +538,27 @@ declare class DeepdotsPopups {
     private showDefinition;
     private shouldShow;
     private matchesSegmentsLang;
+    /** `segments.path`: rutas donde el popup PUEDE mostrarse. Sin lista, en todas. */
     private matchesSegmentsPath;
+    /**
+     * `segments.excludedPaths`: rutas donde el popup NO debe mostrarse. Gana sobre
+     * `segments.path` (`path: ['/']` + `excludedPaths: ['/cart']` = en todas menos el
+     * carrito) y se aplica también cuando no hay `path` (= en todas menos las excluidas).
+     */
+    private isPathExcluded;
+    /**
+     * Compara UN candidato de `segments.path`/`segments.excludedPaths` con la ruta
+     * actual. Mismas reglas para incluir y para excluir:
+     *  - URL absoluta  → href completo exacto
+     *  - empieza por / → subcadena del href (así entran las rutas con hash: `/#/home`)
+     *  - resto         → pathname exacto
+     * Ignora entradas que no sean string: el backend puede colar un `null` en la lista
+     * y no puede tumbar la evaluación del trigger entero (mismo fallo que hubo con
+     * `segments.lang`).
+     */
+    private matchesPathCandidate;
+    /** href + pathname normalizados de la ruta a evaluar, o null si no hay `window.location`. */
+    private resolveCurrentLocation;
     private evaluateCooldown;
     private evaluateLegacyCondition;
     private hasCooldownElapsed;

package/dist/react-native.js

@@ -522,6 +522,29 @@ function ddApplySurveyPrimaryColor(el, style){
   }
 });
 
+// src/ui/busy.ts
+function setSurveyBusy(el, busy) {
+  if (!el) return;
+  el.style.pointerEvents = busy ? "none" : "";
+  el.setAttribute("aria-busy", busy ? "true" : "false");
+  if ("inert" in el) {
+    el.inert = busy;
+  }
+}
+var BUSY_JS;
+var init_busy = __esm({
+  "src/ui/busy.ts"() {
+    "use strict";
+    BUSY_JS = `
+function ddSetSurveyBusy(el, busy){
+  if(!el){ return; }
+  el.style.pointerEvents = busy ? 'none' : '';
+  el.setAttribute('aria-busy', busy ? 'true' : 'false');
+  if('inert' in el){ el.inert = busy; }
+}`;
+  }
+});
+
 // src/assets/style.css
 var style_default;
 var init_style = __esm({
@@ -2964,6 +2987,7 @@ async function renderPopup(container, surveyId, productId, actions, emit, onClos
   revealTimer = setTimeout(reveal, REVEAL_TIMEOUT_MS);
   function setLoading(isLoading) {
     spinnerEl.style.display = isLoading ? "flex" : "none";
+    setSurveyBusy(formWrapper, isLoading);
     if (!isLoading) {
       formHost.style.visibility = "visible";
       revealWhenPainted2();
@@ -3119,6 +3143,7 @@ var init_renderPopup = __esm({
     init_font();
     init_logo();
     init_surveyPalette();
+    init_busy();
     init_logger();
     init_labels();
     init_reveal();
@@ -3338,6 +3363,7 @@ init_font();
 init_labels();
 init_reveal();
 init_surveyPalette();
+init_busy();
 init_style();
 var REVEAL_JS = `
 function ddCreateReveal(popup, onReveal, timeoutMs){
@@ -3557,6 +3583,7 @@ ${customCss}
   var footer=document.getElementById('dd-footer');
   var spinner=document.getElementById('dd-spinner');
   var formHost=document.getElementById('mf');
+  var formWrapper=document.getElementById('dd-form-wrapper');
   var errorHint=document.getElementById('dd-error');
   var backBtn=document.getElementById('dd-back');
   var startBtn=document.getElementById('dd-start');
@@ -3716,6 +3743,7 @@ ${customCss}
   }
 
 ${PRIMARY_COLOR_JS}
+${BUSY_JS}
 ${REVEAL_JS}
   // El popup se ense\xF1a cuando el survey est\xE1 pintado; \`ready\` deja al host abrir su propio
   // Modal en ese momento en vez de montar un WebView en blanco.
@@ -3726,6 +3754,11 @@ ${REVEAL_JS}
 
   function setLoading(isLoading){
     spinner.style.display=isLoading?'flex':'none';
+    // El spinner no tapa el survey (28px en posicion absoluta), asi que sin esto el usuario
+    // puede seguir cambiando de opcion mientras se envia la pagina, y ese cambio ya no viaja
+    // con ella. Se bloquea el wrapper, que es el elemento estable: el div de dentro lo
+    // sustituye @magicfeedback/native por su propio contenedor al generar el survey.
+    ddSetSurveyBusy(formWrapper, isLoading);
     if(!isLoading){
       formHost.style.visibility='visible';
       // Fin de la carga, sea por survey cargado o por error. Entre p\xE1ginas es no-op: la
@@ -5508,6 +5541,9 @@ var DeepdotsPopups = class {
     this.emitEvent("popup_shown", def.surveyId, { popupId: def.id });
   }
   shouldShow(def, pathUrl, skipPathCheck = false) {
+    if (this.isPathExcluded(def, pathUrl)) {
+      return false;
+    }
     if (!skipPathCheck && !this.matchesSegmentsPath(def, pathUrl)) {
       return false;
     }
@@ -5538,40 +5574,85 @@ var DeepdotsPopups = class {
     }
     return matches;
   }
+  /** `segments.path`: rutas donde el popup PUEDE mostrarse. Sin lista, en todas. */
   matchesSegmentsPath(def, pathUrl) {
     const paths = def.segments?.path;
-    if (!paths || paths.length === 0) return true;
-    if (typeof window === "undefined" || !window.location) {
+    if (!Array.isArray(paths) || paths.length === 0) return true;
+    const location = this.resolveCurrentLocation(pathUrl);
+    if (!location) {
       this.debug("No window.location available for path comparison", { popupId: def.id, paths });
       return true;
     }
-    const normalizedHref = this.normalizeUrl(pathUrl || window.location.href || "");
-    const currentUrl = this.safeParseUrl(normalizedHref);
-    const normalizedPath = this.normalizeUrl(currentUrl?.pathname || window.location.pathname || "");
     const matches = paths.some((rawCandidate) => {
-      const candidate = this.normalizeUrl(rawCandidate);
-      let match = false;
-      if (candidate.startsWith("http://") || candidate.startsWith("https://")) {
-        match = normalizedHref === candidate;
-      } else if (candidate.startsWith("/")) {
-        match = normalizedHref.includes(candidate);
-      } else {
-        match = normalizedPath === candidate;
-      }
+      const match = this.matchesPathCandidate(rawCandidate, location);
       this.debug("Path comparison", {
         popupId: def.id,
-        candidate,
-        currentPath: normalizedPath,
-        currentHref: normalizedHref,
+        candidate: rawCandidate,
+        currentPath: location.path,
+        currentHref: location.href,
         match
       });
       return match;
     });
     if (!matches) {
-      this.debug("No path match for popup", { popupId: def.id, paths, currentPath: normalizedPath });
+      this.debug("No path match for popup", { popupId: def.id, paths, currentPath: location.path });
     }
     return matches;
   }
+  /**
+   * `segments.excludedPaths`: rutas donde el popup NO debe mostrarse. Gana sobre
+   * `segments.path` (`path: ['/']` + `excludedPaths: ['/cart']` = en todas menos el
+   * carrito) y se aplica también cuando no hay `path` (= en todas menos las excluidas).
+   */
+  isPathExcluded(def, pathUrl) {
+    const excluded = def.segments?.excludedPaths;
+    if (!Array.isArray(excluded) || excluded.length === 0) return false;
+    const location = this.resolveCurrentLocation(pathUrl);
+    if (!location) {
+      this.debug("No window.location available for excluded path comparison", { popupId: def.id, excluded });
+      return false;
+    }
+    const hit = excluded.find((rawCandidate) => this.matchesPathCandidate(rawCandidate, location));
+    if (hit !== void 0) {
+      this.debug("Popup excluded by path", {
+        popupId: def.id,
+        candidate: hit,
+        currentPath: location.path,
+        currentHref: location.href
+      });
+      return true;
+    }
+    return false;
+  }
+  /**
+   * Compara UN candidato de `segments.path`/`segments.excludedPaths` con la ruta
+   * actual. Mismas reglas para incluir y para excluir:
+   *  - URL absoluta  → href completo exacto
+   *  - empieza por / → subcadena del href (así entran las rutas con hash: `/#/home`)
+   *  - resto         → pathname exacto
+   * Ignora entradas que no sean string: el backend puede colar un `null` en la lista
+   * y no puede tumbar la evaluación del trigger entero (mismo fallo que hubo con
+   * `segments.lang`).
+   */
+  matchesPathCandidate(rawCandidate, location) {
+    if (typeof rawCandidate !== "string") return false;
+    const candidate = this.normalizeUrl(rawCandidate);
+    if (!candidate) return false;
+    if (candidate.startsWith("http://") || candidate.startsWith("https://")) {
+      return location.href === candidate;
+    }
+    if (candidate.startsWith("/")) {
+      return location.href.includes(candidate);
+    }
+    return location.path === candidate;
+  }
+  /** href + pathname normalizados de la ruta a evaluar, o null si no hay `window.location`. */
+  resolveCurrentLocation(pathUrl) {
+    if (typeof window === "undefined" || !window.location) return null;
+    const href = this.normalizeUrl(pathUrl || window.location.href || "");
+    const parsed = this.safeParseUrl(href);
+    return { href, path: this.normalizeUrl(parsed?.pathname || window.location.pathname || "") };
+  }
   evaluateCooldown(def, condition) {
     switch (condition.answered) {
       case "SHOWED":