Skip to content

Commit

Permalink
fix: don't emit tap events on tech when using native controls (#3873)
Browse files Browse the repository at this point in the history
If using nativeControlsForTouch, skip the emitTapEvents listeners from
being setup. This avoids taps being taken away from native video
elements and stopping controls being displayed.
  • Loading branch information
darrennolan authored and gkatsev committed Dec 22, 2016
1 parent e92db4f commit 42507f8
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions src/js/tech/tech.js
Original file line number Diff line number Diff line change
Expand Up @@ -140,8 +140,10 @@ class Tech extends Component {
this.initTextTrackListeners();
this.initTrackListeners();

// Turn on component tap events
this.emitTapEvents();
// Turn on component tap events only if not using native controls
if (!options.nativeControlsForTouch) {
this.emitTapEvents();
}
}

/* Fallbacks for unsupported event types
Expand Down

0 comments on commit 42507f8

Please sign in to comment.