fix loading state when outfit changes

I'm not sure when this regressed, but changing the outfit was clearing out the whole preview and showing an empty loading state, instead of the intended behavior of showing a loading spinner over the old preview. This affected both the home page and the wardrobe page.

Yeah, so, huh. Fixed! I hope I didn't goof anything else with these effect trigger changes though 😅

The reason I'm doing this now is not just that it's annoying, but as a pair with the color change fix from just now, I want those color changes feeling buttery smooth!

src/app/components/OutfitPreview.js

@@ -232,7 +232,23 @@ export function usePreloadLayers(layers) {
   const [error, setError] = React.useState(null);
   const [loadedLayers, setLoadedLayers] = React.useState([]);
 
+  // NOTE: This condition would need to change if we started loading one at a
+  // time, or if the error case would need to show a partial state!
+  const loading = loadedLayers !== layers;
+
   React.useEffect(() => {
+    // HACK: Don't clear the preview when we have zero layers, because it
+    // usually means the parent is still loading data. I feel like this isn't
+    // the right abstraction, though...
+    if (loadedLayers.length > 0 && layers.length === 0) {
+      return;
+    }
+
+    // If the layers already match, we can ignore extra effect triggers.
+    if (!loading) {
+      return;
+    }
+
     let canceled = false;
     setError(null);
 
@@ -258,11 +274,7 @@ export function usePreloadLayers(layers) {
     return () => {
       canceled = true;
     };
-  }, [layers]);
+  }, [layers, loadedLayers.length]);
-
-  // NOTE: This condition would need to change if we started loading one at a
-  // time, or if the error case would need to show a partial state!
-  const loading = loadedLayers !== layers;
 
   return { loading, error, loadedLayers };
 }