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

The focus method was added #82

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

LPMendoza
Copy link

Now the developers can use the focus method to focus the input.

Example:
<PhoneInput ref={inputPhone} /> <Button title='focus' onPress={() => inputPhone.current?.focus()} />

@lghimfus
Copy link

this would be quite useful

@jeremykanso
Copy link

jeremykanso commented Feb 15, 2022

@garganurag893 Hello I also tested that PR, can you just merge it and republish npm package plz ?

Copy link

@ernest0vm ernest0vm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@garganurag893 please merge this PR 🙏🏼, @LPMendoza great job 👍🏼

@FondueBug
Copy link

It's so great that this is still not merged.

@DroidZed
Copy link

DroidZed commented Apr 8, 2024

Please add the necessary typing in index.d.ts to conform with the changes in the js file.

+ focus: () => void;

@LPMendoza
Copy link
Author

LPMendoza commented Apr 13, 2024

I have added the necessary typing to this PR
Can you review it please? I think this is useful for the people @DroidZed @garganurag893 @janziemba @nachozullo 🙌🏻

@LPMendoza
Copy link
Author

LPMendoza commented Jul 3, 2024

Is this library not maintained? The @garganurag893 profile shows that he has not coding all this year and part of the last one 🤔

@ernest0vm
Copy link

Is this library not maintained? The @garganurag893 profile shows that he has not coding all this year and part of the last one 🤔

maybe we can make a fork into our organization

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

Successfully merging this pull request may close these issues.

7 participants