Merge pull request #2368 from thelounge/astorije/window-header

Remove header border on non-chat windows and reduce top margin a bit more

client/css/style.css

@@ -179,7 +179,7 @@ kbd {
 }
 
 .container {
-	margin: 20px auto;
+	margin-bottom: 20px;
 	max-width: 480px;
 	touch-action: pan-y;
 }
@@ -469,7 +469,13 @@ kbd {
 	width: 36px;
 }
 
+/* Channel list button stays fixed when scrolling... */
 #viewport .lt {
+	position: fixed;
+}
+
+/* ... Except on chat windows, relative to include the notification dot */
+#viewport #chat .lt {
 	position: relative;
 }
 
@@ -876,7 +882,6 @@ kbd {
 }
 
 #windows .header {
-	border-bottom: 1px solid #e7e7e7;
 	line-height: 48px;
 	height: 48px;
 	padding: 0 6px;
@@ -885,6 +890,10 @@ kbd {
 	overflow: hidden;
 }
 
+#windows #chat .header {
+	border-bottom: 1px solid #e7e7e7;
+}
+
 #windows .header .title {
 	font-size: 15px;
 	padding-left: 6px;
@@ -2207,10 +2216,6 @@ part/quit messages where we don't load previews (adds a blank line otherwise) */
 		opacity: 0;
 	}
 
-	.container {
-		margin-top: 60px !important;
-	}
-
 	#sidebar button,
 	#sidebar .chan,
 	#sidebar .empty,
@@ -2287,6 +2292,11 @@ part/quit messages where we don't load previews (adds a blank line otherwise) */
 		display: flex;
 	}
 
+	/* On mobile display, channel list button stays at the top */
+	#viewport .lt {
+		position: relative;
+	}
+
 	#chat .userlist {
 		height: 100%;
 		position: absolute;
@@ -2312,7 +2322,7 @@ part/quit messages where we don't load previews (adds a blank line otherwise) */
 
 @media (max-width: 479px) {
 	.container {
-		margin: 10px 0 !important;
+		margin: 0;
 	}
 
 	#sign-in .btn {

client/themes/morning.css

@@ -32,7 +32,7 @@ body {
 
 /* Borders */
 #chat .content,
-#windows .header,
+#windows #chat .header,
 #chat .user-mode::before,
 #chat .userlist {
 	border-color: #2a323d;