SteffeyDev / react-native-popover-view

A well-tested, adaptable, lightweight <Popover> component for react-native
MIT License
613 stars 92 forks source link

Fix touch events through background for Mode.TOOLTIP #111

Closed pavelglebov closed 3 years ago

pavelglebov commented 3 years ago

When using mode={PopoverMode.TOOLTIP}, not actually all events go through containers - one of them is missing pointerEvents="box-none" to allow accessing background views.

pavelglebov commented 3 years ago

@SteffeyDev Hello! Could you please check this out? Would be helpful for me!

SteffeyDev commented 3 years ago

What was the issue that prompted this? Were you trying to tap outside tooltip and it wasn't working?

pavelglebov commented 3 years ago

Yep, I have fullscreen view under the Component with Tooltip. Component is also fullscreen, with pointerEvents="box-none". While Tooltip is OFF, beneath view is tappable. But when I set Tooltip to Visible - beneath component is no longer touchable. I was debugging Popover.js and saw that all of you components have pointerEvents="box-none" - thus touches should reach beneath views. Except one View, which also takes fullscreen, but doesn't have pointerEvents="box-none" - this View is blocking touch events to be passed to beneath views.

Screen Shot 2021-08-04 at 9 09 40 AM
pavelglebov commented 3 years ago
Screen Shot 2021-08-04 at 11 19 09 AM
pavelglebov commented 3 years ago

@SteffeyDev

SteffeyDev commented 3 years ago

Released 4.0.2