-
Notifications
You must be signed in to change notification settings - Fork 756
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
A new feature "Ignored Substitutions" #544
Conversation
…at will be ignored by diff panes.
This is a very good idea. 😊 I can use this feature very well for programmes with German language files that cannot be improved by a pull request or an e-mail. Then I maintain my own language file and can hide a "funny" or suddenly wrong translation of a word when comparing with WinMerge. |
Thankl you! |
…at will be ignored by diff panes. (#544) (2)
…at will be ignored by diff panes. (#544) (3)
It's a good feature, but it doesn't seem to work as expected as below. I also want to be able to replace the PrediffLineFilter.sct plugin, which already has similar functionality, so I'd like to make some changes. However, I want to release a new version this month, so this feature itself may not be included in the new version. |
Yes, thank you. I'll fix that a bit later (Monday). The problem is probbaly
because it's the last line. I could not reliably figure out how to detect
that.
Good luck
Eugene
…On Sat, 9 Jan 2021 at 13:39, Takashi Sawanaka ***@***.***> wrote:
It's a good feature, but it doesn't seem to work as expected as below.
[image: image]
<https://user-images.githubusercontent.com/98126/104092948-9dd84580-52ca-11eb-80f8-fa03742f2a1c.png>
I also want to be able to replace the PrediffLineFilter.sct plugin, which
already has similar functionality, so I'd like to make some changes.
However, I want to release a new version this month, so this feature
itself may not be included in the new version.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#544 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AJT4IYW7RTX4ANAPOJZRZHLSZBMCLANCNFSM4V2X233A>
.
|
The Ignored Substitutions dialog has been modified as follows: By the way, why did you name it Ignored Substitutions instead of "Substitution filters"? I feel that the name is not consistent with other File filters and Line filters. |
Lovely!
"Substituion filter" is a good idea, it was on my short list. I also
considered "Rename filters", as this was what the feature was
specifically meant to be used for, but was not punchy and clear
enough.
In my coding practice I am a naming nazi - it's very important to name
things well. But the verb "filter", likely because of my engineering
background, has ambiguous meaning. The verb sometimes means "filter
out" (="throw away") or the opposite "allow in" (="pass through"),
with roughly the same frequency. So the verb alone does not
immediately tell you what's going on. That's why I went for explicit
term "ignored", which conveys the process under the hood. However, for
the big things such as WinMerge it's important to have consistency
throughout. So please rename to "Substitution filters" if you like it
more.
Good luck
Eugene
…On Tue, 26 Jan 2021 at 23:24, Takashi Sawanaka ***@***.***> wrote:
@EugeneLaptev
The Ignored Substitutions dialog has been modified as follows:
By the way, why did you name it Ignored Substitutions instead of "Substitution filters"?
I feel that the name is not consistent with other File filters and Line filters.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.
|
Thank you for your answer. |
This is a feature to facilitate a programmer workflow. If you deal with a big code base and happen to rename a symbol you may get hundreds of individual changes that are all the same. To let you focus on meaningful changes and filter out those symbol renames I've added another sort of filter called "Ignored Substitutions".
You can edit this filter in a standard way or you can right click on a change and select "Add to Ignored Substitutions", after which those changes will be ignored by the vie panes.
(Sorry is something is trivially wrong- this is my very first experience with github.)