Skip to content

Call replaceNode rather than replaceRange#21833

Merged
amcasey merged 1 commit into
microsoft:masterfrom
amcasey:TextChangesRange
Feb 10, 2018
Merged

Call replaceNode rather than replaceRange#21833
amcasey merged 1 commit into
microsoft:masterfrom
amcasey:TextChangesRange

Conversation

@amcasey
Copy link
Copy Markdown
Member

@amcasey amcasey commented Feb 9, 2018

...where convenient. Same for deleteNode/Range.

Related to, but not a solution for, #21246.

@amcasey amcasey requested a review from a user February 9, 2018 18:41
Copy link
Copy Markdown

@ghost ghost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

But see #21832 (review)

...where convenient.  Same for deleteNode/Range.
@amcasey amcasey merged commit d99cd05 into microsoft:master Feb 10, 2018
@amcasey amcasey deleted the TextChangesRange branch February 10, 2018 02:21
@microsoft microsoft locked and limited conversation to collaborators Jul 3, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant