Merge pull request #10 from RhubarbPHP/patch/RemoveEventProcessingClassOnComplete

acuthbert · web-flow · commit 3599cc138881 · 2019-05-08T10:30:00.000+01:00
Remove 'event-processing' class on completed
diff --git a/src/Leaves/BackgroundTaskViewBridge.js b/src/Leaves/BackgroundTaskViewBridge.js
@@ -36,7 +36,9 @@ bridge.prototype.start = function(){
     xmlhttp.onreadystatechange = function () {
 
         // if the ready state is now 4 we've already processed our entire output.
-        if (xmlhttp.completed) {
+        if (xmlhttp.completed || xmlhttp.readyState === 4) {
+            document.body.classList.remove("event-processing");
+
             return;
         }
 

Original file line number	Diff line number	Diff line change
`@@ -36,7 +36,9 @@ bridge.prototype.start = function(){`
`36`	`36`	`xmlhttp.onreadystatechange = function () {`
`37`	`37`
`38`	`38`	`// if the ready state is now 4 we've already processed our entire output.`
`39`		`- if (xmlhttp.completed) {`
	`39`	`+ if (xmlhttp.completed \|\| xmlhttp.readyState === 4) {`
	`40`	`+ document.body.classList.remove("event-processing");`
	`41`	`+`
`40`	`42`	`return;`
`41`	`43`	`}`
`42`	`44`