Merge pull request #296 from CleverTap/develop

[WEB-3310] Fixed Inbox Open Not Working for SPA App
CleverTap · Nov 7, 2024 · 3354760 · 3354760
2 parents eb727ea + d6b7b9f
commit 3354760
Show file tree

Hide file tree

Showing 6 changed files with 28 additions and 5 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,6 +1,9 @@
 # Change Log
 All notable changes to this project will be documented in this file.
 
+## [1.11.6] 04th Nov, 2024
+- Fix for Checking Web Inbox when inbox Node becomes stale
+
 ## [1.11.5] 30th Oct, 2024
 - Added support for personalisation in visual editor
 

diff --git a/clevertap.js b/clevertap.js
diff --git a/clevertap.js.map b/clevertap.js.map
diff --git a/clevertap.min.js b/clevertap.min.js
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "clevertap-web-sdk",
-  "version": "1.11.5",
+  "version": "1.11.6",
   "description": "",
   "main": "clevertap.js",
   "scripts": {

diff --git a/src/modules/web-inbox/WebInbox.js b/src/modules/web-inbox/WebInbox.js
@@ -377,7 +377,7 @@ export class Inbox extends HTMLElement {
             }
           }
         }
-      } else if (this.inboxSelector.contains(e.target) || this.isInboxOpen) {
+      } else if (this.checkForWebInbox(e) || this.isInboxOpen) {
         if (this.isInboxFromFlutter) {
           this.isInboxFromFlutter = false
         } else {
@@ -387,6 +387,16 @@ export class Inbox extends HTMLElement {
     }
   })()
 
+  /**
+   * This function checks if the current Event Node is same as the already stored inboxSelector or the
+   * inboxSelector present in the document
+   */
+  checkForWebInbox (e) {
+    const config = StorageManager.readFromLSorCookie(WEBINBOX_CONFIG) || {}
+    return this.inboxSelector.contains(e.target) ||
+     document.getElementById(config.inboxSelector).contains(e.target)
+  }
+
   /**
    * This function will be called every time when a message comes into the inbox viewport and it's visibility increases to 50% or drops below 50%
    * If a msg is 50% visible in the UI, we need to mark the message as viewed in LS and raise notification viewed event