Push fix for assessmentResults not consistently being marked complete

joe-replin · Feb 9, 2017 · f17cfc7 · f17cfc7
1 parent 66a3ce2
commit f17cfc7
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");
+                    }
                 }
             }
         },