From 31248336f7ec0ed3edd366c01a24c946d7cef647 Mon Sep 17 00:00:00 2001
From: Sendou <sendou77@gmail.com>
Date: Mon, 9 Mar 2020 15:44:26 +0200
Subject: [PATCH] random fixes

---
 .../src/components/freeagents/Posts.tsx       |  10 +-
 .../src/components/user/BuildFormModal.tsx    | 129 ++++++++++--------
 .../src/components/user/ProfileModal.tsx      |   7 +-
 graphql-schemas/build.js                      |   8 +-
 4 files changed, 83 insertions(+), 71 deletions(-)

diff --git a/frontend-react/src/components/freeagents/Posts.tsx b/frontend-react/src/components/freeagents/Posts.tsx
index 00a97dfc4..d35dca664 100644
--- a/frontend-react/src/components/freeagents/Posts.tsx
+++ b/frontend-react/src/components/freeagents/Posts.tsx
@@ -20,11 +20,7 @@ const Posts: React.FC<PostsAccordionProps> = ({ posts }) => {
   }
   return (
     <>
-      <Grid
-        gridGap="1em"
-        gridTemplateColumns="repeat(auto-fit, minmax(260px, 1fr))"
-        mt="1em"
-      >
+      <Grid gridTemplateColumns="repeat(auto-fit, minmax(260px, 1fr))" mt="1em">
         <InfiniteScroll
           pageStart={1}
           loadMore={page => setAgentsToShow(page * 10)}
@@ -33,7 +29,9 @@ const Posts: React.FC<PostsAccordionProps> = ({ posts }) => {
           {posts
             .filter((post, index) => index < agentsToShow)
             .map(post => (
-              <FreeAgentCard key={post.id} post={post} />
+              <Box my="1em">
+                <FreeAgentCard key={post.id} post={post} />
+              </Box>
             ))}
         </InfiniteScroll>
       </Grid>
diff --git a/frontend-react/src/components/user/BuildFormModal.tsx b/frontend-react/src/components/user/BuildFormModal.tsx
index a983d8d7e..af1ffd291 100644
--- a/frontend-react/src/components/user/BuildFormModal.tsx
+++ b/frontend-react/src/components/user/BuildFormModal.tsx
@@ -267,26 +267,29 @@ const BuildFormModal: React.FC<BuildFormModalProps> = ({
       )}
       <Box asFlex mt="1em" justifyContent="space-between" flexWrap="wrap">
         <Box asFlex flexDirection="column" alignItems="center">
-          <Select
-            label="Headgear"
-            options={headSelectOptions}
-            clearable
-            value={build.headgearItem}
-            setValue={(headgearItem: HeadGear) => {
-              if (
-                (!build.headgear ||
-                  build.headgear.every(ability => ability === "UNKNOWN")) &&
-                existingGear.hasOwnProperty(headgearItem)
-              ) {
-                handleChange({
-                  headgearItem,
-                  headgear: existingGear[headgearItem],
-                })
-              } else {
-                handleChange({ headgearItem })
-              }
-            }}
-          />
+          <Box minW="275px">
+            <Select
+              label="Headgear"
+              options={headSelectOptions}
+              clearable
+              isSearchable
+              value={build.headgearItem}
+              setValue={(headgearItem: HeadGear) => {
+                if (
+                  (!build.headgear ||
+                    build.headgear.every(ability => ability === "UNKNOWN")) &&
+                  existingGear.hasOwnProperty(headgearItem)
+                ) {
+                  handleChange({
+                    headgearItem,
+                    headgear: existingGear[headgearItem],
+                  })
+                } else {
+                  handleChange({ headgearItem })
+                }
+              }}
+            />
+          </Box>
           {build.headgearItem && (
             <Box mt="0.5em">
               <GearImage englishName={build.headgearItem} />
@@ -294,26 +297,29 @@ const BuildFormModal: React.FC<BuildFormModalProps> = ({
           )}
         </Box>
         <Box asFlex flexDirection="column" alignItems="center">
-          <Select
-            label="Clothing"
-            options={clothingSelectOptions}
-            clearable
-            value={build.clothingItem}
-            setValue={(clothingItem: ClothingGear) => {
-              if (
-                (!build.clothing ||
-                  build.clothing.every(ability => ability === "UNKNOWN")) &&
-                existingGear.hasOwnProperty(clothingItem)
-              ) {
-                handleChange({
-                  clothingItem,
-                  clothing: existingGear[clothingItem],
-                })
-              } else {
-                handleChange({ clothingItem })
-              }
-            }}
-          />
+          <Box minW="275px">
+            <Select
+              label="Clothing"
+              options={clothingSelectOptions}
+              clearable
+              isSearchable
+              value={build.clothingItem}
+              setValue={(clothingItem: ClothingGear) => {
+                if (
+                  (!build.clothing ||
+                    build.clothing.every(ability => ability === "UNKNOWN")) &&
+                  existingGear.hasOwnProperty(clothingItem)
+                ) {
+                  handleChange({
+                    clothingItem,
+                    clothing: existingGear[clothingItem],
+                  })
+                } else {
+                  handleChange({ clothingItem })
+                }
+              }}
+            />
+          </Box>
           {build.clothingItem && (
             <Box mt="0.5em">
               <GearImage englishName={build.clothingItem} />
@@ -321,26 +327,29 @@ const BuildFormModal: React.FC<BuildFormModalProps> = ({
           )}
         </Box>
         <Box asFlex flexDirection="column" alignItems="center">
-          <Select
-            label="Shoes"
-            options={shoesSelectOptions}
-            clearable
-            value={build.shoesItem}
-            setValue={(shoesItem: ShoesGear) => {
-              if (
-                (!build.shoes ||
-                  build.shoes.every(ability => ability === "UNKNOWN")) &&
-                existingGear.hasOwnProperty(shoesItem)
-              ) {
-                handleChange({
-                  shoesItem,
-                  shoes: existingGear[shoesItem],
-                })
-              } else {
-                handleChange({ shoesItem })
-              }
-            }}
-          />
+          <Box minW="275px">
+            <Select
+              label="Shoes"
+              options={shoesSelectOptions}
+              clearable
+              isSearchable
+              value={build.shoesItem}
+              setValue={(shoesItem: ShoesGear) => {
+                if (
+                  (!build.shoes ||
+                    build.shoes.every(ability => ability === "UNKNOWN")) &&
+                  existingGear.hasOwnProperty(shoesItem)
+                ) {
+                  handleChange({
+                    shoesItem,
+                    shoes: existingGear[shoesItem],
+                  })
+                } else {
+                  handleChange({ shoesItem })
+                }
+              }}
+            />
+          </Box>
           {build.shoesItem && (
             <Box mt="0.5em">
               <GearImage englishName={build.shoesItem} />
diff --git a/frontend-react/src/components/user/ProfileModal.tsx b/frontend-react/src/components/user/ProfileModal.tsx
index b085a43a3..77a296307 100644
--- a/frontend-react/src/components/user/ProfileModal.tsx
+++ b/frontend-react/src/components/user/ProfileModal.tsx
@@ -39,7 +39,11 @@ const ProfileModal: React.FC<ProfileModalProps> = ({
   const toast = useToast()
 
   useEffect(() => {
-    if (profile.motion_sens !== null && !profile.stick_sens) {
+    if (
+      profile.motion_sens !== null &&
+      !profile.stick_sens &&
+      profile.stick_sens !== 0
+    ) {
       setError("Motion sensitivity entered without stick sensitivity")
     } else if (profile.custom_url && profile.custom_url.length < 2) {
       setError("Custom URL has to be over 2 characters long")
@@ -110,6 +114,7 @@ const ProfileModal: React.FC<ProfileModalProps> = ({
         <Box mt="1em">
           <Select
             label="Country"
+            isSearchable
             value={
               profile.country
                 ? {
diff --git a/graphql-schemas/build.js b/graphql-schemas/build.js
index d8d346840..39de06a2e 100644
--- a/graphql-schemas/build.js
+++ b/graphql-schemas/build.js
@@ -141,14 +141,14 @@ const resolvers = {
   Mutation: {
     addBuild: async (root, args, ctx) => {
       if (!ctx.user) throw new AuthenticationError("Not logged in.")
-      if (args.title)
+      if (args.title) {
         if (args.title.length > 100)
           throw new UserInputError("Title too long.", {
             invalidArgs: args,
           })
-        else {
-          args.title = undefined
-        }
+      } else {
+        args.title = undefined
+      }
 
       if (args.description) {
         if (args.description.length > 1000)