From 3e1e76ac4d179d367e680b7101a2b09f7ba1a9c9 Mon Sep 17 00:00:00 2001
From: Kirk Scheibelhut <kjs@google.com>
Date: Tue, 3 Sep 2019 16:53:48 -0700
Subject: [PATCH] Change away usernames to be colorblind-friendly

'#888' still is light enough to work in dark mode, and apparently
the darker grey improves the contrast.
---
 js/client-chat.js   | 4 ++--
 js/client-topbar.js | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/js/client-chat.js b/js/client-chat.js
index 7f52f2d04..2ace18ae7 100644
--- a/js/client-chat.js
+++ b/js/client-chat.js
@@ -36,7 +36,7 @@
 				this.$chatAdd.html('<form><button name="login">Join chat</button></form>');
 				this.$chatbox = null;
 			} else {
-				var color = app.user.get('away') ? 'color:#AAA;' : BattleLog.hashColor(app.user.get('userid'));
+				var color = app.user.get('away') ? 'color:#888;' : BattleLog.hashColor(app.user.get('userid'));
 				this.$chatAdd.html('<form class="chatbox"><label style="' + color + '">' + BattleLog.escapeHTML(name) + ':</label> <textarea class="textbox" type="text" size="70" autocomplete="off"></textarea></form>');
 				this.$chatbox = this.$chatAdd.find('textarea');
 				this.$chatbox.autoResize({
@@ -1727,7 +1727,7 @@
 			text += (user.away ? ' data-away=true' : '') + (user.status ? ' data-status="' + BattleLog.escapeHTML(user.status) + '"' : '') + '>';
 			var group = user.group;
 			var details = Config.groups[group] || {type: 'user'};
-			var color = user.away ? 'color:#AAA;' : BattleLog.hashColor(userid);
+			var color = user.away ? 'color:#888;' : BattleLog.hashColor(userid);
 			text += '<em class="group' + (details.group === 2 ? ' staffgroup' : '') + '">' + BattleLog.escapeHTML(group) + '</em>';
 			if (details.type === 'leadership') {
 				text += '<strong><em style="' + color + '">' + BattleLog.escapeHTML(user.name) + '</em></strong>';
diff --git a/js/client-topbar.js b/js/client-topbar.js
index 4d3ecd6a4..19987cf8d 100644
--- a/js/client-topbar.js
+++ b/js/client-topbar.js
@@ -32,11 +32,11 @@
 			var name = ' ' + app.user.get('name');
 			var away = app.user.get('away');
 			var status = app.user.get('status');
-			var color = away ? 'color:#AAA;' : BattleLog.hashColor(app.user.get('userid'));
+			var color = away ? 'color:#888;' : BattleLog.hashColor(app.user.get('userid'));
 			if (!app.user.loaded) {
 				buf = '<button disabled>Loading...</button>';
 			} else if (app.user.get('named')) {
-				buf = '<span class="username" data-name="' + BattleLog.escapeHTML(name) + '"' + (away ? ' data-away="true"' : '') + (status ? 'data-status="' + BattleLog.escapeHTML(status) + '"' : '') + ' style="' + color + '"><i class="fa fa-user" style="color:' + (away ? '#AAA;' : '#779EC5') + '"></i> ' + BattleLog.escapeHTML(name) + '</span>';
+				buf = '<span class="username" data-name="' + BattleLog.escapeHTML(name) + '"' + (away ? ' data-away="true"' : '') + (status ? 'data-status="' + BattleLog.escapeHTML(status) + '"' : '') + ' style="' + color + '"><i class="fa fa-user" style="color:' + (away ? '#888;' : '#779EC5') + '"></i> ' + BattleLog.escapeHTML(name) + '</span>';
 			} else {
 				buf = '<button name="login">Choose name</button>';
 			}