Link to CKEditor API: http://docs.cksource.com/ckeditor_api/

CommentFileSizeAuthor
#2 718372.patch1.45 KBAnonymous (not verified)

Comments

markus_petrux’s picture

markus_petrux commented
Status: Active » Needs work

We need to extend the following functions in dirtyforms.js:

- Drupal.dirtyForms._wysiwyg.isEditor
- Drupal.dirtyForms._wysiwyg.getContents

Sadly, I do not use CKEditor myself and I'm constrained because of lack of time to investigate this further. Patches are welcome!

Anonymous’s picture

Anonymous (not verified) commented
Assigned: Unassigned »
Status: Needs work » Needs review
StatusFileSize
new 718372.patch1.45 KB

here is a patch to support ckeditor

mkalkbrenner’s picture

mkalkbrenner
Primary language German
Location 🇩🇪
commented
Status: Needs review » Reviewed & tested by the community

Patch works fine.

markus_petrux’s picture

markus_petrux commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS. Thanks for the patch, and for the feedback! :D

sittard’s picture

sittard commented

Brilliant thanks.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

sittard’s picture

sittard commented
Status: Closed (fixed) » Active

Just noticed that this patch only works with the CKEditor module (http://drupal.org/project/ckeditor) and not the WYSIWYG module plus CKEditor (http://drupal.org/project/wysiwyg).

markus_petrux’s picture

markus_petrux commented
Status: Active » Postponed (maintainer needs more info)

Could you please elaborate? Isn't it transparent to Wysiwyg API and what matters here is how to get access to each editor api? If not, then what makes CKEditor in Wysiwyg API different? I'm not sure to understand...

sittard’s picture

sittard commented

Just did some testing on a vanilla install and it works fine. Sorry about that - think I had some old JS files being cached which created the problem on my development site.

Anyhow it's working now.
Thanks.

markus_petrux’s picture

markus_petrux commented
Status: Postponed (maintainer needs more info) » Closed (fixed)

Good to know. Cheers