Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Multiple TouchEffects cause crash on iOS #9

Open
Hackmodford opened this issue Dec 1, 2023 · 6 comments
Open

Multiple TouchEffects cause crash on iOS #9

Hackmodford opened this issue Dec 1, 2023 · 6 comments
Labels

Comments

@Hackmodford
Copy link
Contributor

I've noticed a crash in my app if there are multiple graphs.

For example I have two pages with graphs.

If I navigate A -> B -> A then try to touch Graph A, the app crashes in the PlotViewBase.Events.cs file.

image

This is do to previousTouchPoints being nil. It seems that the TouchEffect is associated with the wrong view? Not sure. Or they're all sending reports to each other.

@Hackmodford
Copy link
Contributor Author

My current solution was to carefully set the InputTransparency property on the graphs to true if they are not visible, but as you can imagine that is a bit of a pain to manage.

This was the original reason for my earlier commit.

@janusw janusw added the OS:iOS label Dec 2, 2023
@janusw
Copy link
Member

janusw commented Dec 2, 2023

Is it possible to reproduce this problem with the sample app? Could you please provide the full error message / exception ?

@Hackmodford
Copy link
Contributor Author

I'm working on generating a sample project.

I believe I have triggered the issue by having two graphs and starting a drag on one graph and sliding my finger over to the other graph.

@Hackmodford
Copy link
Contributor Author

Hackmodford commented Dec 3, 2023

Here is my minimal example project.
I believe this is the key line causing me issues.

If you try dragging the circle on iOS it will crash.

https://github.com/Hackmodford/oxyplot-maui-crash1

The key seems to be with settings the opacity of the plot.
It may also crash if you have the graphs side by side and try starting in one plot and dragging to the other.

In any case the example in the project is what is causing me headaches in my current app.

@jaison-t-john
Copy link

@Hackmodford, Can I know if pinch to zoom is working for you in iOS?

@RobertStefancic
Copy link
Contributor

@Hackmodford could you please test the latest version 1.0.1 that includes the fix from #10
I took a glance on your issue and this fix has a good probability to solve it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants