Skip to content

Trigger Show In for typed selection in Open Resource dialog#4057

Merged
vogella merged 1 commit into
eclipse-platform:masterfrom
vogella:open-resource-showin-typed-path
Jun 5, 2026
Merged

Trigger Show In for typed selection in Open Resource dialog#4057
vogella merged 1 commit into
eclipse-platform:masterfrom
vogella:open-resource-showin-typed-path

Conversation

@vogella
Copy link
Copy Markdown
Contributor

@vogella vogella commented Jun 3, 2026

In the Open Resource dialog, the Show In action for a non-file resource (for example a project) was gated on the OK button's enabled state. When the resource was reached by typing a pattern instead of clicking the row, that enabled state could lag behind the auto-selected first match, so pressing Enter or the Open button did nothing.

This acts on the selected item directly, the same way the dialog opens a typed file. A first-matched project now triggers Show In via Enter or the Open button, making the keyboard and mouse paths behave consistently.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 3, 2026

Test Results

   861 files  ±0     861 suites  ±0   1h 0m 17s ⏱️ + 5m 41s
 8 034 tests ±0   7 791 ✅ +1  243 💤 ±0  0 ❌  - 1 
20 541 runs  ±0  19 886 ✅ +3  655 💤 ±0  0 ❌  - 3 

Results for commit 8096f7f. ± Comparison against base commit 2c46bb5.

♻️ This comment has been updated with latest results.

@vogella
Copy link
Copy Markdown
Contributor Author

vogella commented Jun 3, 2026

Test failure related to eclipse-platform/eclipse.platform#2700

The Show In action for a non-file resource was gated on the OK button's
enabled state. When the resource was reached by typing a pattern rather
than clicking the row, that state could lag behind the auto-selected
first match, so Enter and the Open button did nothing.

Act on the selected item directly, the same way the dialog opens a typed
file, so a first-matched project triggers Show In via Enter or Open too.
@vogella vogella force-pushed the open-resource-showin-typed-path branch from 2cd8dfb to 8096f7f Compare June 5, 2026 07:53
@vogella vogella merged commit afada46 into eclipse-platform:master Jun 5, 2026
18 checks passed
@vogella vogella deleted the open-resource-showin-typed-path branch June 5, 2026 09:57
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.

1 participant