From db4e62b6934fece8cfa69f311d26b4ace343af15 Mon Sep 17 00:00:00 2001
From: Mattias Erming <mattias@mattiaserming.com>
Date: Wed, 19 Mar 2014 00:08:11 +0100
Subject: [PATCH] More improvements to the rendering

---
 client/index.html | 21 ++++++++++++---
 client/js/chat.js | 68 ++++++++++++++++++++++++++++++-----------------
 2 files changed, 61 insertions(+), 28 deletions(-)

diff --git a/client/index.html b/client/index.html
index e476636..45f4882 100644
--- a/client/index.html
+++ b/client/index.html
@@ -58,7 +58,12 @@
 	<script type="text/html" id="network">
 	{{#networks}}
 	<div class="network list-group" data-id="{{id}}">
-		{{> channels}}
+		{{#channels}}
+		<a href="{{name}}" class="channel list-group-item" data-id="{{id}}" data-name="{{name}}">
+			<span class="badge pull-right"></span>
+			{{name}}
+		</a>
+		{{/channels}}
 	</div>
 	{{/networks}}
 	</script>
@@ -84,10 +89,20 @@
 			<button class="close btn btn-danger btn-sm ">Leave</button>
 		</div>
 		<div class="users">
-			{{> users}}
+			{{#users}}
+			<a href="{{name}}" class="user">
+				{{mode}}{{name}}
+			</a>
+			{{/users}}
 		</div>
 		<div class="messages">
-			{{> messages}}
+			{{#messages}}
+			<div class="message {{type}}">
+				<span class="time">{{time}}</span>
+				<a href="{{user}}" class="user">{{mode}}{{user}}</a>
+				<span class="text">{{text}}</span>
+			</div>
+			{{/messages}}
 		</div>
 		<form onSubmit="return false;">
 			<input type="text" class="input" data-target="{{id}}"/>
diff --git a/client/js/chat.js b/client/js/chat.js
index 390f344..fc0ccdc 100644
--- a/client/js/chat.js
+++ b/client/js/chat.js
@@ -8,7 +8,6 @@ $(function() {
 		"USERS"
 	], function(i, type) {
 		socket.on(type, function(data) {
-			console.log(data);
 			render(type, data);
 		});
 	});
@@ -27,18 +26,16 @@ $(function() {
 		switch (type) {
 
 		case "NETWORKS":
-			var windows = chat
-				.find("#windows")
-				.html("");
+			var html = "";
 			data.forEach(function(network) {
-				windows.append(Mustache.render(window_tpl, network, {
-					users: user_tpl,
-					messages: message_tpl
-				}));
+				html += Mustache.render(window_tpl, network);
 			});
+
+			$("#windows")[0].innerHTML = html;
+
 			sidebar.find("#list").html(
-				Mustache.render(network_tpl, {networks: data}, {
-					channels: channel_tpl
+				Mustache.render(network_tpl, {
+					networks: data
 				})
 			).find(".channel")
 				.first()
@@ -51,23 +48,35 @@ $(function() {
 			break;
 
 		case "CHANNELS":
+			var target = data.target;
 			if (data.action == "remove") {
-				chat.find(".window[data-id='" + data.data.id + "']").remove();
-				sidebar.find(".channel[data-id='" + data.data.id + "']").remove();
+				$("[data-id='" + data.data.id + "']").remove();
 				return;
 			}
 
-			sidebar.find(".network[data-id='" + data.target + "']").append(
-				Mustache.render(channel_tpl, {channels: data.data}, {
-					channels: channel_tpl
+			var network = sidebar
+				.find(".network")
+				.find(".channel")
+				.removeClass("active")
+				.end();
+
+			network = network.filter("[data-id='" + data.target + "']").append(
+				Mustache.render(channel_tpl, {
+					channels: data.data
 				})
-			);
-			chat.find("#windows").append(
-				Mustache.render(window_tpl, {channels: data.data}, {
-					users: user_tpl,
-					messages: message_tpl
+			).find(".channel")
+				.last()
+				.addClass("active");
+
+			$("#windows").append(
+				Mustache.render(window_tpl, {
+					channels: data.data
 				})
-			);
+			).find(".window")
+				.last()
+				.bringToTop()
+				.find(".messages")
+				.sticky();
 			break;
 
 		case "USERS":
@@ -77,7 +86,9 @@ $(function() {
 			}
 
 			target = target.find(".users");
-			target.html(Mustache.render(user_tpl, {users: data.data}));
+			target.html(Mustache.render(user_tpl, {
+				users: data.data
+			}));
 			break;
 
 		case "MESSAGES":
@@ -92,7 +103,9 @@ $(function() {
 			}
 
 			target = target.find(".messages");
-			target.append(Mustache.render(message_tpl, {messages: message}));
+			target.append(Mustache.render(message_tpl, {
+				messages: message
+			}));
 			break;
 
 		}
@@ -141,6 +154,10 @@ $(function() {
 		}
 	});
 
+	chat.on("focus", "input[type=text]", function() {
+		$(this).closest(".window").find(".messages").scrollToBottom();
+	});
+
 	chat.on("click", ".close", function() {
 		var btn = $(this);
 		btn.prop("disabled", true);
@@ -185,8 +202,9 @@ $(function() {
 	$.fn.bringToTop = function() {
 		return this.css('z-index', highest++)
 			.addClass("active")
-			.find(".input")
-			.end()
+			//.find(".input")
+			//.focus()
+			//.end()
 			.siblings()
 			.removeClass("active")
 			.end();