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

onBlur , back button #23308

Closed
sm2017 opened this issue Feb 6, 2019 · 5 comments
Closed

onBlur , back button #23308

sm2017 opened this issue Feb 6, 2019 · 5 comments
Labels
Bug Component: TextInput Related to the TextInput component. Stale There has been a lack of activity on this issue and it may be closed soon.

Comments

@sm2017
Copy link
Contributor

sm2017 commented Feb 6, 2019

Environment

  React Native Environment Info:
    System:
      OS: Windows 10
      CPU: (8) x64 Intel(R) Core(TM) i7-3632QM CPU @ 2.20GHz
      Memory: 915.60 MB / 5.58 GB
    Binaries:
      Yarn: 1.13.0 - C:\Program Files (x86)\Yarn\bin\yarn.CMD
      npm: 6.4.1 - C:\Program Files\nodejs\npm.CMD
    IDEs:
      Android Studio: Version  3.3.0.0 AI-182.5107.16.33.5199772

Description

When text input is focused and you press back button in android , onBlur is not triggered

Reproducible Demo

https://snack.expo.io/rJnc3BuV4

Run snack and focus input , then press back button , the alert must shown

@kelset
Copy link
Contributor

kelset commented Feb 6, 2019

Which version of react native has this issue?

There was an issue in a patch version of 0.57.x, that we then rolled back. Please check the changelogs too.

@sm2017
Copy link
Contributor Author

sm2017 commented Feb 7, 2019

[email protected]

@JKCooper2
Copy link

JKCooper2 commented Mar 9, 2019

I think given the nature of the BackAndroid button it is better for devs to choose to override/add to the the default behaviour, e.g. https://snack.expo.io/@jkcooper/rn23308-textinput-onblur-backandroid. I would expect the current behaviour is similar to pressing back in a browser.

Also this should a TextInput label, not Button

@kelset kelset added Component: TextInput Related to the TextInput component. and removed Component: Button labels Mar 11, 2019
@stale
Copy link

stale bot commented Aug 2, 2019

Hey there, it looks like there has been no activity on this issue recently. Has the issue been fixed, or does it still require the community's attention? This issue may be closed if no further activity occurs. You may also label this issue as a "Discussion" or add it to the "Backlog" and I will leave it open. Thank you for your contributions.

@stale stale bot added the Stale There has been a lack of activity on this issue and it may be closed soon. label Aug 2, 2019
@stale
Copy link

stale bot commented Aug 9, 2019

Closing this issue after a prolonged period of inactivity. If this issue is still present in the latest release, please feel free to create a new issue with up-to-date information.

@stale stale bot closed this as completed Aug 9, 2019
@facebook facebook locked as resolved and limited conversation to collaborators Aug 9, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Bug Component: TextInput Related to the TextInput component. Stale There has been a lack of activity on this issue and it may be closed soon.
Projects
None yet
Development

No branches or pull requests

5 participants