Merge pull request adaptlearning#27 from irisEd/issue_1419

Fix for assessment results not consistently being marked complete in some contexts
joe-replin · Feb 15, 2017 · fb0c4ad · fb0c4ad
2 parents 66a3ce2 + f17cfc7
commit fb0c4ad
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/js/adapt-contrib-assessmentResults.js b/js/adapt-contrib-assessmentResults.js
@@ -143,7 +143,13 @@ define(function(require) {
 
                 if (this._isVisibleTop || this._isVisibleBottom) {
                     this.setCompletionStatus();
-                    this.$el.off("inview");
+
+                    // Sometimes (with mobile and virtual keyboards) inview can be triggered
+                    // but the component is not _visible = true, so it does not get marked
+                    // complete. Delay the unbinding of the inview listener until complete
+                    if (this.isComplete()) {
+                      this.$el.off("inview");
+                    }
                 }
             }
         },