Skip to content

Stop next change from merging with previous one after inactive period.#530

Merged
JordanMartinez merged 2 commits intoFXMisc:masterfrom
JordanMartinez:preventUndoMergeOnInactivity
Jun 22, 2017
Merged

Stop next change from merging with previous one after inactive period.#530
JordanMartinez merged 2 commits intoFXMisc:masterfrom
JordanMartinez:preventUndoMergeOnInactivity

Conversation

@JordanMartinez
Copy link
Copy Markdown
Contributor

Addresses #362.

This should really be changed in UndoFX. But since that won't have a new release in the foreseeable future, this can bring the same functionality until such a release exists.

@JordanMartinez
Copy link
Copy Markdown
Contributor Author

This PR needs a test to prove that the merge prevention actually occurs.

@JordanMartinez
Copy link
Copy Markdown
Contributor Author

JordanMartinez commented Jun 22, 2017

I also realized that the test in UndoManagerTest called deletingTextThatWasJustInsertedShouldNotMergeTheTwoChanges() is no longer needed due to reverting back to the original merge behavior in #496. I'll remove that in a separate PR.

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.

1 participant