diff --git a/css/videolayout_default.css b/css/videolayout_default.css index c9ca1fc870..7bd3083341 100644 --- a/css/videolayout_default.css +++ b/css/videolayout_default.css @@ -101,12 +101,14 @@ } #largeVideo, +#largeVideoWrapper, #largeVideoContainer { overflow: hidden; text-align: center; } -#largeVideo +#largeVideo, +#largeVideoWrapper { object-fit: cover; } @@ -116,6 +118,9 @@ #localVideoWrapper>video, #localVideoWrapper>object, #localVideoWrapper, +#largeVideoWrapper, +#largeVideoWrapper>video, +#largeVideoWrapper>object, .videocontainer>video, .videocontainer>object { position: absolute; diff --git a/index.html b/index.html index 4b5dbacbb0..d43fbb04c6 100644 --- a/index.html +++ b/index.html @@ -25,7 +25,7 @@ - + diff --git a/modules/UI/videolayout/LargeVideo.js b/modules/UI/videolayout/LargeVideo.js index 804d5fb077..5b84930d32 100644 --- a/modules/UI/videolayout/LargeVideo.js +++ b/modules/UI/videolayout/LargeVideo.js @@ -267,7 +267,7 @@ function changeVideo(isVisible) { if (isVisible) { LargeVideo.VideoLayout.largeVideoUpdated(currentSmallVideo); - $('#largeVideo').fadeIn(300); + $('#largeVideoWrapper').fadeIn(300); } } @@ -288,7 +288,9 @@ function createLargeVideoHTML() '' + '' + '' + - '' + + '
' + + '' + + '
' + ''; html += ''; $(html).prependTo("#videospace"); @@ -386,13 +388,13 @@ var LargeVideo = { // or null. this.eventEmitter.emit(UIEvents.SELECTED_ENDPOINT, resourceJid); } - if (RTCBrowserType.isSafari()) { - // FIXME In Safari fadeOut works only for the first time - changeVideo(this.isLargeVideoVisible()); - } else { - $('#largeVideo').fadeOut(300, - changeVideo.bind($('#largeVideo'), this.isLargeVideoVisible())); - } + // We are doing fadeOut/fadeIn animations on parent div which wraps + // largeVideo, because when Temasys plugin is in use it replaces + //