From 031f2dfeb8eb5513d066422f6e4713875b9dec65 Mon Sep 17 00:00:00 2001
From: yanas <yana@jitsi.org>
Date: Thu, 6 Apr 2017 13:28:47 -0500
Subject: [PATCH] Fixes showToolbar in filmstrip-only mode and renames some
 funcs

---
 modules/UI/UI.js                    |  4 +--
 modules/UI/videolayout/FilmStrip.js | 42 +++++++++++++++--------------
 2 files changed, 24 insertions(+), 22 deletions(-)

diff --git a/modules/UI/UI.js b/modules/UI/UI.js
index bc4cb47b1..ab8f31c9a 100644
--- a/modules/UI/UI.js
+++ b/modules/UI/UI.js
@@ -322,8 +322,8 @@ UI.start = function () {
         SidePanels.init(eventEmitter);
     } else {
         $("body").addClass("filmstrip-only");
-        UIUtil.setVisible('mainToolbarContainer', false);
-        FilmStrip.setupFilmStripOnly();
+        UI.showToolbar();
+        FilmStrip.setFilmStripOnly();
         messageHandler.enableNotifications(false);
         JitsiPopover.enabled = false;
     }
diff --git a/modules/UI/videolayout/FilmStrip.js b/modules/UI/videolayout/FilmStrip.js
index 06a6163e6..4e75d5f0c 100644
--- a/modules/UI/videolayout/FilmStrip.js
+++ b/modules/UI/videolayout/FilmStrip.js
@@ -20,36 +20,38 @@ const FilmStrip = {
     },
 
     /**
-     * Initializes the filmstrip toolbar
+     * Initializes the filmstrip toolbar.
      */
     _initFilmStripToolbar() {
-        let toolbar = this._generateFilmStripToolbar();
+        // Do not show the toggle button in film strip only mode.
+        if (interfaceConfig.filmStripOnly)
+            return;
+
+        let toolbarContainerHTML = this._generateToolbarHTML();
         let className = this.filmStripContainerClassName;
         let container = document.querySelector(`.${className}`);
 
-        UIUtil.prependChild(container, toolbar);
+        UIUtil.prependChild(container, toolbarContainerHTML);
 
-        let iconSelector = '#hideVideoToolbar i';
+        let iconSelector = '#toggleFilmStripButton i';
         this.toggleFilmStripIcon = document.querySelector(iconSelector);
     },
 
     /**
-     * Generates HTML layout for filmstrip toolbar
+     * Generates HTML layout for filmstrip toggle button and wrapping container.
      * @returns {HTMLElement}
      * @private
      */
-    _generateFilmStripToolbar() {
+    _generateToolbarHTML() {
         let container = document.createElement('div');
         let isVisible = this.isFilmStripVisible();
         container.className = 'filmstrip__toolbar';
-        if(!interfaceConfig.filmStripOnly) {
-            container.innerHTML = `
-                <button id="hideVideoToolbar">
-                    <i class="icon-menu-${isVisible ? 'down' : 'up'}">
-                    </i>
-                </button>
-            `;
-        }
+        container.innerHTML = `
+            <button id="toggleFilmStripButton">
+                <i class="icon-menu-${isVisible ? 'down' : 'up'}">
+                </i>
+            </button>
+        `;
 
         return container;
     },
@@ -62,7 +64,7 @@ const FilmStrip = {
         // Firing the event instead of executing toggleFilmstrip method because
         // it's important to hide the filmstrip by UI.toggleFilmstrip in order
         // to correctly resize the video area.
-        $('#hideVideoToolbar').on('click',
+        $('#toggleFilmStripButton').on('click',
             () => this.eventEmitter.emit(UIEvents.TOGGLE_FILM_STRIP));
 
         this._registerToggleFilmstripShortcut();
@@ -162,11 +164,11 @@ const FilmStrip = {
         return !this.filmStrip.hasClass('hidden');
     },
 
-    setupFilmStripOnly() {
-        this.filmStrip.css({
-            padding: "0px 0px 18px 0px",
-            right: 0
-        });
+    /**
+     * Adjusts styles for film-strip only mode.
+     */
+    setFilmStripOnly() {
+        this.filmStrip.addClass('filmstrip__videos-filmstripOnly');
     },
 
     /**