onChange not working in browser

[dracoblackie]

I have tested some code in android, and it worked fine. Then when run in chrome it seems like we dont hit onChange.

Neither with a static value <WheelPicker selectedIndex={defaultIndex} options={someOptionsArray} onChange={(index) => this.setState({someStateValue: 2})} /> or if we set someStateValue to someOptionsArray[index]

In chrome devtools, i have set a debugpoint at onChange. When i run it on an android emulator we hit it in the debugger, but when run on a webserver, we dont hit the onChange. Additionally i have tried with a simple console.log("hello world") instead => <WheelPicker selectedIndex={defaultIndex} options={someOptionsArray} onChange={(index) => { console.log("Hello World") } } /> We do not seem to get the message in the console of the browser.

In both tests i have used mouse and keyboard.

I will isolate the code into a modal module to see if this fixes things, but thought this would be helpfull.

[erksch]

Do you mean with react-native-web? I have never used it but I'll try to reproduce.

[dracoblackie]

Yes, i mean with react-native-web. After writing the bug-report i realized that it never states in this library that it supports web, so this might be more of a feature request. Also, isolating the code did not change the outcome.

[erksch]

Sorry, I currently do not come around to reproduce your example, if you want you can create a pull request for react-native-web support, feel free to do so.

react-native-wheely uses a FlatList internally and calls onChange on onMomentumScrollEnd. Maybe onMomentumScrollEnd is not called in web.

[ToshKoevoets]

@erksch indeed looks like onMomentScrollEnd isn't fired on web:

    onScroll={Animated.event(
      [{ nativeEvent: { contentOffset: { y: scrollY } } }],
      { useNativeDriver: true, listener: (event) => handleScroll(event)  },
    )}

Solution might be to call it on scroll all the time although there might be some performance issues there.

[alexkendall]

I've added react-native for web support off this forked repoistory: https://github.com/alexkendall/react-native-wheely

[Jaza]

FYI @alexkendall has a fix for this pending review at https://github.com/erksch/react-native-wheely/pull/31