fix: various issues (#124)

common/common.scss

@@ -551,10 +551,10 @@ a.curriculum-nav:focus {
 }
 
 // theme specific --------------------
-.btn:hover d-icon,
+.btn:hover .d-icon,
 .btn .d-icon,
 .btn-primary .d-icon {
-  color: inherit;
+  color: inherit $i;
 }
 
 .d-header-icons .badge-notification {
@@ -678,7 +678,7 @@ td.poster-names a {
   }
 }
 
-.d-editor-button-bar button {
+.d-editor-button-bar .btn {
   background-color: transparent;
   border: $n;
   box-shadow: $n;
@@ -855,3 +855,20 @@ div.select-kit-header {
   background-color: $primary $i;
   color: $fcc-quaternary-background $i;
 }
+
+.search-menu-container .search-input {
+  // `.search-input` is a div that wraps around the input and the search button.
+  // This div already has an focus outline set by Discourse,
+  // so we need to remove ours to avoid having nested focus outlines.
+  #search-term:focus-visible {
+    outline: none $i;
+  }
+}
+
+.topic-body .contents details {
+  &:not([open]):hover,
+  &:not([open]):focus,
+  &:not([open]):focus-within {
+    background-color: $fcc-d-hover;
+  }
+}