This afternoon I was working on the "Template Image"-integration for CKEditor. This patch is the result.
I'm sorry but this patch is just for wysiwyg+CKEditor...

It does:

  • new field "fid" in {wysiwyg_template}
  • create new image-style "wysiwyg_template_thumbnail"
  • lists the image in the "Template"-dialog of CKEditor

It does not:

  • update {wysiwyg_template} if an older version of the module was installed before

Hope this is a good start for the Template-Image-Integration in wysiwyg_template

CommentFileSizeAuthor
#3 Template button when pressed36.37 KBbtopro
#3 WYSIWYG Template input form145.71 KBbtopro
#3 template-image-integration-1218936-3.patch15.93 KBbtopro
wysiwyg_template_image.patch7.05 KBmathiasgmeiner

Comments

guybedford’s picture

guybedford commented

Very nice - thanks for this functionality.

Will certainly look at adding this to the next release.

imp7’s picture

imp7 commented

Hope to see this feature in makes the plugin so much nicer for visual minded editors.

btopro’s picture

btopro commented
Version: 7.x-2.4 » 7.x-2.7
Status: Active » Needs review
StatusFileSize
new template-image-integration-1218936-3.patch15.93 KB
new WYSIWYG Template input form145.71 KB
new Template button when pressed36.37 KB

Original patch has been refactored against 2.7. File IDs can now be exported, imported, and files can be added / deleted per template added. Also fixes glitches with original patch and removes needless tabs from the module replacing them with spaces. I've attached the patch and two screenshots of what this patch will give you.

Elder Brother’s picture

Elder Brother commented

Are there any obvious pointers someone could suggest to amend this patch for the latest 6.x version?

Thanks.

btopro’s picture

btopro commented

http://drupal.org/node/1645916 -- this patch provides very basic support for template images in elms. It does this by adding the property automatically when rendering with a link to the templatename lowercase without spaces and .png.

Again, this is extremely simplistic and the 7.x patch is much more robust but it's at least something to get by. Because of the increased file support in D7 it'll be slightly more annoying to do. Also there's no core imagecache support in 6 so images would have to be the right size (or create more dependencies).

guybedford’s picture

guybedford commented
Status: Needs review » Reviewed & tested by the community

Thanks so much for this - it is very welcomed functionality.

It all works nicely - there was a minor bug I had to fix to ensure there were no error messages when there is no image for a template.

I've applied the changes, and pushed it out in the 7.8 release.

btopro’s picture

btopro commented

ahh, yeah commit to alter patch made sense. Wish there was a better way of packaging images as exportables but that's not an issue with this module so much as an issue with bundling static files with deployment routines (via features at least).

Thanks for getting that committed, look forward to a 2.8 release (i see its tagged as it but no release node)

guybedford’s picture

guybedford commented

Yeah it's there on the releases page - http://drupal.org/node/487184/release, and will come up publicly soon.

It just takes Drupal some time to push it out there.

guybedford’s picture

guybedford commented
Status: Reviewed & tested by the community » Closed (fixed)
guybedford’s picture

guybedford commented
Issue summary: View changes

update: does not...