fix auto paste issue by fixing undo action #1000
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixing autopaste function issue after undoing and repasting
Description
Fix the autopaste function after undoing by getting the copy_info from UI state and passing it in all undo & redo stack data.
Related Issue
#946
Autopaste should put the strand immediately to the right of the second strand, as happened the first time that we only had two strands. In other words we need Undo to undo not only the change to which strands are in the design, but also undo the change in internal Autopaste data that happens after each time you press Ctrl+Shift+V.
Motivation and Context
This fixes a bug where the index of the auto pasted strand is not updated after the undo action.
How Has This Been Tested?
Tested in the Chrome browser on my computer