Skip to content

fix: remove calls to removed Keyboard.removeListener in useIsKeyboardShown #9457

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

Merged
merged 2 commits into from
Apr 3, 2021

Conversation

janicduplessis
Copy link
Contributor

Keyboard.removeEventListener was restored in facebook/react-native@035718b but not Keyboard.removeListener. In this case we can use the subscription object returned from addListener. The addListener method always returned a subscription object so no need to check if the return value is an object (like in other cases of addEventListener methods).

@janicduplessis janicduplessis requested a review from satya164 March 25, 2021 04:10
@netlify
Copy link

netlify bot commented Mar 25, 2021

Deploy preview for react-navigation-example ready!

Built with commit 86ce9fb

https://deploy-preview-9457--react-navigation-example.netlify.app

@github-actions
Copy link

The Expo app for the example from this branch is ready!

expo.io/@react-navigation/react-navigation-example?release-channel=pr-9457

@satya164 satya164 merged commit d87857e into main Apr 3, 2021
@satya164 satya164 deleted the @janic/patch-useIsKeyboardShown branch April 3, 2021 23:06
@satya164
Copy link
Member

satya164 commented Apr 3, 2021

Thanks

jbaudanza pushed a commit to Hilokal/react-navigation that referenced this pull request Sep 14, 2021
@erickjth
Copy link

erickjth commented Oct 8, 2021

Hey guys,
This is still an issue in @react-navigation/bottom-tabs version 5.11.15 using RN 0.65

@github-actions
Copy link

github-actions bot commented Oct 8, 2021

Hey! This issue is closed and isn't watched by the core team. You are welcome to discuss the issue with others in this thread, but if you think this issue is still valid and needs to be tracked, please open a new issue with a repro.

chrisbobbe pushed a commit to chrisbobbe/react-navigation that referenced this pull request Apr 27, 2022
chrisbobbe pushed a commit to chrisbobbe/react-navigation that referenced this pull request Apr 28, 2022
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.

3 participants