-
Notifications
You must be signed in to change notification settings - Fork 265
Closed
Labels
Description
Please leave your feedback about new 'mini.ai' module here. Feel free to either add new comment or positively upvote existing one.
Some things I am interested to find out (obviously, besides bugs):
- Are configuration and function names intuitive enough?
- Are default values of settings convenient for you?
- Is documentation and examples clear enough?
- What is your opinion about default textobjects (for punctuation, digits, and whitespace)? At the moment it mimics 'targets.vim' idea about separator textobject - the
a
variant includes only one of the edges (the maximum right ones, to be precise). Although useful in many cases (like_
in snake case variables allowsda_
to delete word while preserving snake case), it might be not useful for some characters in most situations (like|
seems to rarely serve as separator and more like surrounding). I see at least three ways to solve this:- Leave it as is. It means that users are encouraged to manually define custom textobjects to include both edges in
a
textobject. It is what currently is done with at least three different ways to achieve this. This is the way if there is little consensus about what character should be treated as surrounding by default. - Add new builtin surrounding to include both edges in
a
textobject. This is the way, if there is a strong consensus about certain character (like|
) being more of a surrounding instead of separator and of which type (balanced or non-balanced). - Add new
default
builtin surrounding to act by default. Maybe some people like include both edges by default (I know I was at first, but then changed my mind because of how useful including only right edge is). Completely not opposed to this, but any options should be considered carefully. This is the way of adding new option.
- Leave it as is. It means that users are encouraged to manually define custom textobjects to include both edges in
Thanks!