Merge pull request #477 from isymchych/fix-safari-audio

do not hide small video until stream has been attached
pull/479/merge
Paweł Domas 9 years ago
commit 8221a59261
  1. 4614
      libs/lib-jitsi-meet.js
  2. 6
      modules/UI/videolayout/RemoteVideo.js

File diff suppressed because it is too large Load Diff

@ -249,7 +249,6 @@ RemoteVideo.prototype.addRemoteStreamElement = function (stream) {
UIUtils.prependChild(this.container, streamElement); UIUtils.prependChild(this.container, streamElement);
let sel = $(`#${newElementId}`); let sel = $(`#${newElementId}`);
sel.hide();
// If the container is currently visible we attach the stream. // If the container is currently visible we attach the stream.
if (!isVideo || (this.container.offsetParent !== null && isVideo)) { if (!isVideo || (this.container.offsetParent !== null && isVideo)) {
@ -258,6 +257,11 @@ RemoteVideo.prototype.addRemoteStreamElement = function (stream) {
stream.attach(sel); stream.attach(sel);
} }
// hide element only after stream was (maybe) attached
// because Temasys plugin requires video element
// to be visible to attach the stream
sel.hide();
// reselect // reselect
if (RTCBrowserType.isTemasysPluginUsed()) { if (RTCBrowserType.isTemasysPluginUsed()) {
sel = $(`#${newElementId}`); sel = $(`#${newElementId}`);

Loading…
Cancel
Save