use flex centering to fix firebox positioning bug

decaporg · Jan 3, 2018 · 794040a · 794040a
1 parent 98a0ba5
commit 794040a
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/src/components/Collection/Sidebar.css b/src/components/Collection/Sidebar.css
@@ -22,7 +22,6 @@
   & input {
     background-color: #eff0f4;
     border-radius: var(--borderRadius);
-    height: 36px; /* issue #907 */
     font-size: 14px;
     padding: 10px 6px 10px 32px;
     width: 100%;
@@ -37,9 +36,13 @@
 
   & .nc-icon {
     position: absolute;
+    top: 0;
     left: 6px;
     z-index: 2;
-    top: 9px; /* issue #907 */
+    height: 100%;
+    display: flex;
+    align-items: center;
+    pointer-events: none;
   }
 }
 

diff --git a/src/components/UI/Toggle/Toggle.css b/src/components/UI/Toggle/Toggle.css
@@ -19,7 +19,7 @@
   @apply(--dropShadowDeep);
   position: absolute;
   left: 0;
-  top: 0; /* Fix 907 */
+  top: 0;
   width: 20px;
   height: 20px;
   border-radius: 50%;