Skip to content

Modernized help UI prototype: Fix deleting and applying a search scope#1781

Merged
iloveeclipse merged 1 commit intoeclipse-platform:masterfrom
howlger:patch-6
Mar 26, 2025
Merged

Modernized help UI prototype: Fix deleting and applying a search scope#1781
iloveeclipse merged 1 commit intoeclipse-platform:masterfrom
howlger:patch-6

Conversation

@howlger
Copy link
Contributor

@howlger howlger commented Mar 19, 2025

This fixes the following issues in the modernized help UI prototype:

  • Deleting a set search scope causes JavaScript error.
  • Applying via search scope drop-down, choosing Scopes..., going to the scope and hitting Apply does not work; as workaround a search scope can directly selected via search scope drop-down.

This fixes the following issues in the modernized help UI prototype:
- Deleting a set search scope causes JavaScript error.
- Applying via search scope drop-down, choosing "Scopes...", going to
the scope and hitting "Apply" does not work; as workaround a search
scope can directly selected via search scope drop-down.
@github-actions
Copy link
Contributor

Test Results

1 190 files   -   162  1 190 suites   - 162   58m 11s ⏱️ - 6m 24s
4 173 tests ±    0  4 126 ✅ ±    0   47 💤 ±0  0 ❌ ±0 
8 955 runs   - 1 034  8 835 ✅  - 1 028  120 💤  - 6  0 ❌ ±0 

Results for commit 378897a. ± Comparison against base commit 15f5d4e.

@hhermann
Copy link
Contributor

Thank you Holger for this fix. I have tested it on Windows 11 and Linux RHEL 9.2 and it works fine.

@iloveeclipse iloveeclipse merged commit e4ebf10 into eclipse-platform:master Mar 26, 2025
16 of 17 checks passed
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.

3 participants