Revert "Remove setNativeProps API" change about react-native-web HOT 4 OPEN

@necolas I appreciate the technical concerns here—thank you!—and I'm sure it's not easy to convey without the reader already having some context around the implementation.

From my point of view, setNativeProps in RNW (0.18) works well-enough for styles (which might be because of how styles work in RNW 0.18) but more importantly works for setting text input values without round-tripping via the React lifecycle. My use case is for things like formatted text inputs between focus/blur.

I have previously tried to remove setNativeProps in favour of the suggested state-based approaches, but I ran into update cascades/loops (certainly in the native code), and overall introduces more complexity and fragility than setNativeProps for these types of tasks. For style updates—such as handling hover—setNativeProps is perceptibly more responsive than the state round-trip.

Do you think there's some way forward where a "use at own risk" type API is introduced for setNativeProps for these difficult edge cases? Or is this a hard line from your point of view?

from react-native-web.

@jonathanj For now I think you could use Platform.OS forks to use setNativeProps on native and direct manipulation on web. Eventually RN will support the same direct manipulation APIs as web, and will support sync updates that make things like managing text input content better than it is today.

from react-native-web.

setNativeProps never worked correctly on web and had undefined behaviour on native. If the fabric implementation has defined behaviour, there's still no reliable way to implement it on web while styles are not simply inline styles. The primary use case of setNativeProps was updating styles without a render, but it's just not reliable on web. Things like text input animations aren't done this way on web either. The new RN architecture will eventually allow for sync events and sync updates, providing the same event loop model as web, so the setNativeProps pattern will likely have fewer reasons to be used on native

from react-native-web.

facebook/react-native@1d3fa40

Add setNativeProps to Fabric

Summary:
changelog: Introduce setNativeProps to Fabric

Add support for setNativeProps in Fabric for backwards compatibility. It is still recommended to move away from setNativeProps because the API will not work with future features.

We can make step Migrating off setNativeProps in migration guide optional.

Reviewed By: yungsters, mdvacca

Differential Revision: D41521523

fbshipit-source-id: 4d9bbd6304b8c5ee24a36b33039ed33ae1fc21f8

from react-native-web.