Skip to content

Show ProgressMonitorJobsDialog after delay in ProgressManager.run#3311

Merged
iloveeclipse merged 1 commit into
eclipse-platform:masterfrom
vi-eclipse:show_progress_dialog_after_delay_if_should_not_run_in_background
Sep 24, 2025
Merged

Show ProgressMonitorJobsDialog after delay in ProgressManager.run#3311
iloveeclipse merged 1 commit into
eclipse-platform:masterfrom
vi-eclipse:show_progress_dialog_after_delay_if_should_not_run_in_background

Conversation

@fedejeanne
Copy link
Copy Markdown
Member

@fedejeanne fedejeanne commented Sep 22, 2025

See #3147 (comment)

Schedule the progress dialog to appear after a delay if the preference "Always run in background" is set to false

image

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Sep 22, 2025

Test Results

 2 904 files  ±0   2 904 suites  ±0   2h 8m 46s ⏱️ + 8m 57s
 8 019 tests ±0   7 773 ✅  - 1  245 💤 ±0  1 ❌ +1 
23 591 runs  ±0  22 808 ✅  - 1  782 💤 ±0  1 ❌ +1 

For more details on these failures, see this check.

Results for commit 51bf991. ± Comparison against base commit 2f6a390.

♻️ This comment has been updated with latest results.

Even for the backwards compatible mode (when the runnable is not forked
or non-cancelable).
This commit improves upon ccafe58 by
letting the dialog pop up (after a delay) if the preferences "Always run
in background" is set to false. When the preference is set to true, no
progress dialog will ever pop up.
@fedejeanne fedejeanne force-pushed the show_progress_dialog_after_delay_if_should_not_run_in_background branch from e2fa6de to 51bf991 Compare September 22, 2025 10:42
@iloveeclipse iloveeclipse merged commit 0f5ed45 into eclipse-platform:master Sep 24, 2025
16 of 18 checks passed
@iloveeclipse
Copy link
Copy Markdown
Member

@fedejeanne : thanks.

@fedejeanne
Copy link
Copy Markdown
Member Author

@fedejeanne : thanks.

You're welcome, thank you for merging and for all the testing!

@fedejeanne fedejeanne deleted the show_progress_dialog_after_delay_if_should_not_run_in_background branch September 24, 2025 11:30
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