(Picklist): make getPrevValue() and getNextValue() not wrap around

Author: msmx-mnakagawa

src/scripts/Picklist.tsx

@@ -192,7 +192,9 @@ export const Picklist: (<MultiSelect extends boolean | undefined>(
         if (!currentValue) return optionValues[0];
 
         const currentIndex = optionValues.indexOf(currentValue);
-        return optionValues[currentIndex + 1] || optionValues[0]; // wrap around
+        return optionValues[
+          Math.min(currentIndex + 1, optionValues.length - 1)
+        ]; // not wrap around
       },
       [getOptionValues]
     );
@@ -206,10 +208,7 @@ export const Picklist: (<MultiSelect extends boolean | undefined>(
         if (!currentValue) return optionValues[optionValues.length - 1];
 
         const currentIndex = optionValues.indexOf(currentValue);
-        return (
-          optionValues[currentIndex - 1] ||
-          optionValues[optionValues.length - 1]
-        ); // wrap around
+        return optionValues[Math.max(currentIndex - 1, 0)]; // not wrap around
       },
       [getOptionValues]
     );