!!!ASAP!!! 0.15.1 crash native apps!!!

[oxyii]

overflowY crash app just importing MenuProvider !!!

[jamesvweston]

Related to this issue: #146

[jamesvweston]

For a temporary fix replace "^0.15.0" with "0.15.0" in package.json

[oxyii]

I know, thanks. Issue created for repo-members )

npm install --save react-native-popup-menu@0.15.0 works fine too

[sodik82]

Thanks for letting us know... Funny thing that 0.15.1 is working fine in expo ( https://snack.expo.io/@sodik82/react-native-popup-menu-with-trigger-long-press ) but overflowY seems not to be official property in RN. Fixed in 0.15.2

[sodik82]

@oxyii is 0.15.2 fine?

[oxyii]

@sodik82 Yep, 0.15.2 works fine on native platforms. But have useNativeDriver warning on web when menu open

[oxyii]

@sodik82 I think in build folder must be 3 files

• index.js // <- just export from "react-native-popup-menu"
• react-native-popup-menu.js // <- for native platforms
• react-native-popup-menu.web.js // <- for web

or do not build at all.

Metro bundler will compile her-self.
Developers who works with web platform already made an ejection and added required node_modules libraries into babel config. Impossible to create a good RNWeb app without ejecting or using others libs for babel configuration like a react-app-rewired and customize-cra. IMHO )

[sodik82]

The build for RNWeb came from community. However I like your idea - two builds could use different env variables to enable/disable native driver.

Could you create something like that? Or at least create new improvement issue :)

[oxyii]

Platform-specific code works fine natively. Just create *.<platform>.js )) *.web.js too. You do not need to do something else.

I don't have enough time to do that now but I just fixed web warning. See PR