From 17666eaef78f1ffab70c446b5bddae2ffdbeefdd Mon Sep 17 00:00:00 2001
From: Grant Woodford <eyesonlyhack@gmail.com>
Date: Sun, 15 Dec 2013 12:02:43 +0200
Subject: [PATCH] Changed to not use inline JavaScript

---
 src/static/js/chat.js  | 17 ++++++++++++-----
 src/templates/pad.html | 14 +-------------
 2 files changed, 13 insertions(+), 18 deletions(-)

diff --git a/src/static/js/chat.js b/src/static/js/chat.js
index ed01600fb..74df5fc27 100644
--- a/src/static/js/chat.js
+++ b/src/static/js/chat.js
@@ -54,11 +54,18 @@ var chat = (function()
     },
     hide: function () 
     {
-      $("#chatcounter").text("0");
-      $("#chaticon").show();
-      $("#chatbox").hide();
-      $.gritter.removeAll();
-      $("#gritter-notice-wrapper").show();
+      // decide on hide logic based on chat window being maximized or not 
+      if ($('#options-stickychat').prop('checked')) {
+        chat.stickToScreen();
+        $('#options-stickychat').prop('checked', false);
+      }
+      else {  
+        $("#chatcounter").text("0");
+        $("#chaticon").show();
+        $("#chatbox").hide();
+        $.gritter.removeAll();
+        $("#gritter-notice-wrapper").show();
+      }
     },
     scrollDown: function()
     {
diff --git a/src/templates/pad.html b/src/templates/pad.html
index eaa949dec..f6adfe9ce 100644
--- a/src/templates/pad.html
+++ b/src/templates/pad.html
@@ -390,7 +390,7 @@
 
         <div id="chatbox">
             <div id="titlebar"><span id ="titlelabel" data-l10n-id="pad.chat"></span>
-              <a id="titlecross" onClick="minimizeChatbox();">-&nbsp;</a>
+              <a id="titlecross" onClick="chat.hide();return false;">-&nbsp;</a>
               <a id="titlesticky" onClick="chat.stickToScreen(true);$('#options-stickychat').prop('checked', true);return false;" title="Stick chat to screen">█&nbsp;&nbsp;</a>
             </div>
             <div id="chattext" class="authorColors">
@@ -476,18 +476,6 @@
               padeditbar = require('ep_etherpad-lite/static/js/pad_editbar').padeditbar;
               padimpexp = require('ep_etherpad-lite/static/js/pad_impexp').padimpexp;
             }());
-
-            function minimizeChatbox()
-            {
-              if ($('#options-stickychat').prop('checked')) {
-                chat.stickToScreen();
-                $('#options-stickychat').prop('checked', false);
-              } else {
-                chat.hide();
-              }
-              
-              return false;
-            }	
         </script>
         <% e.end_block(); %>
 </html>