feat: wrap search option bubbles if not enough space (#918)

standardnotes · Mar 10, 2022 · be6a9fd · be6a9fd
1 parent b31afee
commit be6a9fd
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 16 deletions.
diff --git a/app/assets/javascripts/components/Bubble.tsx b/app/assets/javascripts/components/Bubble.tsx
@@ -12,8 +12,7 @@ const styles = {
 
 const Bubble = ({ label, selected, onSelect }: BubbleProperties) => (
   <span
-    role="tab"
-    className={`bubble ${styles.base} ${
+    className={`${styles.base} ${
       selected ? styles.selected : styles.unselected
     }`}
     onClick={onSelect}

diff --git a/app/assets/javascripts/components/NotesView.tsx b/app/assets/javascripts/components/NotesView.tsx
@@ -182,7 +182,7 @@ export const NotesView: FunctionComponent<Props> = observer(
                 </button>
               </div>
               <div className="filter-section" role="search">
-                <div>
+                <div className="relative">
                   <input
                     type="text"
                     id="search-bar"

diff --git a/app/assets/javascripts/components/SearchOptions.tsx b/app/assets/javascripts/components/SearchOptions.tsx
@@ -19,11 +19,7 @@ export const SearchOptions = observer(({ appState }: Props) => {
   }
 
   return (
-    <div
-      role="tablist"
-      className="search-options justify-center"
-      onMouseDown={(e) => e.preventDefault()}
-    >
+    <div className="search-options" onMouseDown={(e) => e.preventDefault()}>
       <Bubble
         label="Protected Contents"
         selected={includeProtectedContents}

diff --git a/app/assets/stylesheets/_notes.scss b/app/assets/stylesheets/_notes.scss
@@ -52,7 +52,6 @@
 
   .filter-section {
     clear: left;
-    max-height: 80px;
     margin-top: 10px;
     position: relative;
     display: flex;
@@ -77,14 +76,11 @@
     .search-options {
       margin-top: 10px;
 
-      display: grid;
-      grid-template-columns: repeat( 3, 1fr );
-      gap: .5rem;
+      display: flex;
+      flex-wrap: wrap;
+      gap: 0.5rem;
 
       font-size: var(--sn-stylekit-font-size-p);
-      white-space: nowrap;
-
-      overflow-x: auto;
     }
 
     #search-clear-button {
@@ -100,7 +96,7 @@
       line-height: 17px;
       text-align: center;
       position: absolute;
-      top: 20%;
+      top: 50%;
       transform: translateY(-50%);
       right: 10px;
       cursor: pointer;