Add search clearing (#29)

Add search clearing
palmerhq · Apr 27, 2019 · 5b4ddcc · 5b4ddcc
2 parents be5dfec + 66af3e9
commit 5b4ddcc
Show file tree

Hide file tree

Showing 3 changed files with 28 additions and 9 deletions.
diff --git a/site/src/components/Header.js b/site/src/components/Header.js
@@ -4,12 +4,13 @@ import { Close } from 'icons/Close'
 import { ExternalLink } from 'icons/ExternalLink'
 import React from 'react'
 import { onClose } from '../providers/WidgetProvider'
+import { theme } from '../styles/theme'
 import { AnchorButton } from './Button/AnchorButton'
 import { Button } from './Button/Button'
 
 export const Header = ({
   primaryColor: [red, green, blue],
-  onSearchChange,
+  setSearchValue,
   logoSrc,
   homepage,
   htmlUrl,
@@ -63,6 +64,7 @@ export const Header = ({
           borderRadius: 34,
           flexGrow: 1,
           marginLeft: '0.5rem',
+          position: 'relative',
         }}
       >
         <input
@@ -74,7 +76,7 @@ export const Header = ({
               : `rgba(${red}, ${green}, ${blue}, 0.9)`,
             border: '1px solid transparent',
             borderRadius: '4px',
-            padding: '0.25rem 0.75rem',
+            padding: '0.25rem 2rem 0.25rem 0.75rem',
             WebkitAppearance: 'none',
             '::placeholder': {
               color: 'white',
@@ -88,8 +90,27 @@ export const Header = ({
               },
             },
           }}
-          onChange={onSearchChange}
+          value={searchValue}
+          onChange={({ target: { value } }) => setSearchValue(value)}
         />
+        {searchValue && (
+          <Button
+            onClick={() => setSearchValue('')}
+            css={{
+              position: 'absolute',
+              marginRight: '0.25rem',
+              borderRadius: '50%',
+              padding: '0.25rem',
+              right: 0,
+              ':hover': {
+                background: 'initial',
+              },
+            }}
+          >
+            <Close css={{ color: theme.color.accent, width: 22, height: 22 }} />
+            <VisuallyHidden>Clear search</VisuallyHidden>
+          </Button>
+        )}
       </div>
 
       {isWidget ? (

diff --git a/site/src/templates/ReleaseTemplate.js b/site/src/templates/ReleaseTemplate.js
@@ -47,8 +47,7 @@ const ReleaseTemplate = ({
       mark.current.mark(value)
     }, 100)
 
-    search.current = event => {
-      const value = event.target.value
+    search.current = value => {
       setSearchValue(value)
       debouncedMark(value)
     }
@@ -103,7 +102,7 @@ const ReleaseTemplate = ({
         htmlUrl={htmlUrl}
         logoSrc={logoSrc}
         primaryColor={primaryColor}
-        onSearchChange={getReleaseSearch()}
+        setSearchValue={getReleaseSearch()}
         searchValue={searchValue}
       />
       <SiteWrapper>

diff --git a/site/src/templates/ReleasesTemplate.js b/site/src/templates/ReleasesTemplate.js
@@ -55,8 +55,7 @@ const ReleasesTemplate = ({
       setReleases(!!value ? releaseSearch.search(value) : edges)
     }, 100)
 
-    search.current = event => {
-      const value = event.target.value
+    search.current = value => {
       setSearchValue(value)
       debouncedReleaseSearch(value)
     }
@@ -120,7 +119,7 @@ const ReleasesTemplate = ({
       <Header
         homepage={homepage}
         htmlUrl={htmlUrl}
-        onSearchChange={getReleaseSearch()}
+        setSearchValue={getReleaseSearch()}
         searchValue={searchValue}
         logoSrc={logoSrc}
         primaryColor={primaryColor}