@herca/rn-kit 0.1.116 → 0.1.118

package/src/Input/InputMonth.tsx

@@ -70,30 +70,35 @@ const zeroBased = (month: number | null | undefined) =>
     ? null
     : month - 1;
 
-export default function InputMonth({
-  label,
-  placeholder,
-  placeholderEnd,
-  value,
-  valueEnd,
-  mode = 'single',
-  onSelectClick,
-  onPickerClose,
-  onChange,
-  language,
-  hasClear,
-  title,
-  cancelLabel,
-  confirmLabel,
-  testID,
-}: InputMonthProps) {
+export default function InputMonth(props: InputMonthProps) {
+  const {
+    label,
+    placeholder,
+    placeholderEnd,
+    value,
+    valueEnd,
+    mode = 'single',
+    onSelectClick,
+    onPickerClose,
+    onChange,
+    language,
+    hasClear,
+    title,
+    cancelLabel,
+    confirmLabel,
+    testID,
+  } = props;
+
   const [isOpen, setIsOpen] = useState(false);
   const [selected, setSelected] = useState<PickerFieldValue>(EMPTY);
 
-  // A given prop wins, so the field can be driven from outside; otherwise it
-  // shows whatever was last picked.
-  const start = value ?? (mode === 'range' ? selected.startValue : selected.value); // prettier-ignore
-  const end = valueEnd ?? selected.endValue;
+  // Presence of the prop decides ownership, not its contents: `value ??`
+  // would read a parent's clear as "no prop given" and fall back to the last
+  // pick, leaving what was cleared on screen.
+  const start = 'value' in props
+    ? value ?? null
+    : mode === 'range' ? selected.startValue : selected.value; // prettier-ignore
+  const end = 'valueEnd' in props ? (valueEnd ?? null) : selected.endValue;
 
   // What the sheet should open on. Without this the picker only knows about
   // taps it saw itself, so a value coming from a prop -- or a clear -- would
@@ -161,6 +166,7 @@ export default function InputMonth({
         value={pickerValue}
         onClose={handleClose}
         onChange={handleChange}
+        {...(language ? { language } : {})}
         {...(title ? { title } : {})}
         {...(cancelLabel ? { cancelLabel } : {})}
         {...(confirmLabel ? { confirmLabel } : {})}

package/src/Input/InputYear.tsx

@@ -47,27 +47,34 @@ const yearLabel = (year: number | null | undefined) =>
  * Year counterpart to InputMonth. There is no `language` prop: a year is
  * digits, and nothing about it changes between locales.
  */
-export default function InputYear({
-  label,
-  placeholder,
-  placeholderEnd,
-  value,
-  valueEnd,
-  mode = 'single',
-  onSelectClick,
-  onPickerClose,
-  onChange,
-  hasClear,
-  title,
-  cancelLabel,
-  confirmLabel,
-  testID,
-}: InputYearProps) {
+export default function InputYear(props: InputYearProps) {
+  const {
+    label,
+    placeholder,
+    placeholderEnd,
+    value,
+    valueEnd,
+    mode = 'single',
+    onSelectClick,
+    onPickerClose,
+    onChange,
+    hasClear,
+    title,
+    cancelLabel,
+    confirmLabel,
+    testID,
+  } = props;
+
   const [isOpen, setIsOpen] = useState(false);
   const [selected, setSelected] = useState<PickerFieldValue>(EMPTY);
 
-  const start = value ?? (mode === 'range' ? selected.startValue : selected.value); // prettier-ignore
-  const end = valueEnd ?? selected.endValue;
+  // Presence of the prop decides ownership, not its contents: `value ??`
+  // would read a parent's clear as "no prop given" and fall back to the last
+  // pick, leaving what was cleared on screen.
+  const start = 'value' in props
+    ? value ?? null
+    : mode === 'range' ? selected.startValue : selected.value; // prettier-ignore
+  const end = 'valueEnd' in props ? (valueEnd ?? null) : selected.endValue;
 
   // What the sheet should open on. Without this the picker only knows about
   // taps it saw itself, so a value coming from a prop -- or a clear -- would

package/src/Input/README_INPUT_MONTH.md

@@ -60,8 +60,8 @@ const [range, setRange] = useState({ start: undefined, end: undefined });
 | `label`          | `string`                   | ❌     | -          | Teks label di atas trigger. Kalau kosong, elemen labelnya tidak dirender sama sekali.             |
 | `placeholder`    | `string`                   | ✅     | -          | Teks trigger saat belum ada nilai.                                                                |
 | `placeholderEnd` | `string`                   | ✅\*   | -          | Placeholder trigger kedua. Wajib dan hanya berlaku saat `mode="range"`.                           |
-| `value`          | `number`                   | ❌     | -          | Nomor bulan, `1` (Januari) sampai `12` (Desember). Kalau diisi, ia menang atas pilihan internal. Di luar 1–12 dianggap kosong. |
-| `valueEnd`       | `number`                   | ❌     | -          | Nomor bulan akhir, `1`–`12`. Hanya berlaku saat `mode="range"`.                                    |
+| `value`          | `number`                   | ❌     | -          | Nomor bulan, `1` (Januari) sampai `12` (Desember). Begitu prop ini dipasang, ia yang pegang nilai: `undefined` berarti kosong, bukan "pakai pilihan internal". Di luar 1–12 dianggap kosong. |
+| `valueEnd`       | `number`                   | ❌     | -          | Nomor bulan akhir, `1`–`12`. Sama aturannya dengan `value`. Hanya berlaku saat `mode="range"`.      |
 | `mode`           | `'single' \| 'range'`      | ❌     | `'single'` | Satu bulan, atau rentang dua bulan.                                                               |
 | `language`       | `'en' \| 'id'`             | ❌     | `'id'`     | Bahasa nama bulan yang ditampilkan.                                                               |
 | `hasClear`       | `boolean`                  | ❌     | `false`    | Menampilkan tombol clear saat ada nilai.                                                          |

package/src/Input/README_INPUT_YEAR.md

@@ -55,8 +55,8 @@ const [year, setYear] = useState<number>();
 | `label`          | `string`                        | ❌     | -          | Teks label di atas trigger. Kalau kosong, elemen labelnya tidak dirender.    |
 | `placeholder`    | `string`                        | ✅     | -          | Teks trigger saat belum ada nilai.                                           |
 | `placeholderEnd` | `string`                        | ✅\*   | -          | Placeholder trigger kedua. Wajib dan hanya berlaku saat `mode="range"`.      |
-| `value`          | `number`                        | ❌     | -          | Tahun penuh, mis. `2024`. Kalau diisi, ia menang atas pilihan internal.      |
-| `valueEnd`       | `number`                        | ❌     | -          | Tahun akhir. Hanya berlaku saat `mode="range"`.                              |
+| `value`          | `number`                        | ❌     | -          | Tahun penuh, mis. `2024`. Begitu prop ini dipasang, ia yang pegang nilai: `undefined` berarti kosong, bukan "pakai pilihan internal". |
+| `valueEnd`       | `number`                        | ❌     | -          | Tahun akhir. Sama aturannya dengan `value`. Hanya berlaku saat `mode="range"`. |
 | `mode`           | `'single' \| 'range'`           | ❌     | `'single'` | Satu tahun, atau rentang dua tahun.                                          |
 | `hasClear`       | `boolean`                       | ❌     | `false`    | Menampilkan tombol clear saat ada nilai.                                     |
 | `onChange`       | `(v: PickerFieldValue) => void` | ❌     | -          | Dipanggil saat pilihan dikonfirmasi atau dibersihkan.                        |

package/src/hooks/useOptionalFocusEffect.ts

@@ -7,7 +7,6 @@ export function useOptionalFocusEffect(callback: () => void | (() => void)) {
 
   const effect = useCallback(callback, [callback]);
 
-  // eslint-disable-next-line react-hooks/rules-of-hooks
   if (navigation) {
     // eslint-disable-next-line react-hooks/rules-of-hooks
     useFocusEffect(effect);