ProgerXP / Notepad2e

Word highlighting, simultaneous editing, split views, math evaluation, un/grep, comment reformatting, UAC elevation, complete regexps (PCRE), Lua lexers, DPI awareness and more (XP+)
Other
375 stars 52 forks source link

Select To Matching Brace to include brackets #367

Closed ProgerXP closed 1 year ago

ProgerXP commented 3 years ago

Extend this command when FindSelectToMatchingBraceMode is enabled (#293) so that if current selection equals bracketed contents it extends selection by including surrounding brackets (as if FindSelectToMatchingBraceMode were disabled).

<(a|)>

First Ctrl+Shift+B selects a. Second selects (a) because current selection already equals selection the command would produce (a).

cshnik commented 3 years ago

Done.

ProgerXP commented 2 years ago

Bug: extension includes up to one trailing symbol:

([a]).

Press Ctrl+Shift+B to see [(a).] selected instead of [(a)].`.

cshnik commented 2 years ago

Fixed.

ProgerXP commented 2 years ago

Expansion needlessly depends on caret position:

([|a])
([a|])

It is expected that in both cases Ctrl+Shift+B would select the outer brackets but the second case currently only moves the caret to the opposite side, requiring one more Ctrl+Shift+B to select all.

cshnik commented 2 years ago

Fixed.

ProgerXP commented 2 years ago

Selection isn't extended anymore at all:

(|a) -> ([a]) -> must be [(a)] but remains ([a])
cshnik commented 2 years ago

Fixed.

ProgerXP commented 1 year ago

if current selection equals bracketed contents it extends selection by including surrounding brackets

Bug in R261 7b1cf6f

(a[b])

Ctrl+Shift+B selects [(ab)] rather than ([ab]) even though current selection <does not> equals bracketed contents.

ProgerXP commented 1 year ago

The above issue has been fixed in some commit.