Merge pull request #74 from GleapSDK/modal-action-button-url

[v14.6.3] [#80645] fix: modal action url opening in the new window and same window

Author: boehlerlukas

build/cjs/index.js

@@ -1,6 +1,6 @@
 {
   "name": "gleap",
-  "version": "14.6.2",
+  "version": "14.6.3",
   "main": "build/cjs/index.js",
   "module": "build/esm/index.mjs",
   "exports": {

build/esm/index.mjs

@@ -306,7 +306,7 @@ export const registerGleapChecklist = () => {
         this._hasLoaded = false;
         this.renderResponse();
         const sharedKey = this.getAttribute("sharedKey");
-        
+
         this._networkManager
           .validateChecklist(outboundId, sharedKey)
           .then((internalId) => {
@@ -1147,9 +1147,12 @@ export const registerGleapChecklist = () => {
                 if (step.action === "BOT") Gleap.startBot(step.botId);
                 else if (step.action === "CUSTOM_ACTION")
                   Gleap.triggerCustomAction(step.actionBody);
-                else if (step.action === "REDIRECT_URL")
+                else if (step.action === "REDIRECT_URL") {
+                  e.preventDefault();
+                  e.stopPropagation();
+                  e.stopImmediatePropagation();
                   Gleap.openURL(step.actionBody, !!step.actionOpenInNewTab);
-                else if (step.action === "FEEDBACK_FLOW")
+                } else if (step.action === "FEEDBACK_FLOW")
                   Gleap.startFeedbackFlow(step.formId);
                 else if (step.action === "NEWS_ARTICLE")
                   Gleap.openNewsArticle(step.articleId);

package.json

@@ -37,17 +37,7 @@ export default class GleapFrameManager {
     if (url && url.length > 0) {
       if (newTab) {
         const newWindow = window.open(url, "_blank");
-
-        // Check if the new window was successfully created and not blocked
-        if (
-          !newWindow ||
-          newWindow.closed ||
-          typeof newWindow.closed === "undefined"
-        ) {
-          // If the new window was blocked, navigate in the same tab instead
-          window.location.href = url;
-        } else {
-          // If the new window was created successfully, bring it into focus
+        if (newWindow) {
           newWindow.focus();
         }
       } else {