From 92eeba53922ff12a4dd80655b51e225c8f0ea396 Mon Sep 17 00:00:00 2001
From: Leonard Kim <leonard.h.kim@gmail.com>
Date: Wed, 10 May 2017 13:54:38 -0700
Subject: [PATCH] fix(filmstrip): Disable keyboard shortcut for hiding videos

---
 modules/UI/videolayout/Filmstrip.js | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)

diff --git a/modules/UI/videolayout/Filmstrip.js b/modules/UI/videolayout/Filmstrip.js
index b4becc795..1d32698e1 100644
--- a/modules/UI/videolayout/Filmstrip.js
+++ b/modules/UI/videolayout/Filmstrip.js
@@ -15,18 +15,19 @@ const Filmstrip = {
         this.filmstripContainerClassName = 'filmstrip';
         this.filmstrip = $('#remoteVideos');
         this.eventEmitter = eventEmitter;
-        this._initFilmstripToolbar();
-        this.registerListeners();
+
+        // Show the toggle button and add event listeners only when out of
+        // filmstrip only mode.
+        if (!interfaceConfig.filmStripOnly) {
+            this._initFilmstripToolbar();
+            this.registerListeners();
+        }
     },
 
     /**
      * Initializes the filmstrip toolbar.
      */
     _initFilmstripToolbar() {
-        // Do not show the toggle button in filmstrip only mode.
-        if (interfaceConfig.filmStripOnly)
-            return;
-
         let toolbarContainerHTML = this._generateToolbarHTML();
         let className = this.filmstripContainerClassName;
         let container = document.querySelector(`.${className}`);