Posted by markwittens on January 9, 2012 at 12:51pm
4 followers
| Project: | Wysiwyg |
| Version: | 7.x-2.x-dev |
| Component: | Editor - TinyMCE |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
| Issue tags: | needs backport to D6 |
Issue Summary
The longdesc attribute is removed by the editor when using advanced image, if it's added to the extended_valid_elements in tinymce.inc it won't get removed anymore. See the attached patch.
| Attachment | Size |
|---|---|
| wysiwyg.patch | 333 bytes |
Comments
#1
Last patch was created without the right parameters, attached is the right patch.
#2
I'm having the same issue in Drupal 7, in CKEditor.
Any ideas to fix it? Thank you.
#3
Adding longdesc there works and the attribute is valid (though not commonly used AFAIK).
The patch was created from the wrong folder (wysiwyg/editors), instead of the root wysiwyg folder, re-rolled it for you.
I did a quick
grep -r longdesc ckeditor, which revealed that the word 'longdesc' is only used in the iframe plugin, not the image plugin, so I guess CKEditor doesn't support it.CKEditor does not use an externally supplied tag/attribute whitelist in the same matter TinyMCE does anyway so we could probably not have changed it.
#4
Thanks, looks good!
#5
Committed #3 to D5, D6 and D7. Thanks for spotting this!
The patch will be in the -dev snapshots within 12hrs and it'll be part of the next release.