Skip to content
This repository has been archived by the owner on Jun 26, 2020. It is now read-only.

Internal: Disabled clipboardInput when editor is read-only. #47

Merged
merged 1 commit into from
Jun 4, 2018

Conversation

Reinmar
Copy link
Member

@Reinmar Reinmar commented Jun 4, 2018

Suggested merge commit message (convention)

Internal: Disabled clipboardInput when editor is read-only. Closes ckeditor/ckeditor5#2679.


Additional information

Part of ckeditor/ckeditor5-image#209.

@Reinmar
Copy link
Member Author

Reinmar commented Jun 4, 2018

@oskarwrobel, why is it needed for ckeditor/ckeditor5-image#209? It seems to fix a different issue (and I'm not sure I know what issue).

@@ -194,6 +194,29 @@ describe( 'Clipboard feature', () => {
sinon.assert.notCalled( spy );
} );

it( 'stops `clipboardInput` event on highest priority when editor is read-only', () => {
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why? Why wasn't that old return; enough?

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling e0cfac5 on t/ckeditor5-image/208 into ae4ea85 on master.

@Reinmar
Copy link
Member Author

Reinmar commented Jun 4, 2018

OK, I can see that without this change image's tests fail. Still, I'd like to know why :D

@oskarwrobel
Copy link
Contributor

I created a dedicated issue https://github.com/ckeditor/ckeditor5-clipboard/issues/48.

@Reinmar Reinmar merged commit b40ec4b into master Jun 4, 2018
@Reinmar Reinmar deleted the t/ckeditor5-image/208 branch June 4, 2018 14:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Do not fire clipboardInput event when editor is read-only
3 participants