Browse Source

Merge pull request #1562 from virtuacoplenny/lenny/filmstrip-disable-hiding

fix(filmstrip): Disable keyboard shortcut for hiding videos
j8
yanas 8 years ago
parent
commit
e1a9487896
1 changed files with 7 additions and 6 deletions
  1. 7
    6
      modules/UI/videolayout/Filmstrip.js

+ 7
- 6
modules/UI/videolayout/Filmstrip.js View File

15
         this.filmstripContainerClassName = 'filmstrip';
15
         this.filmstripContainerClassName = 'filmstrip';
16
         this.filmstrip = $('#remoteVideos');
16
         this.filmstrip = $('#remoteVideos');
17
         this.eventEmitter = eventEmitter;
17
         this.eventEmitter = eventEmitter;
18
-        this._initFilmstripToolbar();
19
-        this.registerListeners();
18
+
19
+        // Show the toggle button and add event listeners only when out of
20
+        // filmstrip only mode.
21
+        if (!interfaceConfig.filmStripOnly) {
22
+            this._initFilmstripToolbar();
23
+            this.registerListeners();
24
+        }
20
     },
25
     },
21
 
26
 
22
     /**
27
     /**
23
      * Initializes the filmstrip toolbar.
28
      * Initializes the filmstrip toolbar.
24
      */
29
      */
25
     _initFilmstripToolbar() {
30
     _initFilmstripToolbar() {
26
-        // Do not show the toggle button in filmstrip only mode.
27
-        if (interfaceConfig.filmStripOnly)
28
-            return;
29
-
30
         let toolbarContainerHTML = this._generateToolbarHTML();
31
         let toolbarContainerHTML = this._generateToolbarHTML();
31
         let className = this.filmstripContainerClassName;
32
         let className = this.filmstripContainerClassName;
32
         let container = document.querySelector(`.${className}`);
33
         let container = document.querySelector(`.${className}`);

Loading…
Cancel
Save