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.

Sign up for GitHub

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

Jump to bottom

Select All + Delete leaves two paragraphs #306

Closed
fredck opened this issue Aug 22, 2016 · 3 comments
Closed

Select All + Delete leaves two paragraphs #306

fredck opened this issue Aug 22, 2016 · 3 comments
Labels
resolution:duplicate This issue is a duplicate of another issue and was merged into it.

Comments

@fredck
Copy link
Contributor

fredck commented Aug 22, 2016

  1. Go to the online demo.
  2. Click in the editor contents.
  3. Cmd+A to "Select All".
  4. Backspace to "Delete".

Actual results: two empty paragraphs are left (heading and normal paragraph).

Expected result: only one paragraph is left, the heading one (beginning of the selection).
@fredck
Copy link
Contributor Author

fredck commented Aug 22, 2016

I can see that both the first and the last paragraphs are left after deleting. This can be checked by changing the last one the h2 before deleting.

@fredck
Copy link
Contributor Author

fredck commented Aug 22, 2016

The same happens if you manually select more than one paragraph.

@Reinmar
Copy link
Member

Reinmar commented Aug 22, 2016

This is a duplicate of https://github.com/ckeditor/ckeditor5-engine/issues/520. Perhaps there are couple of cases to solve, but let's keep all of them together.

@Reinmar Reinmar closed this as completed Aug 22, 2016
@Reinmar Reinmar added the resolution:duplicate This issue is a duplicate of another issue and was merged into it. label Aug 22, 2016
@jodator jodator mentioned this issue Feb 24, 2020
Merged
mlewand pushed a commit that referenced this issue May 1, 2020
@oleq
Merge pull request #306 from ckeditor/i/6521
0246156 
Internal: The `MergeCellsCommand` should not merge the heading column cells with other cells. Closes #6521.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
resolution:duplicate This issue is a duplicate of another issue and was merged into it.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Reinmar @fredck