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

Bug: cannot add multiple images to table #1027

Closed
keeganstreet opened this issue Oct 11, 2017 · 2 comments · Fixed by #1104
Closed

Bug: cannot add multiple images to table #1027

keeganstreet opened this issue Oct 11, 2017 · 2 comments · Fixed by #1104
Assignees
@msamsel
Labels
plugin:table The plugin which probably causes the issue. plugin:tableselection The plugin which probably causes the issue. regression This issue is a regression. 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
4.8.0

Comments

@keeganstreet
Copy link

keeganstreet commented Oct 11, 2017 •
edited by beatadelura
Loading

Are you reporting a feature request or a bug?

Bug

Provide detailed reproduction steps

  1. Insert a table
  2. Insert an image into a cell of the table
  3. Focus on another cell of the same table
  4. Press the image button in the toolbar to enter an image into this cell too

Expected result

Both cells should have an image

Actual result

Only one image is added to the table

Screen recording

image in table

Other details

Browser: Chrome 61, Safari
CKEditor version: 4.7.3
@beatadelura beatadelura self-assigned this Oct 11, 2017
@beatadelura beatadelura added status:confirmed An issue confirmed by the development team. type:bug A bug. browser:chrome The issue can only be reproduced in the Chrome browser. browser:safari The issue can only be reproduced in the Safari browser. plugin:table The plugin which probably causes the issue. labels Oct 11, 2017
@beatadelura beatadelura removed their assignment Oct 11, 2017
@mlewand mlewand added the regression This issue is a regression. label Oct 11, 2017
@mlewand mlewand added this to the 4.8.0 milestone Oct 11, 2017
@msamsel msamsel self-assigned this Oct 25, 2017
@msamsel
Copy link
Contributor

msamsel commented Oct 25, 2017

possible related bug: #522

@msamsel msamsel mentioned this issue Oct 30, 2017
Merged
2 tasks
@mlewand mlewand added the target:major Any docs related issue that should be merged into a major branch. label Nov 23, 2017
@f1ames f1ames added plugin:tableselection The plugin which probably causes the issue. and removed browser:chrome The issue can only be reproduced in the Chrome browser. browser:safari The issue can only be reproduced in the Safari browser. labels Nov 27, 2017
@f1ames
Copy link
Contributor

f1ames commented Nov 27, 2017

Updated labels, it happened in all browsers supporting tableselection plugin.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@msamsel msamsel

Labels
plugin:table The plugin which probably causes the issue. plugin:tableselection The plugin which probably causes the issue. regression This issue is a regression. 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
Milestone
4.8.0
Development

Successfully merging a pull request may close this issue.

Widget has now correct selection behaviour inside tables ckeditor/ckeditor4
5 participants
@keeganstreet @f1ames @mlewand @msamsel @beatadelura