From dda6de7602e43c74bbd84c0f1e4924f9e8e1dde3 Mon Sep 17 00:00:00 2001
From: Jan-Christoph Borchardt <hey@jancborchardt.net>
Date: Tue, 9 Jan 2018 23:25:28 +0100
Subject: [PATCH] Always show app-sidebar-trigger on top, fullscreen icon below

Signed-off-by: Jan-Christoph Borchardt <hey@jancborchardt.net>
---
 css/style.scss | 35 ++++++++++++++++++++++++-----------
 1 file changed, 24 insertions(+), 11 deletions(-)

diff --git a/css/style.scss b/css/style.scss
index 4be752c2b..b567a45da 100644
--- a/css/style.scss
+++ b/css/style.scss
@@ -454,12 +454,13 @@ video {
 
 #video-fullscreen {
 	position: absolute;
+	top: 44px;
 	right: 0;
 	z-index: 90;
 }
 
 #video-fullscreen.public {
-	top: 45px;
+	top: 89px;
 }
 
 #video-fullscreen,
@@ -601,19 +602,31 @@ video {
 	display: none !important;
 }
 
-/* As there is no header the fullscreen button can be moved to the top. */
-#app-content:-webkit-full-screen #video-fullscreen.public {
+/* As there is no header in fullscreen the buttons can be moved to the top. */
+#app-content:-webkit-full-screen #app-sidebar-trigger {
 	top: 0;
 }
-#app-content:-moz-full-screen #video-fullscreen.public {
+#app-content:-moz-full-screen #app-sidebar-trigger {
 	top: 0;
 }
-#app-content:-ms-fullscreen #video-fullscreen.public {
+#app-content:-ms-fullscreen #app-sidebar-trigger {
 	top: 0;
 }
-#app-content:fullscreen #video-fullscreen.public {
+#app-content:fullscreen #app-sidebar-trigger {
 	top: 0;
 }
+#app-content:-webkit-full-screen #video-fullscreen {
+	top: 44px;
+}
+#app-content:-moz-full-screen #video-fullscreen {
+	top: 44px;
+}
+#app-content:-ms-fullscreen #video-fullscreen {
+	top: 44px;
+}
+#app-content:fullscreen #video-fullscreen {
+	top: 44px;
+}
 
 .localmediaerror h2 {
 	color: red;
@@ -690,11 +703,7 @@ video {
 
 #app-sidebar-trigger {
 	position: fixed;
-	/* Although it would be desirable due to their complementary behaviour, the
-	 * trigger can not be placed at the same position as the close button
-	 * (top: 45px) due to the "Switch to fullscreen" icon shown during calls.
-	 * Set to 45px (header) + 44px (fullscreen icon). */
-	top: 89px;
+	top: 45px;
 	right: 0;
 	/* Higher than the z-index of the emptycontent */
 	z-index: 50;
@@ -750,10 +759,14 @@ video {
 /* As the header is hidden (except for the logo), move the fullscreen button and
  * sidebar to the top during calls */
 #body-public #app-content:not(.participants-1) #app-sidebar,
+#body-public #app-content:not(.participants-1) #app-sidebar-trigger,
 #body-public #app-content:not(.participants-1) #video-fullscreen {
 	top: 0;
 	z-index: 3000;
 }
+#body-public #app-content:not(.participants-1) #video-fullscreen {
+	top: 44px;
+}
 
 /**
  * Right sidebar