The longdesc attribute is removed [#1398560]

Comment	File	Size	Author
#3	wysiwyg-tinymce-longdesc.1398560.3.patch	792 bytes	twod
#1	wysiwyg.patch	732 bytes	markwittens
	wysiwyg.patch	333 bytes	markwittens

Comment #1

markwittens commented 9 January 2012 at 12:55

Status	File	Size
new	wysiwyg.patch	732 bytes

Last patch was created without the right parameters, attached is the right patch.

Log in or register to post comments

Comment #2

ckrina

she/her

Catalan

Barcelona

commented 12 January 2012 at 09:57

I'm having the same issue in Drupal 7, in CKEditor.
Any ideas to fix it? Thank you.

Log in or register to post comments

Comment #3

twod

Swedish

Sweden

commented 12 January 2012 at 11:17

Version:	6.x-2.4	» 7.x-2.x-dev
Component:	Code	» Editor - TinyMCE
Status:	Needs review	» Reviewed & tested by the community

Status	File	Size
new	wysiwyg-tinymce-longdesc.1398560.3.patch	792 bytes

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.