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

Insert Column before and After fails if a cell vertical split exist #591

Closed
f1ames opened this issue Jun 30, 2017 · 5 comments · Fixed by #592
Closed

Insert Column before and After fails if a cell vertical split exist #591

f1ames opened this issue Jun 30, 2017 · 5 comments · Fixed by #592
Labels
plugin:tabletools The plugin which probably causes the issue. status:confirmed An issue confirmed by the development team. target:major Any docs related issue that should be merged into a major branch. type:bug A bug.
Milestone

Comments

@f1ames
Copy link
Contributor

f1ames commented Jun 30, 2017

Are you reporting a feature or a bug?

Bug

Check if the issue is already reported

http://dev.ckeditor.com/ticket/13729 (see also ckeditor/ckeditor4-sdk#164 (comment)).

Provide detailed reproduction steps (if any)

  1. Created a table with 4 columns and 3 rows.
  2. Make a vertical split in the cell 2,2.
  3. Do a column insert before in 4th column.

Expected result

A new column is inserted between 3rd and 4th column.

Actual result

A new column will be created between 2nd and 3rd column.

Other details

  • Browser: All
  • OS: All
  • CKEditor version: 3.5.1 (reproduced in 4.7.1)
  • Installed CKEditor plugins: Full
@f1ames f1ames added plugin:tabletools The plugin which probably causes the issue. status:confirmed An issue confirmed by the development team. type:bug A bug. labels Jun 30, 2017
@mlewand
Copy link
Contributor

mlewand commented Jun 30, 2017

@f1ames can you check say 4.6.0, 4.4.5 to see if it's a regression or not?

@f1ames
Copy link
Contributor Author

f1ames commented Jun 30, 2017

@mlewand It was already checked by @jswiderski in http://dev.ckeditor.com/ticket/13729. I put this information in Other details:

CKEditor version: 3.5.1 (reproduced in 4.7.1)

@f1ames
Copy link
Contributor Author

f1ames commented Jun 30, 2017

@mlewand Unless you meant it was fixed in 4.6.0 and then broken again. I will check 4.6.0 (it was reported in 4.5.3 so there is no point in checking earlier as it was already done).

@f1ames
Copy link
Contributor Author

f1ames commented Jun 30, 2017

Also reproducible in 4.6.0 so it does not look like a regression.

@f1ames f1ames added the target:major Any docs related issue that should be merged into a major branch. label Sep 14, 2017
@f1ames f1ames added this to the 4.8.0 milestone Sep 14, 2017
@f1ames
Copy link
Contributor Author

f1ames commented Sep 14, 2017

Closed with #592 (fc149ee).

@f1ames f1ames closed this as completed Sep 14, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
plugin:tabletools The plugin which probably causes the issue. status:confirmed An issue confirmed by the development team. target:major Any docs related issue that should be merged into a major branch. type:bug A bug.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants