From 076263c7f537777429a7effca4dfe06193d7d7a2 Mon Sep 17 00:00:00 2001
From: Jordan Vidrine <30537603+jordanvidrine@users.noreply.github.com>
Date: Fri, 28 Feb 2025 15:33:06 -0600
Subject: [PATCH] UX: More small fixes (#9)

---
 scss/main.scss      |  7 +++++++
 scss/misc.scss      | 16 ++++++++++++++++
 scss/nav-pills.scss |  8 +++++++-
 scss/topic.scss     |  8 +++++++-
 scss/variables.scss |  1 +
 5 files changed, 38 insertions(+), 2 deletions(-)

diff --git a/scss/main.scss b/scss/main.scss
index b4e5908..a8b2180 100644
--- a/scss/main.scss
+++ b/scss/main.scss
@@ -66,3 +66,10 @@ body:not(.has-full-page-chat) {
 aside.onebox {
   background-color: var(--d-content-background);
 }
+
+.d-editor-preview-wrapper {
+  box-shadow: 0px 0px 1px 2px #e2e8f8, 0px 0px 24px 4px #e2e8f8;
+  border-radius: var(--d-border-radius-large);
+  padding: 1em;
+  background-color: var(--d-content-background);
+}
diff --git a/scss/misc.scss b/scss/misc.scss
index 1e64202..f6a03aa 100644
--- a/scss/misc.scss
+++ b/scss/misc.scss
@@ -27,3 +27,19 @@
 .admin-detail {
   background-color: var(--d-content-background);
 }
+
+.select-kit.combo-box .select-kit-header,
+.select-kit.multi-select .multi-select-header,
+.d-editor-textarea-wrapper,
+.fk-d-menu__inner-content,
+.menu-panel {
+  background-color: var(--d-content-background);
+}
+
+.fk-d-menu .btn {
+  border-radius: 0;
+}
+
+.d-editor-button-bar {
+  padding: 3px;
+}
diff --git a/scss/nav-pills.scss b/scss/nav-pills.scss
index 59a3a80..f5e5731 100644
--- a/scss/nav-pills.scss
+++ b/scss/nav-pills.scss
@@ -13,7 +13,6 @@
     }
   }
   .combo-box .combo-box-header,
-  .select-kit.is-expanded .select-kit-body,
   .discourse-post-event .discourse-post-event-widget {
     // needs more specificy than just in the button file
     background-color: var(--secondary);
@@ -26,6 +25,13 @@
     border-color: transparent;
   }
 }
+
+.select-kit.is-expanded .select-kit-body {
+  border-radius: var(--d-border-radius);
+  border: 1px solid var(--accent-color);
+  background-color: var(--d-content-background);
+}
+
 .nav-pills > li > a:hover,
 .nav-pills > li button:hover {
   background: transparent;
diff --git a/scss/topic.scss b/scss/topic.scss
index 550239e..11b6f93 100644
--- a/scss/topic.scss
+++ b/scss/topic.scss
@@ -1,7 +1,7 @@
 .post-stream {
   .contents p {
     // font-size: var(--font-up-1);
-    line-height: 1.75;
+    line-height: 1.25;
   }
 }
 
@@ -68,3 +68,9 @@
 .discourse-post-event .discourse-post-event-widget {
   background: var(--d-content-background);
 }
+
+.more-topics__container .nav {
+  position: unset;
+  margin-block: unset;
+  margin-bottom: 1em;
+}
diff --git a/scss/variables.scss b/scss/variables.scss
index 9f5bf39..40caf4f 100644
--- a/scss/variables.scss
+++ b/scss/variables.scss
@@ -16,6 +16,7 @@
   --d-sidebar-section-link-icon-size: 1em;
   --d-sidebar-highlight-background: #ebecf9;
   --d-sidebar-active-background: #d8d9f3;
+  --d-input-bg-color: var(--d-content-background);
   // --d-sidebar-row-height: 2.8em;
   //the idea is: block spacing can grow with font-size, but inline spacing should not to maintain horizontal (text) alignment
   --spacing-block-xs: 0.25em;