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

Can't insert a nested table with paste #9016

Closed
Mgsy opened this issue Feb 11, 2021 · 3 comments
Closed

Can't insert a nested table with paste #9016

Mgsy opened this issue Feb 11, 2021 · 3 comments
Labels
domain:v4-compatibility This issue reports a CKEditor 4 feature/option that's missing in CKEditor 5. package:table resolution:invalid This issue is invalid (e.g. reports a non-existent bug or a by-design behavior). type:improvement This issue reports a possible enhancement of an existing feature.

Comments

@Mgsy
Copy link
Member

Mgsy commented Feb 11, 2021

πŸ“ Provide detailed reproduction steps (if any)

  1. Enable nested tables.
  2. Insert a table.
  3. Copy it.
  4. Paste it to a cell.

βœ”οΈ Expected result

The table is pasted to the cell.

❌ Actual result

Only cells are pasted.

πŸ“ƒ Other details

I believe it happens because of our current implementation of pasting tables.

Related - #3232.


If you'd like to see this fixed sooner, add a πŸ‘ reaction to this post.

@Mgsy Mgsy added type:improvement This issue reports a possible enhancement of an existing feature. domain:v4-compatibilty labels Feb 11, 2021
@Reinmar
Copy link
Member

Reinmar commented Feb 11, 2021

We're currently implementing the same behavior that Word has. It's a bit unintuitive to me, but it makes a lot of sense.

@Mgsy Mgsy added this to the unknown milestone Feb 11, 2021
@Reinmar Reinmar added domain:v4-compatibility This issue reports a CKEditor 4 feature/option that's missing in CKEditor 5. and removed domain:v4-compatibilty labels Mar 9, 2021
@Mgsy Mgsy closed this as completed Apr 14, 2021
@Mgsy Mgsy removed this from the unknown milestone Apr 14, 2021
@Mgsy Mgsy added the resolution:invalid This issue is invalid (e.g. reports a non-existent bug or a by-design behavior). label Apr 14, 2021
@joannazinsli
Copy link

What does the resolution:invalid label mean? Should this item be open and not closed? I tested copy and pasting nested tables in MS Word, and they paste the copied nested table inside the cell. On the CKEditor demo https://ckeditor.com/ckeditor-5/demo/feature-rich/, the copied nested table pastes in any adjacent cells, losing the nested behavior.
A workaround is to create an empty nested table and paste there, but that isn't expected.

@cbushmak
Copy link

What does the resolution:invalid label mean? Should this item be open and not closed? I tested copy and pasting nested tables in MS Word, and they paste the copied nested table inside the cell. On the CKEditor demo https://ckeditor.com/ckeditor-5/demo/feature-rich/, the copied nested table pastes in any adjacent cells, losing the nested behavior.

Agreed, I don't think this is resolved. Word only pastes the table inline when you have cells highlighted, otherwise it nests the table. And the workaround to create an empty nested table and paste there doesn't preserve any attributes that were on the table you pasted, like the class.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
domain:v4-compatibility This issue reports a CKEditor 4 feature/option that's missing in CKEditor 5. package:table resolution:invalid This issue is invalid (e.g. reports a non-existent bug or a by-design behavior). type:improvement This issue reports a possible enhancement of an existing feature.
Projects
None yet
Development

No branches or pull requests

4 participants