Temporarily fix #63 and #75 iOS spinner always showing

mderrick · Jul 26, 2017 · 4c4f982 · 4c4f982
1 parent 4b4ec1c
commit 4c4f982
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/src/DefaultPlayer/DefaultPlayer.js b/src/DefaultPlayer/DefaultPlayer.js
@@ -128,7 +128,10 @@ export default videoConnect(
             readyState,
             networkState,
             error: error || networkState === 3,
-            loading: readyState < 4,
+            // TODO: This is not pretty. Doing device detection to remove
+            // spinner on iOS devices for a quick and dirty win. We should see if
+            // we can use the same readyState check safely across all browsers.
+            loading: readyState < (/iPad|iPhone|iPod/.test(navigator.userAgent) ? 1 : 4),
             percentagePlayed: getPercentagePlayed(restState),
             percentageBuffered: getPercentageBuffered(restState),
             ...restState