maverick/docbrown
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Revert search placeholder text to just 'Search' (#4607)

Browse source
This commit is contained in:
Ben Halpern 2019-10-26 12:14:32 -04:00 • committed by GitHub
parent 1808fb5b2b
commit c0640ecbc6
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
4 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
app/javascript/src/components/Search/SearchForm.jsx
View file

@ -21,7 +21,7 @@ export const SearchForm = ({
type="text"
name="q"
id={searchBoxId}
placeholder="Search (Press '/' to focus)"
placeholder="Search"
autoComplete="off"
aria-label="search"
onKeyDown={onSearch}

2
app/javascript/src/components/Search/__tests__/__snapshots__/Search.test.jsx.snap
View file

@ -22,7 +22,7 @@ exports[`<Search /> renders properly 1`] = `
id="nav-search"
name="q"
onKeyDown={[Function]}
placeholder="Search (Press '/' to focus)"
placeholder="Search"
type="text"
value=""
/>

2
app/javascript/src/components/Search/__tests__/__snapshots__/SearchForm.test.jsx.snap
View file

@ -22,7 +22,7 @@ exports[`<SearchForm /> renders properly when given search functions and a searc
id="nav-search"
name="q"
onKeyDown={[MockFunction]}
placeholder="Search (Press '/' to focus)"
placeholder="Search"
type="text"
value="hello"
/>

2
app/views/layouts/_top_bar.html.erb
View file

@ -12,7 +12,7 @@
<div id="nav-search-form-root">
<div class="nav-search-form">
<form acceptCharset="UTF-8" method="get">
<input class="nav-search-form__input" type="text" name="q" id="nav-search" placeholder="Search (Press '/' to focus)" autoComplete="off" />
<input class="nav-search-form__input" type="text" name="q" id="nav-search" placeholder="Search" autoComplete="off" />
</form>
</div>
</div>