Bladeren bron

fix(filmstrip): Disable keyboard shortcut for hiding videos

j8
Leonard Kim 8 jaren geleden
bovenliggende
commit
92eeba5392
1 gewijzigde bestanden met toevoegingen van 7 en 6 verwijderingen
  1. 7
    6
      modules/UI/videolayout/Filmstrip.js

+ 7
- 6
modules/UI/videolayout/Filmstrip.js Bestand weergeven

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}`);

Laden…
Annuleren
Opslaan