react-native-app-tour icon indicating copy to clipboard operation
react-native-app-tour copied to clipboard

TargetRadius 0 on Android doesnt overlay nested components

Open danieljgp2 opened this issue 6 years ago • 19 comments

@prscX on iOS when I set TargetRadius on 0 and I have a nested component, the nested componente showns overlay on the outercircle, this doesnt happens on Android.

On Android the nested component is under the outher circle and if I set targetTransparent it Just overlay the content inside the target.

This is how it looks on android with targetRadius on 0 Captura de Pantalla 2019-08-30 a la(s) 14 51 40

This is how it looks on iOS with targetRadius on 0 Captura de Pantalla 2019-08-30 a la(s) 14 50 39

danieljgp2 avatar Aug 30 '19 17:08 danieljgp2

@danieljgp2 i think you can just use it. It's open.

witalobenicio avatar Aug 30 '19 19:08 witalobenicio

@witalobenicio it work partially, still not working as it should. Any idea about how to solve it?

danieljgp2 avatar Aug 30 '19 19:08 danieljgp2

You are already using my PR?

witalobenicio avatar Aug 30 '19 19:08 witalobenicio

@witalobenicio yes, the problem seems to be related with the change of forView for forBounds, using your pr the component inside of the touchableOpacity with a TargetRadius 0 overlays the outercircle but it freeze the javascript after a while

danieljgp2 avatar Aug 30 '19 19:08 danieljgp2

Yeah... I use forView and seems to work just fine in my app. I don't use forBounds.

witalobenicio avatar Aug 30 '19 19:08 witalobenicio

Using for view doesnt break the title and the description of the tour?

danieljgp2 avatar Aug 30 '19 19:08 danieljgp2

Nope. It work just fine. I don't have the code here to show to you cause I'm on vacations...but as far as I know...it's all good. You just have to be careful of where you are using. When you use in deep views it doesn't work so nice. And remember to set collapsable to false.

witalobenicio avatar Aug 30 '19 19:08 witalobenicio

If you have a <Option> component that you want to highlight...maybe the highlight may have to go to the view inside the component. Got it?

witalobenicio avatar Aug 30 '19 19:08 witalobenicio

Is there a way to use your fork as a dependency? It is the first time i have this kind of issue, i Just took your fork and insert it on node module react native app tour 😁

danieljgp2 avatar Aug 30 '19 19:08 danieljgp2

Yes. When you do a fork, you create a new repo in your own account. So, for you to use my fork, you just have to add my repo url as a dependency.

react-native-app-tour:

And you can add with yarn too:

yarn add react-native-app-tour@

witalobenicio avatar Aug 30 '19 19:08 witalobenicio

Thanks for answering wita, still having the problem on my project, even with your fork.

danieljgp2 avatar Aug 30 '19 21:08 danieljgp2

Maybe on Sunday I can help you. I'll be back from vacation.

witalobenicio avatar Aug 30 '19 21:08 witalobenicio

@witalobenicio how can i contact with you?

danieljgp2 avatar Sep 02 '19 19:09 danieljgp2

@danieljgp2 lets talk by direct message on twitter. It's the same '@' of GitHub.

witalobenicio avatar Sep 02 '19 19:09 witalobenicio

@witalobenicio Already send you a Twit. Check it out

danieljgp2 avatar Sep 02 '19 19:09 danieljgp2

@witalobenicio You set collapsable false to the view inside the TouchableOpacity that is setted as the appviewtarget or to the appviewtarget itself ?

danieljgp2 avatar Sep 03 '19 01:09 danieljgp2

@witalobenicio Im using your fork and it seems to work fine, but when i change from a tab view to another one, opening the tour again freezes the app, it is like the tour is called but not displayed, this doesnt happen when i use forBounds, but with forBounds i have the problem that i just wrote above.

danieljgp2 avatar Sep 03 '19 01:09 danieljgp2

@danieljgp2 if you try the same with the default project it works? Maybe @prscX could help better on this one.

witalobenicio avatar Sep 03 '19 01:09 witalobenicio

@witalobenicio it is working for now, i did a fork and change the forBounds for the forViews, the problem is totally related with this change.

danieljgp2 avatar Sep 05 '19 20:09 danieljgp2