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

[v2, windows] Make sure to open the CommonFileDialogs on the main thread #2606

Merged
merged 1 commit into from
Apr 17, 2023

Conversation

stffabi
Copy link
Collaborator

@stffabi stffabi commented Apr 17, 2023

Otherwise it might have some strange side-effects like blocking the completion of Deferrals in WebView2 when using custom IStreams.

Otherwise it might have some strange side-effects like blocking
the completion of Deferrals in WebView2 when using custom
IStreams.
@stffabi stffabi force-pushed the bugfix/win-cfd-dialogs-thread branch from 74e0d7d to 3a20167 Compare April 17, 2023 10:11
@stffabi stffabi marked this pull request as ready for review April 17, 2023 10:11
@stffabi stffabi requested a review from leaanthony April 17, 2023 10:11
Copy link
Member

@leaanthony leaanthony left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Beast mode!!! 🔥

@leaanthony leaanthony merged commit 767bfcf into master Apr 17, 2023
@leaanthony leaanthony deleted the bugfix/win-cfd-dialogs-thread branch April 17, 2023 10:50
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.

None yet

2 participants