Browse Source

fix(LargeVideoManager): enable video problems filter on VideoContainer

Only the VideoContainer is interested in showing the video problems
filter which is meant to be displayed when ICE is disconnected.
j8
paweldomas 8 years ago
parent
commit
ecfc05bcc8

+ 0
- 4
modules/UI/shared_video/SharedVideo.js View File

@@ -567,10 +567,6 @@ class SharedVideoContainer extends LargeContainer {
567 567
         this.player = player;
568 568
     }
569 569
 
570
-    get $video () {
571
-        return this.$iframe;
572
-    }
573
-
574 570
     show () {
575 571
         let self = this;
576 572
         return new Promise(resolve => {

+ 1
- 2
modules/UI/videolayout/LargeVideoManager.js View File

@@ -213,8 +213,7 @@ export default class LargeVideoManager {
213 213
      * @param enable <tt>true</tt> to enable, <tt>false</tt> to disable
214 214
      */
215 215
     enableVideoProblemFilter (enable) {
216
-        let container = this.getContainer(this.state);
217
-        container.$video.toggleClass("videoProblemFilter", enable);
216
+        this.videoContainer.enableVideoProblemFilter(enable);
218 217
     }
219 218
 
220 219
     /**

+ 11
- 0
modules/UI/videolayout/VideoContainer.js View File

@@ -184,6 +184,17 @@ export class VideoContainer extends LargeContainer {
184 184
         this.$video[0].onplay = onPlay;
185 185
     }
186 186
 
187
+    /**
188
+     * Enables a filter on the video which indicates that there are some
189
+     * problems with the media connection.
190
+     *
191
+     * @param {boolean} enable <tt>true</tt> if the filter is to be enabled or
192
+     * <tt>false</tt> otherwise.
193
+     */
194
+    enableVideoProblemFilter (enable) {
195
+        this.$video.toggleClass("videoProblemFilter", enable);
196
+    }
197
+
187 198
     /**
188 199
      * Get size of video element.
189 200
      * @returns {{width, height}}

Loading…
Cancel
Save