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

Pasting inside editor which contains a table after selecting all content is not working #875

Closed
Comandeer opened this issue Sep 7, 2017 · 2 comments
Assignees
Labels
plugin:tableselection The plugin which probably causes the issue. regression This issue is a regression. status:confirmed An issue confirmed by the development team. type:bug A bug.
Milestone

Comments

@Comandeer
Copy link
Member

Are you reporting a feature request or a bug?

Bug

Provide detailed reproduction steps (if any)

  1. Open https://jsfiddle.net/9a6aenLg/2/
  2. Select table via elements path and copy it.
  3. Press Ctrl + A.
  4. Paste table.

Expected result

All content is replaced by pasted table.

Actual result

Table inside editor's content becomes selected.

It seems like editor performs paste inside the table.

Other details

  • CKEditor version: 4.7.2
  • Installed CKEditor plugins: tableselection

It's a regression from 4.7.1, probably due to the refactorization of paste algorithm.

@Comandeer Comandeer added plugin:tableselection The plugin which probably causes the issue. regression This issue is a regression. status:confirmed An issue confirmed by the development team. type:bug A bug. labels Sep 7, 2017
@f1ames f1ames changed the title Pasting table inside the editor after selecting all content is not working Pasting inside editor which contains a table after selecting all content is not working Jul 24, 2019
@f1ames
Copy link
Contributor

f1ames commented Jul 24, 2019

It happens for any elements/content, not only table - reported also in #1168, #2513.

@f1ames f1ames added this to the Next milestone Sep 25, 2019
@f1ames f1ames added the target:minor Any docs related issue that can be merged into a master or major branch. label Sep 26, 2019
@Dumluregn Dumluregn self-assigned this Oct 3, 2019
@f1ames f1ames removed the target:minor Any docs related issue that can be merged into a master or major branch. label Nov 12, 2019
@f1ames
Copy link
Contributor

f1ames commented Dec 4, 2019

Fixed with a72568f.

@f1ames f1ames closed this as completed Dec 4, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
plugin:tableselection The plugin which probably causes the issue. regression This issue is a regression. status:confirmed An issue confirmed by the development team. type:bug A bug.
Projects
None yet
Development

No branches or pull requests

3 participants