fix item remove button in item search

Huh, weird how we seem to need preventDefault for buttons, but stopPropagation for links? idk, in any case, the Remove button was doing that thing where it clicks Remove but also bubbles up to the container and clicks it too :p

src/app/WardrobePage/Item.js

@@ -59,7 +59,10 @@ function Item({ item, itemNameId, isWorn, isInOutfit, dispatchToOutfit }) {
             <ItemActionButton
               icon={<EditIcon />}
               label="Support"
-              onClick={() => setSupportDrawerIsOpen(true)}
+              onClick={(e) => {
+                setSupportDrawerIsOpen(true);
+                e.preventDefault();
+              }}
             />
           </SupportOnly>
           <ItemActionButton
@@ -68,14 +71,16 @@ function Item({ item, itemNameId, isWorn, isInOutfit, dispatchToOutfit }) {
             href={`https://impress.openneo.net/items/${
               item.id
             }-${item.name.replace(/ /g, "-")}`}
+            onClick={(e) => e.stopPropagation()}
           />
           {isInOutfit && (
             <ItemActionButton
               icon={<DeleteIcon />}
               label="Remove"
-              onClick={() =>
+              onClick={(e) => {
-                dispatchToOutfit({ type: "removeItem", itemId: item.id })
+                dispatchToOutfit({ type: "removeItem", itemId: item.id });
-              }
+                e.preventDefault();
+              }}
             />
           )}
         </Box>
@@ -233,10 +238,7 @@ function ItemActionButton({ icon, label, href, onClick }) {
         color="gray.400"
         href={href}
         target={href ? "_blank" : null}
-        onClick={(e) => {
+        onClick={onClick}
-          if (onClick) onClick();
-          e.stopPropagation();
-        }}
         className={css`
           opacity: 0;
           transition: all 0.2s;