From 51bd46b8716b9a84cff13834408beb2b9b5aaa28 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?J=C3=A9r=C3=A9mie=20Astori?= <jeremie@astori.fr>
Date: Tue, 22 Aug 2017 20:18:34 -0400
Subject: [PATCH] Fix time placeholder showing on mobile format for condensed
 messages

As an extra, make sure time placeholder cannot be selected anymore, and do not do an extra call to `tz` helper when time is not relevant/displayed.

I tried to entirely remove `time`/`from` blocks, but because of flexbox layout, that broke message alignment...
---
 client/css/style.css           | 13 +++++++++----
 client/views/msg_condensed.tpl |  4 ++--
 2 files changed, 11 insertions(+), 6 deletions(-)

diff --git a/client/css/style.css b/client/css/style.css
index 6844fc9e..934f3205 100644
--- a/client/css/style.css
+++ b/client/css/style.css
@@ -837,6 +837,10 @@ kbd {
 	display: none;
 }
 
+#chat .condensed .time {
+	visibility: hidden;
+}
+
 #windows .header .topic,
 .messages .msg,
 .sidebar {
@@ -2043,6 +2047,11 @@ part/quit messages where we don't load previews (adds a blank line otherwise) */
 		padding: 0;
 	}
 
+	#chat .condensed .time,
+	#chat .condensed .from {
+		display: none;
+	}
+
 	#chat .date-marker,
 	#chat .unread-marker {
 		margin: 0;
@@ -2181,7 +2190,3 @@ part/quit messages where we don't load previews (adds a blank line otherwise) */
 #chat table.channel-list .topic {
 	white-space: pre-wrap;
 }
-
-.hide-text {
-	color: transparent !important;
-}
diff --git a/client/views/msg_condensed.tpl b/client/views/msg_condensed.tpl
index 914ef27c..e2becd66 100644
--- a/client/views/msg_condensed.tpl
+++ b/client/views/msg_condensed.tpl
@@ -1,5 +1,5 @@
-<div class="msg {{type}} closed" data-time="{{time}}">
-	<span class="time hide-text">{{tz time}}</span>
+<div class="msg condensed closed" data-time="{{time}}">
+	<span class="time">{{tz time}}</span>
 	<span class="from"></span>
 	<span class="content">
 		<span class="condensed-text"></span>