From e67940245363b2569c61ea12f1290790645c6387 Mon Sep 17 00:00:00 2001
From: Art Cancro <ajc@citadel.org>
Date: Fri, 23 Feb 2018 23:49:42 -0500
Subject: [PATCH] css tuning

---
 webcit-ng/static/css/webcit.css | 10 ++++++++++
 webcit-ng/static/js/views.js    | 19 ++++++++++++-------
 2 files changed, 22 insertions(+), 7 deletions(-)

diff --git a/webcit-ng/static/css/webcit.css b/webcit-ng/static/css/webcit.css
index 1bea8139b..6c157d1e7 100644
--- a/webcit-ng/static/css/webcit.css
+++ b/webcit-ng/static/css/webcit.css
@@ -18,6 +18,16 @@ div.ctdl-msg-wrapper
 	padding: 4px;		/* inside the box */
 }
 
+div.ctdl-avatar
+{
+	width: 32px;
+}
+
+div.ctdl-msg-content
+{
+	margin-left: 32px;
+}
+
 div.ctdl-avatar
 {
 	float: left;
diff --git a/webcit-ng/static/js/views.js b/webcit-ng/static/js/views.js
index fbc829902..4acd6b399 100644
--- a/webcit-ng/static/js/views.js
+++ b/webcit-ng/static/js/views.js
@@ -41,7 +41,8 @@ function render_room_view(gt_msg, lt_msg)
 			forum_readmessages("ctdl-main", gt_msg, lt_msg);
 			break;
 		default:
-			document.getElementById("ctdl-main").innerHTML = "The view for " + current_room + " is " + current_view + " but there is no renderer." ;
+			document.getElementById("ctdl-main").innerHTML =
+				"The view for " + current_room + " is " + current_view + " but there is no renderer." ;
 			break;
 	}
 
@@ -107,7 +108,10 @@ function forum_readmessages(target_div, gt_msg, lt_msg)
 				// Render the divs (we will fill them in later)
 				for (var i in msgs)
 				{
-					document.getElementById(target_div).innerHTML += "<div id=\"ctdl_msg_" + msgs[i] + "\">#" + msgs[i] + "</div>" ;
+					document.getElementById(target_div).innerHTML +=
+						"<div id=\"ctdl_msg_"
+						+ msgs[i] + "\">#" + msgs[i]
+						+ "</div>" ;
 				}
 				if (lt_msg == 9999999999)
 				{
@@ -122,8 +126,8 @@ function forum_readmessages(target_div, gt_msg, lt_msg)
 					}
 					document.getElementById(target_div).innerHTML +=
 						"<div id=\"" + new_new_div_name + "\">" +
-						"<a href=\"javascript:forum_readmessages('" + new_new_div_name + "', " + newgt + ", 9999999999);\">" +
-						"link to msgs greater than " + newgt + "</a></div>" ;
+						"<a href=\"javascript:forum_readmessages('" + new_new_div_name + "', " + newgt + ", 9999999999);\">"
+						+ "link to msgs greater than " + newgt + "</a></div>" ;
 				}
 
 				// Now figure out where to scroll to after rendering.
@@ -149,7 +153,8 @@ function forum_readmessages(target_div, gt_msg, lt_msg)
 			}
 			else
 			{
-				document.getElementById(target_div).innerHTML = original_text;		// this will make the link reappear so the user can try again
+				// if xhr fails, this will make the link reappear so the user can try again
+				document.getElementById(target_div).innerHTML = original_text;
 			}
 		}
 	};
@@ -187,8 +192,8 @@ function forum_render_one(div, msgnum, scroll_to)
 				+ "<div class=\"ctdl-avatar\">"					// begin avatar
 				+ "<i class=\"fa fa-user-circle fa-2x\"></i> "			// FIXME temporary avatar
 				+ "</div>"							// end avatar
-				+ "<div>"							// begin content
-				+ "<div>"							// begin header
+				+ "<div class=\"ctdl-msg-content\">"				// begin content
+				+ "<div class=\"ctdl-msg-header\">"				// begin header
 				+ "<span class=\"ctdl-username\"><a href=\"#\">"		// FIXME link to user profile
 				+ msg.from
 				+ "</a></span> "
-- 
2.30.2