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

Tables pasted from Excel get bogus extra spacing/padding on Chrome #14516

Closed
mlewand opened this issue Jul 3, 2023 · 0 comments · Fixed by #14720
Closed

Tables pasted from Excel get bogus extra spacing/padding on Chrome #14516

mlewand opened this issue Jul 3, 2023 · 0 comments · Fixed by #14720
Assignees
@Dumluregn
Labels
browser:chrome package:paste-from-office package:table squad:features Issue to be handled by the Features team. type:bug This issue reports a buggy (incorrect) behavior. type:regression This issue reports a bug that was not present in the previous releases.
Milestone
iteration 66 (v39...

Comments

@mlewand
Copy link
Contributor

mlewand commented Jul 3, 2023 •
edited
Loading

📝 Provide detailed reproduction steps (if any)

  1. Download Excel (3).xlsx file & open it in Excel.
  2. Select last three columns, like so:

  1. Paste it into https://ckeditor.com/docs/ckeditor5/latest/examples/builds-custom/full-featured-editor.html

✔️ Expected result

Cell widths are preserved (cell with "color on text and row" is wider than the other two).

❌ Actual result

Extra spacing is visible at the right hand side.

📃 Other details

Regression in 38.1.0.

Not reproducible on Firefox.

  • Browser: …
  • OS: …
  • First affected CKEditor version: …
  • Installed CKEditor plugins: …

If you'd like to see this fixed sooner, add a 👍 reaction to this post.
@mlewand mlewand added type:bug This issue reports a buggy (incorrect) behavior. package:table package:paste-from-office browser:chrome type:regression This issue reports a bug that was not present in the previous releases. labels Jul 3, 2023
@mlewand mlewand mentioned this issue Jul 3, 2023
Closed
@Dumluregn Dumluregn mentioned this issue Aug 2, 2023
Merged
@Dumluregn Dumluregn self-assigned this Aug 2, 2023
@Dumluregn Dumluregn added the squad:features Issue to be handled by the Features team. label Aug 2, 2023
@CKEditorBot CKEditorBot added the status:in-progress Set automatically when an issue lands in the "In progress" column. We are working on it. label Aug 2, 2023
mlewand added a commit that referenced this issue Aug 14, 2023
@mlewand
Merge pull request #14720 from ckeditor/ck/14521-support-cell-widths-…
cc319ca 
…in-paste-from-excel

Fix (table,paste-from-office): Tables pasted from MS Excel will have proper column widths. Closes #14521. Closes #14516.
@CKEditorBot CKEditorBot removed the status:in-progress Set automatically when an issue lands in the "In progress" column. We are working on it. label Aug 14, 2023
@CKEditorBot CKEditorBot added this to the iteration 66 milestone Aug 14, 2023
@CKEditorBot CKEditorBot mentioned this issue Aug 15, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Dumluregn Dumluregn

Labels
browser:chrome package:paste-from-office package:table squad:features Issue to be handled by the Features team. type:bug This issue reports a buggy (incorrect) behavior. type:regression This issue reports a bug that was not present in the previous releases.
Projects
None yet
Milestone
iteration 66 (v39.0.2)
Development

Successfully merging a pull request may close this issue.

Support col[span] and pts in paste from excel ckeditor/ckeditor5
3 participants
@mlewand @Dumluregn @CKEditorBot