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

(typescript) type scrollIntoView method #537

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

hduprat
Copy link

@hduprat hduprat commented Apr 27, 2022

I noticed the scrollIntoView method of KeyboardAwareScrollView was not typed in Typescript. I added it, using native Typescript Types from React Native in order to ensure compatibility.

It's a slightly more complete version than #378 that should have been merged back in 2019.

@hduprat
Copy link
Author

hduprat commented May 16, 2022

@alvaromb @slorber Do you have time to read this PR? If not, how can I be of help?

@TacticCoder
Copy link

TacticCoder commented May 24, 2022

@hduprat, you can use patch-package until they approve

@mjvestal
Copy link

I just ran into this issue. Without this PR, linters show an error if you try to use scrollIntoView

@slorber
Copy link
Collaborator

slorber commented Oct 26, 2022

Hi

FYI I'm not maintaining this package. It just happened that I got merge/publish access and was the last one to do a release as a service to the community, but I have never been the project maintainer and don't plan to become one. If you want PRs to be reviewed/merged/published, it's best to contact the owners of this repo

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.

4 participants