To make this module usable by a larger universe of Drupal sites, it is desirable to allow additional editors to be used. Providing support for the WYSIWYG module would be one effective path for achieving this goal.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

gooddesignusa’s picture

I would imagine a lot of people who are using FCKeditor will eventually end up switching to the WYSIWYG module. I personally need the extra settings that WYSIWYG module doesn't provide.
I use CKeditor which is the newer rebuilt version of fckeditor.

http://cksource.com/about

http://drupal.org/project/ckeditor

The module does sound useful but I'll have to wait until other editors are supported before I can try it out.

jrabeemer’s picture

Title: Remove dependancy on FCKeditor » Remove dependancy on FCKeditor, support WYSIWYG

+1 FCKEditor is deprecated. WYSIWYG is the future.

Nick_vh’s picture

+1 To clarify we would like to work towards supporting WYSIWIG module in general if this is possible. Not only CKEditor..!

geerlingguy’s picture

Subscribe - for some sites, I use CKEditor (drupal module + editor) since it has a bit more advanced customization options.

But as time goes on, and WYSIWYG support for more advanced features (like custom toolbar settings) gets better, I will still use the CKEditor editor, but wrapped in WYSIWYG. So WYSIWYG module support would be a very good idea.

Manuel Garcia’s picture

Definetly +1 on this. If this is fixed, the module would prove 10 times more useful to everyone.

BMorse’s picture

+1 for WYSISYG API support.

Scott J’s picture

Me too...

aaronp’s picture

+1 Agreed.

jstoller’s picture

+1 I am dying to use this module, but lack of WYSIWYG support could kill it for me.

ha5bro’s picture

Yes, WYSIWYG is must have. Very excited to try this.

BenK’s picture

Subscribing....

bryancasler’s picture

+1 for all the reasons listed above.

talatnat’s picture

Another vote for WYSIWYG

momper’s picture

+1 for single CKEditor module - much more to the point then WYSIWYG

1timer’s picture

+1 for single CKEditor module - seems as though if FCKeditor is supported that CKeditor might be relatively easy to add(?)

rcross’s picture

+1 - just came across this module and realized that we extended the nodepicker.module to do this. Would've been much better to use this module instead if it integrated with WYSIWYG api.

couloir007’s picture

+1 WYSIWYG.

emackn’s picture

should have an update on this in the next couple of days.

pkej’s picture

Subscribing and anticipating the new version.

emackn’s picture

added ckeditor plugin support with 1.2, still no word on wysiwyg yet.

aaron’s picture

Node Reference / Embed Media Browser supports WYSIWYG, and uses the same filter as Node Embed, allowing for easy migration.

jec006’s picture

FileSize
9.92 KB

I've implemented the Wysiwyg api and have everything working. I couldn't create a diff because there are new folders and I don't have CVS access. I've attached a tar with the changed module instead.

febbraro’s picture

jec006 thanks for this (and the CKEditor integration) if you apply for a CVS account I'll make you a co-maintainer.

http://drupal.org/cvs-application/requirements

apaderno’s picture

FYI, I have made jec006 co-maintainer of this project.

pwolanin’s picture

Status: Active » Needs review

Any update on getting this revised functionality committed to CVS?

jcnventura’s picture

Status: Needs review » Needs work

@pwolanin: this got committed by jec006 back in September 8. I couldn't get it to work anyway.

jcnventura’s picture

Status: Needs work » Reviewed & tested by the community
FileSize
990 bytes
7.7 KB

I just figured out that the commit was incomplete. The tar.gz file in #22 includes a plugins directory that was never committed to the repository.

The attached file provides a patch that will add the text files of that tgz back, and includes a couple of simple fixes to some persistent warnings introduced by this module.

Note that the node_embed/plugins/images/icon.gif is NOT included in the patch and must be added to the repository before a commit.

I am marking this RTBF, as I have just tested it, and I haven't altered jec006's code. :)

alexjarvis’s picture

Status: Reviewed & tested by the community » Needs work
FileSize
47.22 KB

Using this code with the latest releases of WYSIWYG w/CKEditor I get a Javascript error when clicking on the Node Embed button:

Uncaught TypeError: Cannot read property '$' of null /sites/all/modules/wysiwyg/editors/js/ckeditor-3.0.js?N:186
Drupal.wysiwyg.editor.instance.ckeditor.addPlugin.CKEDITOR.plugins.add.init.pluginCommand.exec /sites/all/modules/wysiwyg/editors/js/ckeditor-3.0.js?N:186
... (see attached image for full error)

Danny Englander’s picture

The patch in #27 worked for me.

jec006’s picture

Hey Guys,

Sorry for the delay - i'm gonna try to fix the check in and make a new release.

I didn't do the original check in of the code and it looks like the new folders contents weren't added (as you said).

jec006’s picture

Status: Needs work » Fixed

I've created a new release (6-x--1-4) to resolve this issue. Marking as fixed.

Status: Fixed » Closed (fixed)
Issue tags: -wysiwyg, -interoperability

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