Pasting from Excel Online comes with extra text

rturella3 commented 1 year ago

📝 Provide detailed reproduction steps (if any)

Open an Excel file in https://onedrive.live.com/, insert some text and copy the table.
Paste in a ckeditor (like https://ckeditor.com/docs/ckeditor5/latest/features/pasting/paste-from-office.html#demo)

The pasted content contains some text and then the table that was copied first.

✔️ Expected result

The extra text is not present.

❌ Actual result

The pasted content contains some text and then the table that was copied first.

❓ Possible solution

Since the extra text looks like css from a style tag, I was able to fix by including the General HTML Support plugin by allowing styletags:

allow: [
            {
                name: 'style',
            },
],

Also, this behavior may be considered the correct behavior, because CKeditor is originally blocking the insertion of a disalowed tag and converting its content to plain text. If that is the case, then it would be nice to have documented in the documentation and how to prevent with the General HTML Support plugin.

📃 Other details

Browser: Edge and Firefox
OS: Windows 10
First affected CKEditor version: 38.1.1
Installed CKEditor plugins: Essentials, Autoformat, Bold, Italic, Underline, Strikethrough, BlockQuote, Heading, Link, List, Paragraph, FontSize, FontColor, FontBackgroundColor, Table, TableToolbar, TableProperties, TableCellProperties, PasteFromOffice, GeneralHtmlSupport.
My client sent me a video showing the bug with Office 365, but I was not able reproduce because I don't have Office 365. I don't know if it happens with Excel. It does not happen with Libre Office.

Witoso commented 1 year ago

Issue confirmed. CKEditor should not populate the editor's content with such data. It should be stripped unless GHS is configured to preserve it.

CKEditorBot commented 1 month ago

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.

CKEditorBot commented 3 hours ago

We've closed your issue due to inactivity. We understand that the issue may still be relevant. If so, feel free to open a new one (and link this issue to it).

ckeditor / ckeditor5