-
-
Notifications
You must be signed in to change notification settings - Fork 84
"Clear after" doesn't work as expected #208
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
Comments
Well after inner selection is just "end of". So far only outer selection has anything to do with delimiters. So I would argue that it actually does exactly what I expected it to :D |
Then how would you propose to do what I wanted to have happen? It is useful functionality. I guess I could do "clear outer after". But we're kind of wasting the "after" / "end of" distinction in this case. We could have a "trailing delimiter" / "leading delimiter" modifier rather than reusing "before" / "after". We might then even remove support for positional modifiers in arbitrary places and restrict them to things like "bring after". We'd then still need an answer for "past end of". Idk |
This is a hard one. We could always add a proper position for end of but we still have a problem with |
So I think based on our last discussion, we want to support this one, no? Because "after" would transform the selection into something where the content itself is the delimiter, so "clear" would target it |
Yes this will be supported after we do the new oo targets. |
Better to solve this one with #309 |
I would expect it to delete the trailing delimiter and leave the cursor there
The text was updated successfully, but these errors were encountered: