From 8453f072057b7deedf297c55a5993a2dee38a20c Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Tristram=20Gr=C3=A4bener?= <tristramg@gmail.com>
Date: Mon, 15 Apr 2019 12:15:12 +0200
Subject: [PATCH] Chat bubble: by default hide in CSS

The current behaviour is to show the chat bubble and hide if chat is
disabled.

Because of this, the bubble appears wrongfully for a short time.

With this PR, by default it is hidden and displayed only if chat is
enabled.

Fixes: #3088
---
 src/static/css/pad.css | 1 +
 src/static/js/pad.js   | 2 +-
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/static/css/pad.css b/src/static/css/pad.css
index 18b936478..be8b1f0be 100644
--- a/src/static/css/pad.css
+++ b/src/static/css/pad.css
@@ -643,6 +643,7 @@ table#otheruserstable {
   border-top-right-radius: 5px;
   background-color: #fff;
   cursor: pointer;
+  display: none;
 }
 #chaticon a {
   text-decoration: none
diff --git a/src/static/js/pad.js b/src/static/js/pad.js
index b12058237..69fa65902 100644
--- a/src/static/js/pad.js
+++ b/src/static/js/pad.js
@@ -74,7 +74,7 @@ function randomString()
 var getParameters = [
   { name: "noColors",         checkVal: "true",  callback: function(val) { settings.noColors = true; $('#clearAuthorship').hide(); } },
   { name: "showControls",     checkVal: "false", callback: function(val) { $('#editbar').addClass('hideControlsEditbar'); $('#editorcontainer').addClass('hideControlsEditor'); } },
-  { name: "showChat",         checkVal: "false", callback: function(val) { $('#chaticon').hide(); } },
+  { name: "showChat",         checkVal: "true", callback: function(val) { $('#chaticon').show(); } },
   { name: "showLineNumbers",  checkVal: "false", callback: function(val) { settings.LineNumbersDisabled = true; } },
   { name: "useMonospaceFont", checkVal: "true",  callback: function(val) { settings.useMonospaceFontGlobal = true; } },
   // If the username is set as a parameter we should set a global value that we can call once we have initiated the pad.