haskal
/
jiti-meet
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

ref(filmstrip): create an empty container for local filmstrip move (#3303) 

* ref(filmstrip): create an empty container for local filmstrip move

This might be necessary for tile view. To support making the
local video display at the end of remote videos while in tile
view, but separateed from scrollable remote videos, moving
the local video might be necessary. By creating an empty
container, there is a target for local video to move to.

* squash: rename id
This commit is contained in:
virtuacoplenny 2018-07-26 10:51:15 -07:00 committed by yanas
parent b8daf0a9f9
commit 34d1eb6768
3 changed files with 15 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
modules/UI/shared_video/SharedVideoThumb.js
View File

@ -50,10 +50,14 @@ SharedVideoThumb.prototype.createContainer = function(spanId) {
displayNameContainer.className = 'displayNameContainer';
container.appendChild(displayNameContainer);
const remotes = document.getElementById('filmstripRemoteVideosContainer');
const remoteVideosContainer
= document.getElementById('filmstripRemoteVideosContainer');
const localVideoContainer
= document.getElementById('localVideoTileViewContainer');
remoteVideosContainer.insertBefore(container, localVideoContainer);
return remotes.appendChild(container);
return container;
};
/**

8
modules/UI/videolayout/RemoteVideo.js
View File

@ -642,10 +642,14 @@ RemoteVideo.createContainer = function(spanId) {
<div class ='presence-label-container'></div>
<span class = 'remotevideomenu'></span>`;
const remotes = document.getElementById('filmstripRemoteVideosContainer');
const remoteVideosContainer
= document.getElementById('filmstripRemoteVideosContainer');
const localVideoContainer
= document.getElementById('localVideoTileViewContainer');
remoteVideosContainer.insertBefore(container, localVideoContainer);
return remotes.appendChild(container);
return container;
};
export default RemoteVideo;

4
react/features/filmstrip/components/web/Filmstrip.js
View File

@ -119,7 +119,9 @@ class Filmstrip extends Component <Props> {
className = 'remote-videos-container'
id = 'filmstripRemoteVideosContainer'
onMouseOut = { this._onMouseOut }
onMouseOver = { this._onMouseOver } />
onMouseOver = { this._onMouseOver }>
<div id = 'localVideoTileViewContainer' />
</div>
</div>
</div>
</div>