Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Types for PickerSelectProps['Icon'] expects React.ReactNode when runtime expects a component #488

Closed
tquinlan1992 opened this issue Oct 4, 2022 · 10 comments

Comments

@tquinlan1992
Copy link

Describe the bug

The react-native-picker-select/index.d.ts has a type of React.ReactNode for PickerSelectProps['Icon'], but following that gives the following runtime error

Error: Element type is invalid: expected a string (for built-in components) or a class/function (for composite components) but got: object.

Check the render method of `RNPickerSelect`.

Passing in a component has a type error, but works as expected during runtime
To Reproduce

In a typescript project using the following the code will give the following typescript error. Though it works at runtime

code

import RNPickerSelect from 'react-native-picker-select';
import Icon from './Icon';

const TestComponent = () => <RNPickerSelect Icon={() => <Icon />} />

No overload matches this call.
Overload 1 of 2, '(props: PickerSelectProps | Readonly): Picker', gave the following error.
Type '() => JSX.Element' is not assignable to type 'ReactNode'.
Overload 2 of 2, '(props: PickerSelectProps, context: any): Picker', gave the following error.
Type '() => JSX.Element' is not assignable to type 'ReactNode'.


Using the following code instead will give no typescript error, but gives the error listed prior at runtime

import RNPickerSelect from 'react-native-picker-select';
import Icon from './Icon';

const TestComponent = () => <RNPickerSelect Icon={}



**Expected behavior**<br /><!-- do not remove heading -->
The expected behavior is for the types to match the runtime expectation 

**Screenshots**<br /><!-- do not remove heading -->
Add screenshots to help explain your problem. If screenshots aren't applicable to this issue, write "n/a".
"n/a"

**Additional details**<br /><!-- do not remove heading -->
-   Device: [Iphone13 Pro Max]
-   OS: [iOS15.5]
-   react-native-picker-select version: [8.0.4]
-   react-native version: [0.69.6]
-   expo sdk version: [46.0.13]

**Reproduction and/or code sample**<br /><!-- do not remove heading -->
Provide a link to a reproduction of this issue on https://snack.expo.io **or an explanation why you can not**. Not including a snack link will result in a significant delay in our ability to address this issue.

https://snack.expo.dev/iI_6XNg2f

<!-- YOU MUST FILL OUT EVERY SECTION. INCOMPLETE BUG REPORTS WILL BE CLOSED. -->
<!-- YOU MUST FILL OUT EVERY SECTION. INCOMPLETE BUG REPORTS WILL BE CLOSED. -->
@Aure77
Copy link

Aure77 commented Oct 19, 2022

Same issue. Seems to happen since react 18 TS types.
Icon?: () => JSX.Element; should be better ?

@Josh2941
Copy link

Josh2941 commented Nov 5, 2022

Facing the same issue.

@mccraveiro
Copy link

mccraveiro commented Nov 24, 2022

I'm using the following patch as a workaround:

diff --git a/index.d.ts b/index.d.ts
index 9f111bc0fca89c51ca7a2b666959e8ad6196bd12..09644ca8b989858021913b61625c079f96331664 100644
--- a/index.d.ts
+++ b/index.d.ts
@@ -83,7 +83,7 @@ export interface PickerSelectProps {
     pickerProps?: CustomPickerProps;
     touchableDoneProps?: CustomTouchableDoneProps;
     touchableWrapperProps?: CustomTouchableWrapperProps;
-    Icon?: React.ReactNode;
+    Icon?: React.FC;
     InputAccessoryView?: React.ReactNode;
 }
 

@KhimGurung
Copy link

@mccraveiro I am trying to patch it but it is taking forever. it keep saying installing. any idea?

@mccraveiro
Copy link

@KhimGurung no idea. What are you using to patch it? Yarn? patch-package?
Are there any error messages?

marioviana added a commit to ThreadsStyling/react-native-picker-select that referenced this issue Feb 24, 2023
As exposed here
(lawnstarter#488)
and here
(lawnstarter#499)
we need to change teh icon type as its required to be a function
@nhotalling-viagio
Copy link

@mccraveiro - thanks for providing a fix! I couldn't get patch-package to create a patch file, so I had to manually create one. Using your example, I had to add node_modules/react-native-picker-select/ to the file paths. Also had to ensure there were 2 new lines after the closing curly brace before patch-package would recognize it as a valid patch file (react-native-picker-select+8.0.4.patch)

diff --git a/node_modules/react-native-picker-select/index.d.ts b/node_modules/react-native-picker-select/index.d.ts
index 9f111bc..09644ca 100644
--- a/node_modules/react-native-picker-select/index.d.ts
+++ b/node_modules/react-native-picker-select/index.d.ts
@@ -83,7 +83,7 @@ export interface PickerSelectProps {
     pickerProps?: CustomPickerProps;
     touchableDoneProps?: CustomTouchableDoneProps;
     touchableWrapperProps?: CustomTouchableWrapperProps;
-    Icon?: React.ReactNode;
+    Icon?: React.FC;
     InputAccessoryView?: React.ReactNode;
 }


@itayperry
Copy link

I'm having the same issue :(

@aelfannir
Copy link

Workaround

<View style={{position: 'relative'}}>
    <RNPickerSelect />
    <YourIcon style={{position: 'absolute', end: 12, top: 12, zIndex: -1}}/>
</View>
    ```

@lfkwtz
Copy link
Collaborator

lfkwtz commented Nov 27, 2023

dupe of #478 - prs to fix this are welcome

@lfkwtz lfkwtz closed this as not planned Won't fix, can't repro, duplicate, stale Nov 27, 2023
@emiliospot
Copy link

emiliospot commented Mar 25, 2024

this works:

// Create a custom select icon component

import React from 'react'
import React from 'react';
import { StyleSheet } from 'react-native';
import { MaterialIcons } from '@expo/vector-icons';

const DropdownIcon: React.FC = () => {
const styles = StyleSheet.create({
icon: {
position: 'absolute',
end: 0,
top: 12,
zIndex: -1,
},
});

return (

);
};

export default DropdownIcon;

// import and use it in Component/Screen that uses the picker:
...
<RNPickerSelect
//other props
Icon={SelectIcon}
useNativeAndroidPickerStyle={false}
fixAndroidTouchableBug={true}/>

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

10 participants