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

Removing table columns throw an error #577

Closed
Comandeer opened this issue Jun 29, 2017 · 1 comment
Closed

Removing table columns throw an error #577

Comandeer opened this issue Jun 29, 2017 · 1 comment
Assignees
@msamsel
Labels
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.7.3

Comments

@Comandeer
Copy link
Member

Are you reporting a feature or a bug?

Bug

Provide detailed reproduction steps (if any)

Case 1

  1. Open http://sdk.ckeditor.com/samples/table.html
  2. Select whole 2013 section inside the table (see screenshot below)
  3. Right click inside the selection and choose "Column"→"Delete Columns"

Example of selected columns

Expected result

Columns are deleted.

Actual result

Columns are not deleted and the error is thrown:

ckeditor.js:66 Uncaught TypeError: Cannot read property 'ownerDocument' of undefined

Case 2

  1. Open http://sdk.ckeditor.com/samples/table.html
  2. Select 2013 section without the header inside the table (see screenshot below)
  3. Right click inside the selection and choose "Column"→"Delete Columns"

Example of selected columns

Expected result

Columns are deleted.

Actual result

Columns are deleted, but the error is thrown:

ckeditor.js:66 Uncaught TypeError: Cannot read property 'ownerDocument' of undefined

Additionally selection is put inside incorrect cell (see screenshot below).

Incorrect position of cursor

Other details

  • CKEditor version: 4.7.0
@Comandeer Comandeer added plugin:tableselection The plugin which probably causes the issue. status:confirmed An issue confirmed by the development team. type:bug A bug. labels Jun 29, 2017
@mlewand mlewand added this to the Backlog milestone Jun 29, 2017
@msamsel msamsel self-assigned this Jul 14, 2017
@msamsel
Copy link
Contributor

msamsel commented Jul 14, 2017

During testing this issue I found additional case: #648

@msamsel msamsel mentioned this issue Jul 17, 2017
Merged
2 tasks
@mlewand mlewand added the target:minor Any docs related issue that can be merged into a master or major branch. label Aug 30, 2017
@Comandeer Comandeer mentioned this issue Aug 31, 2017
Closed
@Comandeer Comandeer modified the milestones: 4.7.3, Backlog Aug 31, 2017
This was referenced Sep 12, 2017
Merged
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@msamsel msamsel

Labels
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.7.3
Development

No branches or pull requests
3 participants
@Comandeer @mlewand @msamsel