From 93507cc777a024cb0527c9cfbed4486c9b2ee599 Mon Sep 17 00:00:00 2001
From: Matt Dunn-Rankin <mattdunnrankin@khanacademy.org>
Date: Wed, 22 Apr 2020 16:36:08 -0700
Subject: [PATCH] oh oops, use gql for search prototype too!

---
 src/WardrobePage.js | 54 +++++++++++++++++++++++++++++++++++----------
 1 file changed, 42 insertions(+), 12 deletions(-)

diff --git a/src/WardrobePage.js b/src/WardrobePage.js
index 13b8e40..876b683 100644
--- a/src/WardrobePage.js
+++ b/src/WardrobePage.js
@@ -21,9 +21,10 @@ import {
   useToast,
 } from "@chakra-ui/core";
 
-import ItemList, { ItemListSkeleton } from "./ItemList";
-import useOutfitState from "./useOutfitState.js";
 import { ITEMS } from "./data";
+import ItemList, { ItemListSkeleton } from "./ItemList";
+import useItemData from "./useItemData";
+import useOutfitState from "./useOutfitState.js";
 
 function WardrobePage() {
   const { loading, error, data, wearItem } = useOutfitState();
@@ -174,20 +175,47 @@ function SearchToolbar({ query, onChange }) {
 }
 
 function SearchPanel({ query, wornItemIds, onWearItem }) {
+  const { loading, error, itemsById } = useItemData(ITEMS.map((i) => i.id));
+
   const normalize = (s) => s.toLowerCase();
-  const results = ITEMS.filter((item) =>
+  const results = Object.values(itemsById).filter((item) =>
     normalize(item.name).includes(normalize(query))
   );
   results.sort((a, b) => a.name.localeCompare(b.name));
 
-  const resultsSection =
-    results.length > 0 ? (
-      <ItemList
-        items={results}
+  return (
+    <Box color="green.800">
+      <Heading1 mb="6">Searching for "{query}"</Heading1>
+      <SearchResults
+        loading={loading}
+        error={error}
+        results={results}
         wornItemIds={wornItemIds}
         onWearItem={onWearItem}
       />
-    ) : (
+    </Box>
+  );
+}
+
+function SearchResults({ loading, error, results, wornItemIds, onWearItem }) {
+  if (loading) {
+    return <ItemListSkeleton />;
+  }
+
+  if (error) {
+    return (
+      <Text color="green.500">
+        We hit an error trying to load your search results
+        <span role="img" aria-label="(sweat emoji)">
+          😓
+        </span>{" "}
+        Try again?
+      </Text>
+    );
+  }
+
+  if (results.length === 0) {
+    return (
       <Text color="green.500">
         We couldn't find any matching items{" "}
         <span role="img" aria-label="(thinking emoji)">
@@ -196,12 +224,14 @@ function SearchPanel({ query, wornItemIds, onWearItem }) {
         Try again?
       </Text>
     );
+  }
 
   return (
-    <Box color="green.800">
-      <Heading1 mb="6">Searching for "{query}"</Heading1>
-      {resultsSection}
-    </Box>
+    <ItemList
+      items={results}
+      wornItemIds={wornItemIds}
+      onWearItem={onWearItem}
+    />
   );
 }