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

[tableselection] table contents can be removed in readonly mode #1489

Closed
ufdada opened this issue Jan 23, 2018 · 1 comment
Closed

[tableselection] table contents can be removed in readonly mode #1489

ufdada opened this issue Jan 23, 2018 · 1 comment
Labels
good first issue Relatively easy to fix. This is a perfect issue if you are willing to create a Pull Request. plugin:tableselection The plugin which probably causes the issue. status:confirmed An issue confirmed by the development team. target:minor Any docs related issue that can be merged into a master or major branch. type:bug A bug.
Milestone
4.11.0

Comments

@ufdada
Copy link

ufdada commented Jan 23, 2018

Are you reporting a feature request or a bug?

Bug

Provide detailed reproduction steps (if any)

  1. Insert a table into the editor and enter contents (or just use https://ckeditor.com/ckeditor-4/#document)
  2. Set editor in readonly mode
  3. Now drag and select the table cells (using the tableselection plugin)
  4. Press delete (actually any key works)

Expected result

The content shouldn't change at all

Actual result

Completely removing cells or deleting their content

Other details

  • Browser: chrome, ie11, edge, firefox
  • OS: Windows
  • CKEditor version: 4.8.0
  • Installed CKEditor plugins: tableselection
@f1ames f1ames added type:bug A bug. status:confirmed An issue confirmed by the development team. plugin:tableselection The plugin which probably causes the issue. labels Jan 24, 2018
@mlewand mlewand added this to the Backlog milestone Jan 29, 2018
@mlewand mlewand added the target:minor Any docs related issue that can be merged into a master or major branch. label Jan 29, 2018
This was referenced Jan 29, 2018
Closed
Merged
@mlewand mlewand added the good first issue Relatively easy to fix. This is a perfect issue if you are willing to create a Pull Request. label Mar 22, 2018
@mlewand mlewand modified the milestones: Backlog, 4.10.2 Sep 1, 2018
@Comandeer
Copy link
Member

Issue is fixed in #1527, fix will be released in 4.10.2.

@mlewand mlewand modified the milestones: 4.10.2, 4.11.0 Oct 4, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
good first issue Relatively easy to fix. This is a perfect issue if you are willing to create a Pull Request. plugin:tableselection The plugin which probably causes the issue. status:confirmed An issue confirmed by the development team. target:minor Any docs related issue that can be merged into a master or major branch. type:bug A bug.
Projects
None yet
Milestone
4.11.0
Development

No branches or pull requests
4 participants
@f1ames @Comandeer @ufdada @mlewand