mirror of
https://github.com/facebook/react-native.git
synced 2025-11-01 09:14:26 +00:00
Valentin Shergin
3f7e0a2c96
Summary: This is the first diff in the series. It removes Flow types for this feature to verify that we actually do not have any usages. After it lands, we will remove actual support on the native side. There are several reasons why removing it is a good idea: * There is no any evidence that this feature is actually useful. That was discussed several times (e.g. see T7936714) during RN lifetime and the overall consensus is: We need something else, something like sync `onChange` event instead of it. * Supporting the previous point, it's not used (at least inside Facebook). I searched hard and I could find only one place where it's used: in the TextInput Example. * To deliver more functionality we should lean towards W3C specs, this one is not W3C compliant. * Supporting this Feature in Fabric is quite challenging, so I want to do it sooner than later. * This feature was never documented. Changelog: [Breaking] `<TextInput>::onTextInput` event was removed Reviewed By: TheSavior Differential Revision: D18456175 fbshipit-source-id: c7a8ed7a86b33ecc01d45497645fe249556fdf96