feat: Remove onunload handler before redirecting

Fixes #138

Author: nathanbuchar

src/embedded.js

@@ -669,6 +669,8 @@ class HelloSign extends Emitter {
     this._sendConfigurationMessage();
     this._clearInitTimeout();
 
+    window.addEventListener('beforeunload', this._onBeforeUnload);
+
     this.emit(settings.events.READY, payload);
   }
 
@@ -827,6 +829,8 @@ class HelloSign extends Emitter {
 
     this._isSentOrSigned = true;
 
+    window.removeEventListener('beforeunload', this._onBeforeUnload);
+
     this.emit(settings.events.SEND, payload);
   }
 
@@ -848,6 +852,8 @@ class HelloSign extends Emitter {
 
     this._isSentOrSigned = true;
 
+    window.removeEventListener('beforeunload', this._onBeforeUnload);
+
     this.emit(settings.events.SIGN, payload);
   }
 
@@ -1049,7 +1055,6 @@ class HelloSign extends Emitter {
     this._isOpen = true;
 
     window.addEventListener('message', this._onMessage);
-    window.addEventListener('beforeunload', this._onBeforeUnload);
 
     this.emit(settings.events.OPEN, {
       url: this._iFrameURL.href,