-
Notifications
You must be signed in to change notification settings - Fork 78
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
Add help context for the find/replace overlay #2171
Add help context for the find/replace overlay #2171
Conversation
<description>Overlay to find and replace specified text.</description> | ||
<topic label="Find/Replace with regular expressions" href="reference/ref-regex-find-replace.htm" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please correct indentation (and maybe also fix the one in the context where this was copied from).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The indentation is inconsistent, which is why different viewers render different visible indentations. One line uses whitespaces and one uses tabs. In the IDE, by accident, the number of whitespaces aligns with the tab size (you could even tell the IDE to render tabs differently and would then also see a difference):
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry, I didn't see that!
I have changed all tabs into four spaces
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks. Unfortunately, just transforming tabs into whitespaces is not correct, as there have been combinations of whitespaces and tabs which still behaves like a single tab (e.g. 1 whitespace + 1 tab behaves like 1 tab). So there are now places with too many whitespaces.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks like Eclipse has no autoformat for XML by default (??!)
I copied the file into vscode (which also doesn't autoformat XMLs by default?! I used the RedHat XML extension) and formatted it there.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, if I am not mistaken, both require (additional) extensions to (auto-)format XMLs.
177ad21
to
32cebf4
Compare
f02586a
to
6e7f00a
Compare
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
6e7f00a
to
4405b07
Compare
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