Closed (fixed)
Project:
File Framework
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
23 Dec 2009 at 19:40 UTC
Updated:
23 Mar 2012 at 18:24 UTC
I am using ckeditor with ckeditor- WYSIWYG and the embed functionality no longer works (I was using Fckeditor before).
Currently, upon embedding, it no longer makes a lightbox, but instead fills the whole screen with what was a pop-up such that you can't exit out. Most importantly, I can't get the embed to function.
Comments
Comment #1
jvieille commentedInterested by this feature.
Any follow-up?
Comment #2
jvieille commented+1
Comment #3
jvieille commented+1
Comment #4
mrangryfish commented+1
Comment #5
jvieille commentedThis issue is still there, but only preventing the use of the embed button in the ckeditor editing area.
What is possible is embedding in raw text and then switching to rich text: the image will be there
Comment #6
bejam commentedIs this a matter of changing a few tags in the embed function to recognise "cckeditor" instead of "fckeditor"? It would be good if someone managed to get this working as fck has been replaced with ck. The workaround is not so acceptable for average users who will have a meltdown if they see the raw html.
Comment #7
mrangryfish commentedI've got part way there. It now adds the [file xyz] text into the ckeditor without having to switch to plain text mode. It doesn't actually show the thumbnail/video etc in the ckeditor yet.. or put the embed file into a button in ckeditor. But someone might appreciate this:
in jquery-fieldselection.js add:
In file_embed.js, in the two if/else areas that check for the wysiwyg editor type, add:
and
If I make the wysiwyg ckeditor button or make the embedded files show the actual file content in the editor I'll figure out how to make a patch and add it here. Thought this was worth contributing in the mean time for others. This is my first use of ckeditor javascript api so there may be more 'correct' solutions. For example using the wysiwyg api (2.0) module itself would be cleaner.
Cheers,
Christiaan Kortekaas
Comment #8
mrangryfish commentedI got the file embed ckeditor button and full wysiwyg integration working nicely now too. I'll try and send it to the fileframework maintainers when I get a chance. The above code bits are a bit different now too that I understand the CKEditor API better.
Comment #9
Equinger commentedThanks for your work mrangryfish! Contributing your newest code would be great. Thanks, again!
Comment #10
johanneshahn commentedone up
Comment #11
johanneshahn commentedinserted code from mrangryfish
this works fine with latest ckeditor module 4 drupal from http://ckeditor.com/download
Comment #12
jvieille commentedStill the same: you can embed only in raw text mode.
One would expect that the embedding would apply in rich text mode, at the place the cursor is, but nothing happens in these conditions.
Ideally, a CK Editor button would be fantastic.
Comment #13
johanneshahn commentedHi
Can u try it with the new version from today?
Just forgot to write that
And would u write a new Issue then 4 the Button
Best regards
Johannes
Comment #14
jvieille commentedNow it is OK!
Comment #15
johanneshahn commentedok,
can u create a new issue for ckeditor button?
Comment #16
jvieille commentedIt is there
http://drupal.org/node/1494736
You will be granted the "Best Drupal Maintainer" Award , keeping begging for feature requests!
sorry, seems not to exist yet :(
Comment #17
johanneshahn commentedhi jvieille,
thanx for that really nice post.
i never heard before.
ill do my best :D, u do not have to begging.
greetz johannes