From b9aac7d8d2fe955df80f8a8c586ff40db8d70840 Mon Sep 17 00:00:00 2001 From: Matchu Date: Thu, 4 Feb 2021 22:23:34 -0800 Subject: [PATCH] Add "Search all items" placeholder to item search I was starting to write a Cypress test, and noticed there was no placeholder to use for searching, and I don't know how that escaped my notice for so long! I guess I commented it out for some reason, but I forget why, and this seems fine now! (Looks like we removed it when we added zone suggestions? Idk!) --- src/app/WardrobePage/SearchToolbar.js | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/app/WardrobePage/SearchToolbar.js b/src/app/WardrobePage/SearchToolbar.js index 456dce0..ebba126 100644 --- a/src/app/WardrobePage/SearchToolbar.js +++ b/src/app/WardrobePage/SearchToolbar.js @@ -270,8 +270,7 @@ function SearchToolbar({ )} inputProps={{ - // placeholder: "Search for items to add…", - "aria-label": "Search for items to add…", + placeholder: "Search all items…", focusBorderColor: focusBorderColor, value: query.value || "", ref: searchQueryRef,