Replace defaultProps

[elliottkember]

Fixes #39

[hjjayakrishnan]

@xcarpentier Could you help merge these changes or assign someone to do so? Seems all checks have passed. Thx

[Tiagochicoo]

Please merge, thank you!

[vricosti]

If it's not merged soon I will create a @vricosti/react-native-country-picker-modal package as I already did for some unmaintained package (@vricosti/react-native-asset).
https://github.com/vricosti/react-native-country-picker-modal

[vricosti]

I tried to generate on my fork and I have ts errors:

vricosti@bifrost-Z490-VISION-D:~/Dev/react-native-country-picker-modal-vricosti$ yarn verify
src/CountryFilter.tsx:48:7 - error TS2783: 'placeholder' is specified more than once, so this usage will be overwritten.

48       placeholder={placeholder}
         ~~~~~~~~~~~~~~~~~~~~~~~~~

  src/CountryFilter.tsx:50:7
    50       {...props}
             ~~~~~~~~~~
    This spread always overwrites this property.

src/CountryFilter.tsx:49:7 - error TS2783: 'autoFocus' is specified more than once, so this usage will be overwritten.

49       autoFocus={autoFocus}
         ~~~~~~~~~~~~~~~~~~~~~

  src/CountryFilter.tsx:50:7
    50       {...props}
             ~~~~~~~~~~
    This spread always overwrites this property.


Found 2 errors in the same file, starting at: src/CountryFilter.tsx:48

so what about:

<TextInput
  testID="text-input-country-filter"
  autoCorrect={false}
  placeholderTextColor={filterPlaceholderTextColor}
  style={[
    styles.input,
    { fontFamily, fontSize, color: onBackgroundTextColor },
  ]}
  {...props} // Spread first to ensure other props are passed correctly
/>

{...props} already includes placeholder and autoFocus because it's built from CountryFilterProps.
If {...props} is spread before specific props, those props would overwrite it, but in your case, the reverse happens.

[KnotScientific]

Hi Can this get merged please? @xcarpentier

[vricosti]

Hi Can this get merged please? @xcarpentier

Merged here: https://github.com/vricosti/react-native-country-picker-modal