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

Partial selection outside and inside the table cause cells to be deleted when pressing Del or Backspace #541

Open
jswiderski opened this issue Jun 21, 2017 · 2 comments
Open

Partial selection outside and inside the table cause cells to be deleted when pressing Del or Backspace #541

jswiderski opened this issue Jun 21, 2017 · 2 comments
Labels
status:confirmed An issue confirmed by the development team. support:3 An issue reported by a commercially licensed client. type:bug A bug.

Comments

@jswiderski
Copy link
Contributor

jswiderski commented Jun 21, 2017 •
edited by mlewand
Loading

Are you reporting a feature or a bug?

Bug

Check if the issue is already reported

https://dev.ckeditor.com/ticket/16863

Provide detailed reproduction steps (if any)

  1. Open CKEditor with a table.
  2. Make a selection staring in ^except DSO and ending in dk^
  3. Press delete or backspace

Expected result

Table contents get deleted but table stays 3X3

Actual result

Cells 1.1 get removed

Other details

EDIT: Problem can also be reproduced when selecting content between cells. Select whole cell 1.1 and part of cell 1.2.

  • Browser: Webkit
  • OS: Windows, Mac
  • CKEditor version: 4.4.2
  • Installed CKEditor plugins: N/A
@jswiderski jswiderski added status:confirmed An issue confirmed by the development team. type:bug A bug. support An issue reported by a commercially licensed client. labels Jun 21, 2017
@ckeditor ckeditor deleted a comment from jswiderski Jun 21, 2017
@mlewand
Copy link
Contributor

mlewand commented Jun 21, 2017

Adjusted the original description, removed .txt file.

@mlewand mlewand added the target:minor Any docs related issue that can be merged into a master or major branch. label Mar 27, 2018
@msamsel msamsel self-assigned this Mar 27, 2018
@mlewand mlewand added target:major Any docs related issue that should be merged into a major branch. and removed target:minor Any docs related issue that can be merged into a master or major branch. labels Apr 9, 2018
@mlewand
Copy link
Contributor

mlewand commented Apr 9, 2018

Changing the target branch, as the patched behavior might introduce some breaking changes.

This was referenced Apr 10, 2018
Open
Open
Closed
@lslowikowska lslowikowska added support:1 An issue reported by a commercially licensed client. and removed support An issue reported by a commercially licensed client. labels Dec 21, 2018
@msamsel msamsel removed their assignment Sep 16, 2019
@f1ames f1ames removed the target:major Any docs related issue that should be merged into a major branch. label Nov 12, 2019
This was referenced Nov 13, 2019
Closed
Merged
@lslowikowska lslowikowska added support:3 An issue reported by a commercially licensed client. and removed support:1 An issue reported by a commercially licensed client. labels Feb 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
status:confirmed An issue confirmed by the development team. support:3 An issue reported by a commercially licensed client. type:bug A bug.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix for wrong deletion of mixed text and table content ckeditor/ckeditor4
5 participants
@f1ames @jswiderski @mlewand @msamsel @lslowikowska