Skip to content

Let most widgets remove trailing ; from history lines #370

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.

Sign up for GitHub

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

Jump to bottom
Closed
marlonrichert opened this issue Nov 20, 2021 · 3 comments
Closed

Let most widgets remove trailing ; from history lines #370

marlonrichert opened this issue Nov 20, 2021 · 3 comments
Assignees
@marlonrichert

Comments

@marlonrichert
Copy link
Owner

Semis are not being removed from completions when you move to edit them. If you then complete this edited command, you end up with two trailing semis which is a no-op.

Perhaps leftwards cursor movements such as ←, HOME, etc. could be added as semi-removal triggers similar to how Space works since fe20fc5?

Originally posted by @pcjmfranken in #361 (comment)
@marlonrichert marlonrichert self-assigned this Nov 20, 2021
@pcjmfranken
Copy link

PS I was looking into a reliable method for generating a trigger list, but most search results were StackOverflow posts on related matters written by you :)

@marlonrichert
Copy link
Owner Author

What, are you saying my SO posts are not reliable? 😉

generating a trigger list

What do you mean by that?

@marlonrichert marlonrichert changed the title Let non-self-insert widgets remove trailing ; from history lines Let cursor movement remove trailing ; from history lines Nov 20, 2021
@marlonrichert marlonrichert changed the title Let cursor movement remove trailing ; from history lines Let most widgets remove trailing ; from history lines Nov 20, 2021
@marlonrichert
Copy link
Owner Author

@pcjmfranken @amomchilov @adinklotz Done.

PS: If you enjoy using my software, please consider sponsoring me. 🙂

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 19, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@marlonrichert marlonrichert

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@marlonrichert @pcjmfranken