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

Link drag and drop still possible even when links are deactivated in CKEditor profile #4848

Closed
julmot opened this issue Aug 14, 2019 · 1 comment

Comments

@julmot
Copy link
Contributor

commented Aug 14, 2019

Bug Report

When having a WYSIWYG editable within the CMS area, with a CKEditor configuration (JS) without links:

CKEDITOR.editorConfig = function(config) {
  config.removeButtons = 'Form,Checkbox,Radio,TextField,Textarea,Select,Button,HiddenField,Link,Unlink,Anchor,Image,Flash,Table,HorizontalRule,Smiley,SpecialChar,PageBreak,Iframe,Format,Font,FontSize,Styles,About';
};

you can still drag and drop a document from the tree into a WYSIWYG field, even though the link button in the CKEditor toolbar is hidden and thus the feature deactivated.

Expected behavior

It should not be possible to drag and drop documents into a WYSIWYG field with deactivated links.

Actual behavior

It's still possible.

Steps to reproduce

Open a new document and place a brick with a WYSIWYG field on the page. Then drag and drop a document on the field. This is posible – even if the link is deactivated in the CKEditor configuration of the field.

000163

@brusch

This comment has been minimized.

Copy link
Member

commented Aug 19, 2019

The config is just to remove the link button, not the link functionality, that's why it's called removeButtons.
If you'd like to disallow link elements, please have a look at CKEditor's possibilities.

@brusch brusch closed this Aug 19, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.