From 791ac667f0ae16f3d9ada3fc7c5121b382b384bc Mon Sep 17 00:00:00 2001
From: Lim Chee Aun <cheeaun@gmail.com>
Date: Tue, 24 Jan 2023 17:06:13 +0800
Subject: [PATCH] Style adjustments for Settings page

---
 src/pages/settings.css | 21 ++++++++-------------
 src/pages/settings.jsx |  4 ++--
 2 files changed, 10 insertions(+), 15 deletions(-)

diff --git a/src/pages/settings.css b/src/pages/settings.css
index 0f786504..2b940da9 100644
--- a/src/pages/settings.css
+++ b/src/pages/settings.css
@@ -14,13 +14,14 @@
 
 #settings-container :is(section, .section) {
   background-color: var(--bg-color);
-  margin: 0 -16px;
+  margin: 0;
   padding: 8px 16px;
   border-top: var(--hairline-width) solid var(--outline-color);
   border-bottom: var(--hairline-width) solid var(--outline-color);
+  border-radius: 8px;
 }
-#settings-container :is(section, .section) > li + li {
-  border-top: var(--hairline-width) solid var(--outline-color);
+#settings-container :is(section, .section) > li:last-child {
+  border-bottom: none;
 }
 
 #settings-container ul {
@@ -29,11 +30,12 @@
   list-style: none;
 }
 #settings-container ul li {
-  padding: 8px 0;
+  padding: 8px 0 16px;
   display: flex;
   justify-content: space-between;
   align-items: center;
   flex-wrap: wrap;
+  border-bottom: var(--hairline-width) solid var(--outline-color);
 }
 #settings-container ul li .current {
   margin-right: 8px;
@@ -44,14 +46,14 @@
   opacity: 1;
 }
 #settings-container ul li .current.is-current + .avatar {
-  box-shadow: 0 0 0 1.5px var(--green-color);
+  box-shadow: 0 0 0 1.5px var(--green-color), 0 0 8px var(--green-color);
 }
 #settings-container ul li > div {
   flex-grow: 1;
   max-width: 100%;
 }
 #settings-container ul li > div.actions {
-  flex-basis: min-content;
+  flex-basis: fit-content;
   margin-top: 8px;
 }
 #settings-container ul li > div:last-child {
@@ -98,10 +100,3 @@
 #settings-container .radio-group label:has(input:checked) input:checked + span {
   color: inherit;
 }
-
-@media (min-width: 40em) {
-  #settings-container :is(section, .section) {
-    margin-inline: 0;
-    border-radius: 8px;
-  }
-}
diff --git a/src/pages/settings.jsx b/src/pages/settings.jsx
index 524256ce..8b086384 100644
--- a/src/pages/settings.jsx
+++ b/src/pages/settings.jsx
@@ -92,7 +92,7 @@ function Settings({ onClose }) {
                         <Icon icon="transfer" /> Switch
                       </button>
                     )}
-                    <div>
+                    <span>
                       {!isDefault && moreThanOneAccount && (
                         <button
                           type="button"
@@ -128,7 +128,7 @@ function Settings({ onClose }) {
                           </button>
                         </>
                       )}
-                    </div>
+                    </span>
                   </div>
                 </li>
               );