Skip to content

Add a feature to disable undo boundary temporarily#1034

Merged
cxxxr merged 1 commit intolem-project:mainfrom
fukamachi:undo-boundary-control
Sep 4, 2023
Merged

Add a feature to disable undo boundary temporarily#1034
cxxxr merged 1 commit intolem-project:mainfrom
fukamachi:undo-boundary-control

Conversation

@fukamachi
Copy link
Copy Markdown
Collaborator

This PR allows users to enable/disable auto undo boundary addition temporarily.

This feature is required in vi-mode to take all modifications done while in INSERT mode as a single undo stack.

I made it in a hacky way in vi-mode (ref #991), but it's not perfect since Lem automatically adds an undo boundary after every command invocation. For example, Lem adds a boundary even after completion.

Take this as a feature request with a code. I'm fine if the feature is done in a different way. Any advice is welcome.

@fukamachi fukamachi mentioned this pull request Aug 31, 2023
@cxxxr
Copy link
Copy Markdown
Member

cxxxr commented Sep 4, 2023

Thank you for the nice PR.
In the future, undo management may need to be improved, but for now, this is a good idea.

@cxxxr cxxxr merged commit 9844ca1 into lem-project:main Sep 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants