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

fix(filmstrip): create a specific target for local video appending 

Instead of targetting a div that contains multiple elements
and risking the elements appearing out of order, create a
specific div for local video to append to.
This commit is contained in:
Leonard Kim 2017-11-02 09:22:33 -07:00 committed by yanas
parent 90070d9e9f
commit 4a90e6dc71
2 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
modules/UI/videolayout/LocalVideo.js
View File

@ -22,7 +22,7 @@ function LocalVideo(VideoLayout, emitter) {
this.container = this.createContainer(); this.container = this.createContainer();
this.$container = $(this.container); this.$container = $(this.container);
$('#filmstripLocalVideo').append(this.container); $('#filmstripLocalVideoThumbnail').append(this.container);
this.localVideoId = null; this.localVideoId = null;
this.bindHoverHandler(); this.bindHoverHandler();

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

@ -111,6 +111,7 @@ class Filmstrip extends Component<*> {
onMouseOut = { this._onMouseOut } onMouseOut = { this._onMouseOut }
onMouseOver = { this._onMouseOver }> onMouseOver = { this._onMouseOver }>
{ this.props.filmstripOnly ? null : <InviteButton /> } { this.props.filmstripOnly ? null : <InviteButton /> }
<div id = 'filmstripLocalVideoThumbnail' />
</div> </div>
<div <div
className = 'filmstrip__videos' className = 'filmstrip__videos'