stop spoor from handling beforeunload if either browser prompt settin…

…g is enabled (fixes #10)
cgkineo · Mar 1, 2019 · 38c2417 · 38c2417
1 parent 89f5b13
commit 38c2417
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/bower.json b/bower.json
@@ -1,6 +1,6 @@
 {
 	"name": "adapt-close",
-	"version": "1.1.0",
+	"version": "1.1.1",
 	"framework": ">=2.0.4",
 	"displayName": "Close",
 	"extension": "close",

diff --git a/js/adapt-close.js b/js/adapt-close.js
@@ -74,6 +74,7 @@ define([ 'core/js/adapt' ], function(Adapt) {
         }
 
         if (config.browserPromptIfIncomplete || config.browserPromptIfComplete) {
+            $(window).off('beforeunload');// stop spoor from handling beforeunload - if it handles the event first, LMSFinish will get called regardless of what the user selects in the prompt
             $(window).on('beforeunload.close', _.partial(onBeforeUnload, config));
         }
     }