Jöran Karl

@ptrcnull: This has been fixed with #2810 too, right?

Closed with #3090.

Ok, this significantly changes the behavior, which before would also create a new `())` brace pair before the closing one and previously allowed `[])`. It should be discussed, if we...

I was triggered by #3101 (@niten94 :+1:) that this issue exist and would like to do the change. As already mentioned in zyedidia/tcell#26 it would be nice if not fork...

As already mentioned in #2951, it would be more common in case a regex search is used to implicit replace `\` by `\\` within the search string/argument, as we do...

Correct me if I'm wrong, but in regex the character `$` is escaped with `\$` and not `$$`. The latter is a special thing of the replacement string aka template...

So what you're basically referring to is to still keep the "template" functionality of the `replaceStr` in case we do a regex instead of a literal replace and therefore you...

As already mentioned, it was never the intention to remove any functionality targeting regex/power users. The only thing I was complaining about was the following fact: > But at this...

> I think after closing this PR you will need to close this too #2440 I don't think so. It doesn't freeze or crash. But test it on your own...

The real column selection is a good feature I was missing too. Unfortunately the multi-cursor mode can't fulfill this, but definitely has different advantages. So yes, it should be added...