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

Make error on load show on UI thread #986

Closed
wants to merge 1 commit into from

Conversation

Miepee
Copy link
Contributor

@Miepee Miepee commented Jun 16, 2022

Description

Makes it so errors on loading a data.win file are invoked on the UI thread and don't freeze the program.

Caveats

None

Notes

Addresses the issue #984 brought up.

@Miepee Miepee marked this pull request as draft June 16, 2022 14:53
@github-actions
Copy link

github-actions bot commented Jun 16, 2022

@him7778
Copy link

him7778 commented Jun 16, 2022

Snipaste_2022-06-16_23-12-35
It still has problems.

@Miepee
Copy link
Contributor Author

Miepee commented Jun 16, 2022

I wasn't saying i fixed the game loading.
I was saying, that I fixed the problem your issue brought up. Which is that the message boxes didn't appear.

@Miepee Miepee marked this pull request as ready for review June 16, 2022 15:16
@him7778
Copy link

him7778 commented Jun 16, 2022

I wasn't saying i fixed the game loading. I was saying, that I fixed the problem your issue brought up. Which is that the message boxes didn't appear.

So sorry,is there a solution to this problem currently?

@Miepee
Copy link
Contributor Author

Miepee commented Jun 16, 2022

You need to wait until #945 gets implemented. Which this PR does not do.

@Miepee
Copy link
Contributor Author

Miepee commented Jun 17, 2022

Obsoleted in favour of #987 for now.

@Miepee Miepee closed this Jun 17, 2022
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.

2 participants