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

Find/Replace Overlay: add help context to Overlay #2045

Conversation

Wittmaxi
Copy link

@Wittmaxi Wittmaxi commented Jul 6, 2024

Add the find/replace help context to the overlay which was also used for the find/replace dialog. In particular, display help for find/replace with regex patterns.

fixes #1994
24

Copy link
Contributor

github-actions bot commented Jul 6, 2024

Test Results

 1 815 files  ±0   1 815 suites  ±0   1h 29m 43s ⏱️ - 4m 21s
 7 665 tests ±0   7 437 ✅ ±0  228 💤 ±0  0 ❌ ±0 
24 156 runs  ±0  23 407 ✅ ±0  749 💤 ±0  0 ❌ ±0 

Results for commit 65de8cf. ± Comparison against base commit 21f5812.

♻️ This comment has been updated with latest results.

Copy link
Contributor

@HeikoKlare HeikoKlare left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It makes sense to reuse the help for the find/replace dialog. Since I find it a little confusing that the help title refers to a "dialog to [...]" instead of an "overlay to [...]" or the like. Since this is easy adapt and does not produce problematic redundancies, I propose to add a find_replace_overlay_context. It has to be added here (by copy-and-adapt of the existing find_replace_dialog_context:
https://github.com/eclipse-platform/eclipse.platform.releng.aggregator/blob/3650575d30e4d7b09158120bf66c1a0243f78154/eclipse.platform.common/bundles/org.eclipse.platform.doc.user/contexts_Workbench.xml#L509-L512

@Wittmaxi Wittmaxi force-pushed the MW_add_find_replace_overlay_help_context branch from 7118f66 to 4d5cac8 Compare July 8, 2024 08:38
Wittmaxi pushed a commit to Wittmaxi/eclipse.platform.releng.aggregator that referenced this pull request Jul 8, 2024
Adds a help context for the find/replace overlay which links to the help
context for the dialog.

related to
eclipse-platform/eclipse.platform.ui#2045
@Wittmaxi
Copy link
Author

Wittmaxi commented Jul 8, 2024

eclipse-platform/eclipse.platform.releng.aggregator#2171

This PR adds the help context which is now used in the newest version of this PR @HeikoKlare

@Wittmaxi
Copy link
Author

Wittmaxi commented Jul 8, 2024

grafik

Wittmaxi pushed a commit to Wittmaxi/eclipse.platform.releng.aggregator that referenced this pull request Jul 8, 2024
Adds a help context for the find/replace overlay which links to the help
context for the dialog.

related to
eclipse-platform/eclipse.platform.ui#2045
Wittmaxi pushed a commit to Wittmaxi/eclipse.platform.releng.aggregator that referenced this pull request Jul 8, 2024
Adds a help context for the find/replace overlay which links to the help
context for the dialog.

related to
eclipse-platform/eclipse.platform.ui#2045
HeikoKlare pushed a commit to Wittmaxi/eclipse.platform.releng.aggregator that referenced this pull request Jul 8, 2024
Adds a help context for the find/replace overlay which links to the help
context for the dialog.

related to
eclipse-platform/eclipse.platform.ui#2045
HeikoKlare pushed a commit to Wittmaxi/eclipse.platform.releng.aggregator that referenced this pull request Jul 9, 2024
Adds a help context for the find/replace overlay which links to the help
context for the dialog.

related to
eclipse-platform/eclipse.platform.ui#2045
HeikoKlare pushed a commit to Wittmaxi/eclipse.platform.releng.aggregator that referenced this pull request Jul 9, 2024
Adds a help context for the find/replace overlay which links to the help
context for the dialog.

related to
eclipse-platform/eclipse.platform.ui#2045
HeikoKlare pushed a commit to eclipse-platform/eclipse.platform.releng.aggregator that referenced this pull request Jul 9, 2024
Adds a help context for the find/replace overlay which links to the help
context for the dialog.

related to
eclipse-platform/eclipse.platform.ui#2045
Add the find/replace help context to the overlay which was also used for
the find/replace dialog. In particular, display help for find/replace
with regex patterns.

fixes eclipse-platform#1994
@HeikoKlare HeikoKlare force-pushed the MW_add_find_replace_overlay_help_context branch from 4d5cac8 to 65de8cf Compare July 9, 2024 20:11
Copy link
Contributor

@HeikoKlare HeikoKlare left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

eclipse-platform/eclipse.platform.releng.aggregator#2171 has been merged and I've successfully tested this PR based on today's I-Build containing that change.

@HeikoKlare HeikoKlare merged commit 579b936 into eclipse-platform:master Jul 11, 2024
16 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.

Find/Replace overlay: missing help for regular expressions
2 participants