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

Range-deleting multiple paragraphs leads to corrupt state #8219

Closed
Discordius opened this issue Oct 6, 2020 · 1 comment
Closed

Range-deleting multiple paragraphs leads to corrupt state #8219

Discordius opened this issue Oct 6, 2020 · 1 comment
Labels
resolution:duplicate This issue is a duplicate of another issue and was merged into it. type:bug This issue reports a buggy (incorrect) behavior.

Comments

@Discordius
Copy link

In CkEditor, create four paragraphs, like this:

1

2

3

4

Place the cursor before the '4'. Shift-select after the '1' to create a selection that includes paragraphs 2 and 3 and all of the paragraph-dividers. Press backslash. The result should be:

14

but instead it's

13

4

Now press Cmd+Z to undo what you just did. The result should have all four paragraphs, but instead it's

1

2

4

(paragraph 3 is missing)

📃 Other details

Comments

As you can imagine, this is pretty bad. This is a pretty trivial way to end up in a state in which undo doesn't work, the devtools crash, and all kinds of weird stuff starts happening. So my guess is that fixing this is pretty high priority.


If you'd like to see this fixed sooner, add a 👍 reaction to this post.

@jodator
Copy link
Contributor

jodator commented Oct 7, 2020

Oh, it looks like we've fixed that right after a release: #8145.

I've checked master branch and nightly docs and the problem does not occur there. I'm closing this as a duplicate of #8145.

The fix should be available in the upcoming release.

@jodator jodator closed this as completed Oct 7, 2020
@jodator jodator added the resolution:duplicate This issue is a duplicate of another issue and was merged into it. label Oct 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
resolution:duplicate This issue is a duplicate of another issue and was merged into it. type:bug This issue reports a buggy (incorrect) behavior.
Projects
None yet
Development

No branches or pull requests

2 participants