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 a large table from Excel causes whole page to scroll away from editor #17079

Closed
jbalinski opened this issue Sep 11, 2024 · 2 comments · Fixed by #17314
Closed

Pasting a large table from Excel causes whole page to scroll away from editor #17079

jbalinski opened this issue Sep 11, 2024 · 2 comments · Fixed by #17314
Assignees
Labels
package:paste-from-office support:2 An issue reported by a commercially licensed client. type:bug This issue reports a buggy (incorrect) behavior.

Comments

@jbalinski
Copy link

📝 Provide detailed reproduction steps (if any)

  1. Copied a simple table with lots of rows (200) from Excel (Version 2406, Build 17726.20206)
  2. Navigated to CKEditor, making sure to scroll down the page so that the top of the page isn't visible. The editor is completely visible.
  3. Pasted the content into CKEditor
  4. My browser is scrolled to the top of the page. CKEditor is no longer visible.

✔️ Expected result

Do not scroll to the top of the page when pasting in a table.

❌ Actual result

Browser scrolls to the top of the page.

❓ Possible solution

NA

📃 Other details

NOTE: This can be reproduced on the CKEditor 5 demo page here:
https://ckeditor.com/productivity-pack/paste-from-office-enhanced-demo/

  • Browser: Edge 128, Chrome 128, Firefox 130
  • OS: Windows 11
  • First affected CKEditor version: Not sure, but occurs in 43.1.0
  • Installed CKEditor plugins: See plugins used on demo page above.

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

@jbalinski jbalinski added the type:bug This issue reports a buggy (incorrect) behavior. label Sep 11, 2024
@lkszzajac lkszzajac added support:2 An issue reported by a commercially licensed client. package:paste-from-office labels Sep 20, 2024
@Mati365
Copy link
Member

Mati365 commented Oct 24, 2024

Reproducible on tables pasted from LibreOffice Calc in our manual demos. It seems that setting fixed height on .ck-content makes this issue reproducible.

@Mati365
Copy link
Member

Mati365 commented Oct 24, 2024

I proposed fix #17314

Mati365 added a commit that referenced this issue Oct 30, 2024
Fix (utils): No longer scroll to the top of the document if pasted element is larger than scrollable editable. Closes #17079
@CKEditorBot CKEditorBot added this to the iteration 80 milestone Oct 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
package:paste-from-office support:2 An issue reported by a commercially licensed client. type:bug This issue reports a buggy (incorrect) behavior.
Projects
None yet
4 participants