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

Null VirtualView #137

Merged
merged 2 commits into from
Sep 30, 2024
Merged

Null VirtualView #137

merged 2 commits into from
Sep 30, 2024

Conversation

IeuanWalker
Copy link
Contributor

@IeuanWalker IeuanWalker commented Sep 17, 2024

Might fix #91

I was getting this error with a very specific use case within the app. I did this fix before resolving the underlining issue causing it.

I'm not sure what the issue was/ how I was causing it, but for some reason even though the popup was closed tapping anywhere on the screen would trigger HitTest method and then crash.

Its as if the PopupWindow was never getting cleaned up, and just sitting on top of the app.

Originally I just had return null; in the catch, but every time I went through this specific use case the number of PopupWindow would add up and each tap, would trigger HitTest x number of times. So added the cleanup code, and all seemed ok after that.

I've altered my code since, and I no longer get into the catch, so not sure what I was doing that would of caused it

@LuckyDucko LuckyDucko merged commit 66f585d into LuckyDucko:master Sep 30, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

VirtualView cannot be null here
2 participants