Skip to content

Commit

Permalink
fix(FEC-9085): playState is marked as "paused" although playing (#4010)
Browse files Browse the repository at this point in the history
Added isIPad checker to the interface update, in case EmbedPlayer.EnableIpadHTMLControls config is set to false.
  • Loading branch information
Shalom Meoded authored and OrenMe committed May 19, 2019
1 parent ac16ae3 commit eac9cd4
Showing 1 changed file with 7 additions and 3 deletions.
10 changes: 7 additions & 3 deletions modules/EmbedPlayer/resources/mw.EmbedPlayerNative.js
Original file line number Diff line number Diff line change
Expand Up @@ -1309,9 +1309,13 @@
}

// Update the interface ( if paused )
if (!this.ignoreNextNativeEvent && this._propagateEvents && this.paused && ( mw.getConfig('EmbedPlayer.EnableIpadHTMLControls') === true )) {
this.parent_play();
} else {
var enableIpadHTMLControls = mw.isIpad() ?
( mw.getConfig('EmbedPlayer.EnableIpadHTMLControls') === true ) : true;

if (!this.ignoreNextNativeEvent && this._propagateEvents && this.paused && enableIpadHTMLControls) {
this.parent_play();
}
else {
// make sure the interface reflects the current play state if not calling parent_play()
this.playInterfaceUpdate();
this.absoluteStartPlayTime = new Date().getTime();
Expand Down

0 comments on commit eac9cd4

Please sign in to comment.