🩹 change default repeatHourNumbersNTimes to 7 for better consistency with minutes/seconds pickers

Author: troberts-28

README.md

@@ -449,7 +449,7 @@ return (
 |         use12HourPicker          | Switch the hour picker to 12-hour format with an AM / PM label                  |                                                                                             Boolean                                                                                             |  false  |  false   |
 |             amLabel              | Set the AM label if using the 12-hour picker                                    |                                                                                             String                                                                                              |   am    |  false   |
 |             pmLabel              | Set the PM label if using the 12-hour picker                                    |                                                                                             String                                                                                              |   pm    |  false   |
-|             repeatHourNumbersNTimes              | Set the number of times the list of hours is repeated in the picker                                   |                                                                                             Number                                                                                              |   6    |  false   |
+|             repeatHourNumbersNTimes              | Set the number of times the list of hours is repeated in the picker                                   |                                                                                             Number                                                                                              |   7    |  false   |
 |             repeatMinuteNumbersNTimes              | Set the number of times the list of minutes is repeated in the picker                                   |                                                                                             Number                                                                                              |   3    |  false   |
 |             repeatSecondNumbersNTimes              | Set the number of times the list of seconds is repeated in the picker                                   |                                                                                             Number                                                                                              |   3    |  false   |
 |      disableInfiniteScroll       | Disable the infinite scroll feature                                             |                                                                                             Boolean                                                                                             |  false  |  false   |
@@ -491,7 +491,7 @@ Note the minor limitations to the allowed styles for `pickerContainer` and `pick
 
 When the `disableInfiniteScroll` prop is not set, the picker gives the appearance of an infinitely scrolling picker by auto-scrolling forward/back when you near the start/end of the list. When the picker auto-scrolls, a momentary flicker is visible if you are scrolling very slowly.
 
-To mitigate for this, you can modify the `repeatHourNumbersNTimes`, `repeatMinuteNumbersNTimes` and `repeatSecondNumbersNTimes` props. These set the number of times the list of numbers in each picker is repeated. These have a performance trade-off: higher values mean the picker has to auto-scroll less to maintain the infinite scroll, but has to render a longer list of numbers. By default, the props are set to 6, 3 and 3, respectively, which balances that trade-off effectively.
+To mitigate for this, you can modify the `repeatHourNumbersNTimes`, `repeatMinuteNumbersNTimes` and `repeatSecondNumbersNTimes` props. These set the number of times the list of numbers in each picker is repeated. These have a performance trade-off: higher values mean the picker has to auto-scroll less to maintain the infinite scroll, but has to render a longer list of numbers. By default, the props are set to 7, 3 and 3, respectively, which balances that trade-off effectively.
 
 Note that you can avoid the auto-scroll flickering entirely by disabling infinite scroll. You could then set the above props to high values, so that a user has to scroll far down/up the list to reach the end of the list.
 

example/App.tsx

@@ -107,7 +107,6 @@ export default function App() {
                 <TimerPickerModal
                     Audio={Audio}
                     closeOnOverlayPress
-                    disableInfiniteScroll
                     Haptics={Haptics}
                     LinearGradient={LinearGradient}
                     modalProps={{
@@ -119,9 +118,6 @@ export default function App() {
                         setAlarmStringExample1(formatTime(pickedDuration));
                         setShowPickerExample1(false);
                     }}
-                    repeatHourNumbersNTimes={1}
-                    repeatMinuteNumbersNTimes={1}
-                    repeatSecondNumbersNTimes={1}
                     setIsVisible={setShowPickerExample1}
                     styles={{
                         theme: "dark",

src/components/TimerPicker/index.tsx

@@ -39,7 +39,7 @@ const TimerPicker = forwardRef<TimerPickerRef, TimerPickerProps>(
             padWithNItems = 1,
             pickerContainerProps,
             pmLabel = "pm",
-            repeatHourNumbersNTimes = 6,
+            repeatHourNumbersNTimes = 7,
             repeatMinuteNumbersNTimes = 3,
             repeatSecondNumbersNTimes = 3,
             secondLabel,