Skip to content

Comments

Revert "Issue #450 - Control.setFocus must not bring the application to front if it is in the background"#503

Merged
akurtakov merged 1 commit intoeclipse-platform:masterfrom
iloveeclipse:issue_450
Dec 15, 2022
Merged

Revert "Issue #450 - Control.setFocus must not bring the application to front if it is in the background"#503
akurtakov merged 1 commit intoeclipse-platform:masterfrom
iloveeclipse:issue_450

Conversation

@iloveeclipse
Copy link
Member

This reverts commit 961810f.

The change causes regression in the code that opens new shells and expects some widget to have a focus in that new shell.

See comments on
#450

…the application to

front if it is in the background"

This reverts commit 961810f.

The change causes regression in the code that opens new shells and
expects some widget to have a focus in that new shell.

See comments on
eclipse-platform#450
@github-actions
Copy link
Contributor

Test Results

     303 files       303 suites   7m 20s ⏱️
  4 047 tests   4 034 ✔️   8 💤 4  1 🔥
12 068 runs  11 992 ✔️ 70 💤 5  1 🔥

For more details on these failures and errors, see this check.

Results for commit c001284.

@akurtakov akurtakov merged commit a18fd18 into eclipse-platform:master Dec 15, 2022
@iloveeclipse iloveeclipse deleted the issue_450 branch December 15, 2022 12:09
@iloveeclipse
Copy link
Member Author

@akurtakov : thanks for merge, I actually wanted to give Thomas some time about possible fix before merging.
But now he has more time to do this :-)

@akurtakov
Copy link
Member

I'm going to run extra I-build for releng changes and I want next build to be usable and the issue wouldn't be solved over night IMO.

iloveeclipse added a commit to iloveeclipse/eclipse.platform.text that referenced this pull request Dec 15, 2022
This reverts commit 01906e9 as the
original SWT change was reverted too via
eclipse-platform/eclipse.platform.swt#503.
iloveeclipse added a commit to eclipse-platform/eclipse.platform.text that referenced this pull request Dec 15, 2022
This reverts commit 01906e9 as the
original SWT change was reverted too via
eclipse-platform/eclipse.platform.swt#503.
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.

Control.setFocus must not bring the application to front if it is in the background

2 participants