search

ckeditor / ckeditor4

The best enterprise-grade WYSIWYG editor. Fully customizable with countless features and plugins.
https://ckeditor.com/ckeditor-4
Other
5.79k stars 2.47k forks source link

Failing test: plugins/widget/widgetsintegration [SAFARI] #4815

Closed sculpt0r closed 3 years ago

sculpt0r commented 3 years ago

Type of report

Bug

Provide detailed reproduction steps (if any)

Failing test: test content is removed after cutting (single widget)

Full test path: /plugins/widget/widgetsintegration#tests%2Fplugins%2Fwidget%2Fwidgetsintegration%20test%20content%20is%20removed%20after%20cutting%20(single%20widget)

Expected result

What is the expected result of the above steps?

Test passed.

Actual result

What is the actual result of the above steps?

Test failed.

selection is collapsed
Expected: true (boolean)
Actual:   false (boolean)

Other details

sculpt0r commented 3 years ago

It is not failing on Safari 13.1 🤔 Also, not failing on Chrome, FF.

Failing even back to 4.14.0 so it looks like safari problem here.

f1ames commented 3 years ago

DUP of https://github.com/ckeditor/ckeditor4/issues/4759.

f1ames commented 3 years ago

In fact https://github.com/ckeditor/ckeditor4/issues/4759 lists more than one failing test with not that much information so let's leave this open and close #4759.

We had already concluded that this is new Safari upstream not caused by changes in CKEditor 4 itself. Still, it would be good to check if it's only the test which is failing or something is broken with widgets handling. cc @sculpt0r

sculpt0r commented 3 years ago

Duplicate: https://github.com/ckeditor/ckeditor4/issues/4691