There is a issue about adding a configuration about wysiwyg module http://drupal.org/node/1434118. I think it will be great to do such a thing for CKEditor module too.

CommentFileSizeAuthor
#6 media-ckeditor_integration-1988792-6.patch2.17 KBboychev
#5 media-ckeditor_integration-1988792-5.patch2.2 KBboychev
#4 media-ckeditor_integration-1988792-4.patch1.86 KBboychev
#1 media-ckeditor_integration-1988792-1.patch1.9 KBboychev

Comments

boychev’s picture

boychev commented
StatusFileSize
new media-ckeditor_integration-1988792-1.patch1.9 KB

Here is my solution. The patch adds CKEditor integration but only for browser plugins. The new things are checkboxes added to the media config page /admin/config/media/browser for each browser plugin. We can now enable only the plugins we need.

boychev’s picture

boychev commented
Status: Active » Needs review

Needs review

Status: Needs review » Needs work

The last submitted patch, media-ckeditor_integration-1988792-1.patch, failed testing.

boychev’s picture

boychev commented
Status: Needs work » Needs review
StatusFileSize
new media-ckeditor_integration-1988792-4.patch1.86 KB

The patch is fixed. ready for a new test

boychev’s picture

boychev commented
StatusFileSize
new media-ckeditor_integration-1988792-5.patch2.2 KB

Some improvements of the patch

boychev’s picture

boychev commented
StatusFileSize
new media-ckeditor_integration-1988792-6.patch2.17 KB

We don't need to check if ckeditor module exist.

boyan.borisov’s picture

boyan.borisov commented

I've tested this one and it works fine! Thanks!

renatovargas’s picture

renatovargas commented

There's an CKEditor's issue to add support to the WYSIWYG configuration inside its module (taking another approach to this problem) at #1649464: Allow media-7.x-2.x to configure browser plugins for CKeditor plugin.
If we're going to add support inside Media, I think it would be great to support the other options too (file directory and allowed types).

devin carlson’s picture

devin carlson commented
Issue summary: View changes
Status: Needs review » Closed (duplicate)
Related issues: +#1504696: Integration with CKEditor module

This will be taken care of in #1504696: Integration with CKEditor module.