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

Table cells are shifted when media is embedded over multiple cells #15472

Open
michalbilik opened this issue Dec 7, 2023 · 1 comment
Open
Labels
domain:typing/ime This issue reports a problem with standard typing & IME (typing method for CJK languages). package:media-embed package:table squad:core Issue to be handled by the Core team. status:stale type:bug This issue reports a buggy (incorrect) behavior.

Comments

@michalbilik
Copy link
Contributor

michalbilik commented Dec 7, 2023

📝 Provide detailed reproduction steps (if any)

  1. Open classic editor example.
  2. Create a table, or use example table:
editor.setData('<figure class="table"><table><tbody><tr><td>1</td><td>2</td><td>3</td></tr><tr><td>4</td><td>5</td><td>6</td></tr><tr><td>7</td><td>8</td><td>9</td></tr></tbody></table></figure>')
  1. Select any two cells within the table.
  2. Paste media content (e.g., a YouTube video link) using CMD + V.

✔️ Expected result

The media content should be embedded either in the first selected cell or in all cells that are chosen.

❌ Actual result

The media content is not embedded, and cells are shifted.

Screen.Recording.2023-12-07.at.12.58.03.mov

📃 Other details

This behaviour was introduced in v34.0.0, but in v33.0.0 same steps crashed the table.

Might be related to #3224.

  • Browser: All
  • OS: macOS
  • First affected CKEditor version: v34.0.0

If you'd like to see this fixed sooner, add a 👍 reaction to this post.

@michalbilik michalbilik added type:bug This issue reports a buggy (incorrect) behavior. package:table package:media-embed release:potential-blocker This issue potentially blocks the upcoming release (should be checked). squad:core Issue to be handled by the Core team. labels Dec 7, 2023
@pomek pomek added domain:typing/ime This issue reports a problem with standard typing & IME (typing method for CJK languages). and removed release:potential-blocker This issue potentially blocks the upcoming release (should be checked). labels Dec 8, 2023
@CKEditorBot
Copy link
Collaborator

There has been no activity on this issue for the past year. We've marked it as stale and will close it in 30 days. We understand it may still be relevant, so if you're interested in the solution, leave a comment or reaction under this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
domain:typing/ime This issue reports a problem with standard typing & IME (typing method for CJK languages). package:media-embed package:table squad:core Issue to be handled by the Core team. status:stale type:bug This issue reports a buggy (incorrect) behavior.
Projects
None yet
Development

No branches or pull requests

3 participants