Hi,
When Comment driven is used, I loose the WYSIWYG on the comment form. But I do not use the module Wysiwyg (http://drupal.org/project/wysiwyg) like #747616: Compatibility with WYSIWYG, I use the module CKEditor (http://drupal.org/project/ckeditor).

Drupal 6.19
Comment driven 6.x-1.x-dev (2010-Jul-11)
Driven API 6.x-1.x-dev (2010-Jul-11)
CKEditor - WYSIWYG HTML editor 6.x-1.1

Any idea?

CommentFileSizeAuthor
#8 cked.png50.15 KBDarthDrupal

Comments

j0nathan’s picture

Status: Active » Closed (won't fix)

No more needed for me.
Feel free to reopen if needed.

DarthDrupal’s picture

Version: 6.x-1.x-dev » 6.x-1.0-alpha3
Status: Closed (won't fix) » Active

Hi,
same problem here...
J0nathan did you solve the problem?

Thanks in advance

j0nathan’s picture

Hi,
I no more use these modules.
I did not solve the problem.

menteora’s picture

Hi,
i have the same problem,
can anyone suggest a solution?

Thanks

anybody’s picture

The Problem still exists, even using the newest version of CKeditor and this module!

The Comment Text area has the following classes, but no CKeditor appears!

class="form-textarea resizable ckeditor-mod checkxss1 textarea-processed"

Could anyone please have a look at it? I have no clue even after searching for the reasons for a long period ... :(

menteora’s picture

Status: Active » Closed (fixed)

This problem has been solved with 6.x-1x-dev (2011-Feb-25) version.
Special thanks to DarthDrupal who helped me to find the problem.

menteora’s picture

Version: 6.x-1.0-alpha3 » 6.x-1.x-dev
Status: Closed (fixed) » Active

Sorry, the version 6.x-1x-dev (2011-Feb-25) doesn't solve the problem.
We are going to analyze the problem again.

DarthDrupal’s picture

StatusFileSize
new50.15 KB

Sorry wrong way.... the problem is not solved...
The cause of this behaviour is that the id of the textarea inside the comment form have as value "edit-comment", while the right value should be "edit-cdriven-comment" to allow ckeditor to detect it.
Where is the right place in the code to modify this ID?

arhak’s picture

DarthDrupal’s picture

It works!
but when could be necessary to revert the IDs?

arhak’s picture

@#10
as stated in that comment, when themers come in
you may track down its origin at #733354-8: parents' names should be form_clean_id compliant (i.e. no colon)

charos’s picture

Status: Active » Needs work

The patch causes a side-effect with Notifications. Once you disable the line , selection of notifying users don't work ( e.g. everyone who uses OA). I suspect that it can cause issues with other modules that their data interfere with driven comment

charos’s picture

Status: Needs work » Active

sorry, I should have put this in the post regarding BUE editor