Skip to content

Commit e281e0d

Browse files
ENH Move search form after "search options" button (#2091)
The search form was before the "search options" button in the tab order, which meant that after expanding the search form you had to tab backwards to actually access the form. Also moved the search icon as this is visually at the start of the search box, so makes sense for the markup to be first too.
1 parent 1529fea commit e281e0d

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

client/dist/js/bundle.js

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

client/src/components/Search/SearchBox.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -320,12 +320,12 @@ const SearchBox = ({
320320
<Label for={id} id={`${id}_label`} hidden>
321321
{i18n._t('Admin.SEARCH', 'Search')}
322322
</Label>
323+
<div className="icon font-icon-search" aria-hidden="true" />
323324
{ renderTags() }
324325
{ renderInput() }
325326
{ showEnter && renderEnterHint() }
326-
{ children }
327-
<div className="icon font-icon-search" aria-hidden="true" />
328327
{ showFilters && renderFilterButton() }
328+
{ children }
329329
{ hideable && renderHideButton() }
330330
</div>
331331
</ResizeAware>

0 commit comments

Comments
 (0)