Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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.
Comment | File | Size | Author |
---|---|---|---|
#28 | Node Embed WYSIWYG.jpg | 47.22 KB | alexjarvis |
#27 | node_embed_wysiwyg.patch | 7.7 KB | jcnventura |
#27 | icon.gif | 990 bytes | jcnventura |
#22 | node_embed.tgz | 9.92 KB | jec006 |
Comments
Comment #1
gooddesignusa CreditAttribution: gooddesignusa commentedI 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.
Comment #2
jrabeemer CreditAttribution: jrabeemer commented+1 FCKEditor is deprecated. WYSIWYG is the future.
Comment #3
Nick_vh+1 To clarify we would like to work towards supporting WYSIWIG module in general if this is possible. Not only CKEditor..!
Comment #4
geerlingguy CreditAttribution: geerlingguy commentedSubscribe - 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.
Comment #5
Manuel Garcia CreditAttribution: Manuel Garcia commentedDefinetly +1 on this. If this is fixed, the module would prove 10 times more useful to everyone.
Comment #6
BMorse CreditAttribution: BMorse commented+1 for WYSISYG API support.
Comment #7
Scott J CreditAttribution: Scott J commentedMe too...
Comment #8
aaronp CreditAttribution: aaronp commented+1 Agreed.
Comment #9
jstoller+1 I am dying to use this module, but lack of WYSIWYG support could kill it for me.
Comment #10
ha5bro CreditAttribution: ha5bro commentedYes, WYSIWYG is must have. Very excited to try this.
Comment #11
BenK CreditAttribution: BenK commentedSubscribing....
Comment #12
bryancasler CreditAttribution: bryancasler commented+1 for all the reasons listed above.
Comment #13
talatnat CreditAttribution: talatnat commentedAnother vote for WYSIWYG
Comment #14
momper CreditAttribution: momper commented+1 for single CKEditor module - much more to the point then WYSIWYG
Comment #15
1timer CreditAttribution: 1timer commented+1 for single CKEditor module - seems as though if FCKeditor is supported that CKeditor might be relatively easy to add(?)
Comment #16
rcross CreditAttribution: rcross commented+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.
Comment #17
couloir007 CreditAttribution: couloir007 commented+1 WYSIWYG.
Comment #18
emackn CreditAttribution: emackn commentedshould have an update on this in the next couple of days.
Comment #19
pkej CreditAttribution: pkej commentedSubscribing and anticipating the new version.
Comment #20
emackn CreditAttribution: emackn commentedadded ckeditor plugin support with 1.2, still no word on wysiwyg yet.
Comment #21
aaron CreditAttribution: aaron commentedNode Reference / Embed Media Browser supports WYSIWYG, and uses the same filter as Node Embed, allowing for easy migration.
Comment #22
jec006 CreditAttribution: jec006 commentedI'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.
Comment #23
febbraro CreditAttribution: febbraro commentedjec006 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
Comment #24
apadernoFYI, I have made jec006 co-maintainer of this project.
Comment #25
pwolanin CreditAttribution: pwolanin commentedAny update on getting this revised functionality committed to CVS?
Comment #26
jcnventura CreditAttribution: jcnventura commented@pwolanin: this got committed by jec006 back in September 8. I couldn't get it to work anyway.
Comment #27
jcnventura CreditAttribution: jcnventura commentedI 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. :)
Comment #28
alexjarvis CreditAttribution: alexjarvis commentedUsing 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)
Comment #29
Danny EnglanderThe patch in #27 worked for me.
Comment #30
jec006 CreditAttribution: jec006 commentedHey 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).
Comment #31
jec006 CreditAttribution: jec006 commentedI've created a new release (6-x--1-4) to resolve this issue. Marking as fixed.