From 65f3654878ca1dc31c646b5099d1481f1f73de71 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?garlicOS=C2=AE?=
 <14845347+Grosserly@users.noreply.github.com>
Date: Wed, 23 Sep 2020 17:56:30 -0500
Subject: [PATCH 1/2] (#289) Remove black outlines on focus

By default, most browsers place a black outline around the currently-focused element. On mstacm.org, it looks kind of ugly, so some people want it changed. As a quick fix, this commit completely up removes the black outline around focused elements.
People who use Tab to navigate  webpages aren't going to be too happy though, as it's their often only indication of what their cursor is currently on. So I'm hoping we can eventually come up with something else that adds that funcionality back in the future.
---
 apps/web/src/static/css/App.css | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/apps/web/src/static/css/App.css b/apps/web/src/static/css/App.css
index b0df68b0..a154c93e 100644
--- a/apps/web/src/static/css/App.css
+++ b/apps/web/src/static/css/App.css
@@ -44,4 +44,8 @@ body {
   margin: 0;
   padding: 0;
   font-family: "Nunito Sans", sans-serif;
-}
\ No newline at end of file
+}
+
+:focus {
+  outline: none;
+}

From 3504e82621522fdc844de4572b105c59ca74e25a Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?garlicOS=C2=AE?=
 <14845347+Grosserly@users.noreply.github.com>
Date: Wed, 23 Sep 2020 18:00:49 -0500
Subject: [PATCH 2/2] Add comments

---
 apps/web/src/static/css/App.css | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/apps/web/src/static/css/App.css b/apps/web/src/static/css/App.css
index a154c93e..9827f87b 100644
--- a/apps/web/src/static/css/App.css
+++ b/apps/web/src/static/css/App.css
@@ -46,6 +46,8 @@ body {
   font-family: "Nunito Sans", sans-serif;
 }
 
+/* Issue #289 - Outlines on buttons and textfields */
+/* TODO: Add back a visual focus indicator for accessibility's sake */
 :focus {
   outline: none;
 }