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

FileDialog & DirectoryDialog hang after closing them with opened menu #1221

Merged
merged 1 commit into from
May 8, 2024

Conversation

iloveeclipse
Copy link
Member

gtk_file_chooser_native_new handle() should be dereferenced after use as proposed in https://docs.gtk.org/gtk3/class.FileChooserNative.html.

Otherwise gtk_native_dialog_run() seem to continue to run (probably because the opened menu still references the dialog), with the effect that: the recursive main loop gives the effect of a modal dialog (it prevents the user from interacting with other windows in the same window group while the dialog is run).

See https://docs.gtk.org/gtk3/method.NativeDialog.run.html

Fixes #1062

`gtk_file_chooser_native_new handle()` should be dereferenced after use
as proposed in https://docs.gtk.org/gtk3/class.FileChooserNative.html.

Otherwise `gtk_native_dialog_run()` seem to continue to run (probably
because the opened menu still references the dialog), with the effect
that: `the recursive main loop gives the effect of a modal dialog (it
prevents the user from interacting with other windows in the same window
group while the dialog is run)`.

See https://docs.gtk.org/gtk3/method.NativeDialog.run.html

Fixes eclipse-platform#1062
Copy link
Contributor

github-actions bot commented May 8, 2024

Test Results

   418 files  ±0     418 suites  ±0   7m 47s ⏱️ +50s
 4 121 tests ±0   4 113 ✅ ±0   8 💤 ±0  0 ❌ ±0 
16 313 runs  ±0  16 221 ✅ ±0  92 💤 ±0  0 ❌ ±0 

Results for commit 28e72ec. ± Comparison against base commit bd900fb.

@iloveeclipse iloveeclipse merged commit d2bb303 into eclipse-platform:master May 8, 2024
14 checks passed
@iloveeclipse iloveeclipse deleted the issue_1062 branch May 8, 2024 14:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant