🩹 harsher padWithNItems limit to ensure picker scrolls in all scenarios

Author: troberts-28

src/components/DurationScroll/index.tsx

@@ -145,6 +145,7 @@ const DurationScroll = forwardRef<DurationScrollRef, DurationScrollProps>(
                     setClickSound(sound);
                 }
             };
+
             loadSound();
 
             // Unload sound when component unmounts
@@ -154,27 +155,6 @@ const DurationScroll = forwardRef<DurationScrollRef, DurationScrollProps>(
             // eslint-disable-next-line react-hooks/exhaustive-deps
         }, [Audio]);
 
-        useImperativeHandle(ref, () => ({
-            reset: (options) => {
-                flatListRef.current?.scrollToIndex({
-                    animated: options?.animated ?? false,
-                    index: initialScrollIndex,
-                });
-            },
-            setValue: (value, options) => {
-                flatListRef.current?.scrollToIndex({
-                    animated: options?.animated ?? false,
-                    index: getInitialScrollIndex({
-                        numberOfItems,
-                        padWithNItems,
-                        repeatNumbersNTimes: safeRepeatNumbersNTimes,
-                        value: value,
-                    }),
-                });
-            },
-            latestDuration: latestDuration,
-        }));
-
         const renderItem = useCallback(
             ({ item }: { item: string }) => {
                 let stringItem = item;
@@ -401,6 +381,27 @@ const DurationScroll = forwardRef<DurationScrollRef, DurationScrollProps>(
                 },
             ]);
 
+        useImperativeHandle(ref, () => ({
+            reset: (options) => {
+                flatListRef.current?.scrollToIndex({
+                    animated: options?.animated ?? false,
+                    index: initialScrollIndex,
+                });
+            },
+            setValue: (value, options) => {
+                flatListRef.current?.scrollToIndex({
+                    animated: options?.animated ?? false,
+                    index: getInitialScrollIndex({
+                        numberOfItems,
+                        padWithNItems,
+                        repeatNumbersNTimes: safeRepeatNumbersNTimes,
+                        value: value,
+                    }),
+                });
+            },
+            latestDuration: latestDuration,
+        }));
+
         return (
             <View
                 pointerEvents={isDisabled ? "none" : undefined}

src/components/TimerPicker/index.tsx

@@ -55,7 +55,7 @@ const TimerPicker = forwardRef<TimerPickerRef, TimerPickerProps>(
                 return 0;
             }
 
-            const maxPadWithNItems = hideHours ? 30 : 12;
+            const maxPadWithNItems = hideHours ? 15 : 6;
 
             if (padWithNItems > maxPadWithNItems) {
                 return maxPadWithNItems;