-
Notifications
You must be signed in to change notification settings - Fork 17
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
Replace syntax has misleading hover text #32
Comments
Enumerate does not refer to regular expression back reference. The verb enumerate means to count and name each item in a list. E.g., %n replaces with 1, 2, 3, …; %00n replaces with 001, 002, 003, …. and so on. I think when the Regular expression option is ticked in the dialog, the hint text should change to mention back references. |
That's pretty useful. I wish there was a help file or wiki page that described this stuff.
Does not work for me. |
I meant: I think this is an oversight and that this isn't programmed yet. That the hint text is static and it isn't programmed yet to change it based on context. |
Oh, my mistake. |
The hover text makes it look like you do
%1
,%2
, etc to insert captured groups, but it's actually\1
,\2
, etc because it's using Python regex.This had me really confused until I looked at the source code and realized why
%1
was treated as a literal rather than inserting a captured group, and I believe it should be changed to reflect its actual usage.If "enumerate" means anything other than "insert capture groups," I think there should be more information on what it does, and I think there should also be a note somewhere (in both the application and repo readme, preferably) that it uses Python regex so nobody else will get confused like I was.
The text was updated successfully, but these errors were encountered: