From 4a90e6dc712caffc28055d12dada95325a86868e Mon Sep 17 00:00:00 2001 From: Leonard Kim Date: Thu, 2 Nov 2017 09:22:33 -0700 Subject: [PATCH] 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. --- modules/UI/videolayout/LocalVideo.js | 2 +- react/features/filmstrip/components/Filmstrip.web.js | 1 + 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/modules/UI/videolayout/LocalVideo.js b/modules/UI/videolayout/LocalVideo.js index 2e5233f25..3c1d77c20 100644 --- a/modules/UI/videolayout/LocalVideo.js +++ b/modules/UI/videolayout/LocalVideo.js @@ -22,7 +22,7 @@ function LocalVideo(VideoLayout, emitter) { this.container = this.createContainer(); this.$container = $(this.container); - $('#filmstripLocalVideo').append(this.container); + $('#filmstripLocalVideoThumbnail').append(this.container); this.localVideoId = null; this.bindHoverHandler(); diff --git a/react/features/filmstrip/components/Filmstrip.web.js b/react/features/filmstrip/components/Filmstrip.web.js index 5f5d9326f..6b34dfccd 100644 --- a/react/features/filmstrip/components/Filmstrip.web.js +++ b/react/features/filmstrip/components/Filmstrip.web.js @@ -111,6 +111,7 @@ class Filmstrip extends Component<*> { onMouseOut = { this._onMouseOut } onMouseOver = { this._onMouseOver }> { this.props.filmstripOnly ? null : } +