Skip to content
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

Too low degree of undo granularity #2057

Closed
Ace-Who opened this issue Mar 4, 2020 · 6 comments
Closed

Too low degree of undo granularity #2057

Ace-Who opened this issue Mar 4, 2020 · 6 comments

Comments

@Ace-Who
Copy link

Ace-Who commented Mar 4, 2020

For Notepad3, only backspace or moving cursor breaks undo.
For Notepad2,Enter also breaks undo,that's not enough but still better.

@leeoniya
Copy link

leeoniya commented Mar 4, 2020

there has been a lot of work done to refine & tweak undo behavior. try the latest NP3 build in #1129

there is Edit -> Miscellaneous -> Split Undo Transaction at Line Breaks which corresponds to SplitUndoTypingSeqOnLnBreak=true in Notepad3.ini.

there is also a manual config param in Notepad3.ini called UndoTransactionTimeout=500 (mine is set to 500ms).

previous discussions: #1421 and #1548

@hpwamr
Copy link
Collaborator

hpwamr commented Mar 4, 2020

there has been a lot of work done to refine & tweak undo behavior. try the latest NP3 build in #1129

Or if you are not a fan of beta version, you can try the latest version Notepad3 RC (Release Candidate).

The Notepad3 "Release version" will be published in a few days at the same address... 😃

@RaiKoHoff
Copy link
Collaborator

Maybe we should make SplitUndoTypingSeqOnLnBreak=true the default on new installations 🤔

@leeoniya
Copy link

leeoniya commented Mar 4, 2020

Maybe we should make SplitUndoTypingSeqOnLnBreak=true the default on new installations

👍

and i still vote for exposing the UndoTransactionTimeout setting next to it in the UI. maybe even with a default of 500ms ;)

@hpwamr
Copy link
Collaborator

hpwamr commented Mar 4, 2020

Maybe we should make SplitUndoTypingSeqOnLnBreak=true the default on new installations

👍

and i still vote for exposing the UndoTransactionTimeout setting next to it in the UI. maybe even with a default of 500ms ;)

I agree. 👍

@Ace-Who
Copy link
Author

Ace-Who commented Mar 4, 2020

Thank you everyone.

@Ace-Who Ace-Who closed this as completed Mar 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants