Creating a new issue from comment http://drupal.org/node/524410#comment-3016212

With Embedded Inline Media as the top (first) filter, and with YouTube embeds working normally, there's a bug with embedding the videos tightly between html tags. What I mean is that:

<p class="rteleft">http://www.youtube.com/watch?v=8iU-N_tJET0</p> won't work, whereas

<p class="rteleft"> http://www.youtube.com/watch?v=8iU-N_tJET0 </p> will work (notice the two spaces), and
<p class="rteleft">
http://www.youtube.com/watch?v=8iU-N_tJET0
</p> also will work (because of the line breaks).

This becomes an issue, particularly for non-technical users, when the WYSIWYG (http://drupal.org/project/fckeditor in my case) removes the white space or line breaks when editing the page. In other words, when white space is added to make the embed work, if somebody edits the page anywhere, after saving the embed won't work anymore.

Currently I'm giving instructions to ensure that YouTube links are not enclosed in any html tags at all, which is a little problematic for users who are not comfortable with the "Source" WYSIWYG view.

Comments

johnbarclay’s picture

johnbarclay commented

I could not replicate this with 6.x-1.20 on drupal 6.17 with the embedded inline filter first.

EndEd’s picture

EndEd commented
trolster’s picture

trolster commented

I have been able to replicate this with several editors (TinyMCE, CKeditor and FCKeditor). As far as I can tell a "clean" p-tag, that is, a the tag without any attributes, needs no whitespace to work. Once you get attributes in there, it only works with whitespaces on both sides.

xpla’s picture

xpla commented

+1

brightbold’s picture

brightbold
she/her
Primary language English
Location Boston, MA
commented

I can't get this to work even with an attribute-free p tag, although I confess I can't quite tell whether I'm suffering from this problem or from this one (or possibly both - but hard to tell until I can get one of them fixed!) CKEditor clears out all the white space, and I need my non-technical content entry people to be able to embed video, so it would be great if there were a solution to this.

danny englander’s picture

danny englander
(he/him)
commented

Subscribing

geerlingguy’s picture

geerlingguy commented
Status: Active » Closed (duplicate)

Marking as a duplicate of #811090: Embedded Inline Media filter regex out of date - URL can't be wrapped in HTML tags - please continue further discussion there. I'm going to try patching the regex to get it to work better. I've submitted a patch to that issue - please test and see if it works for you!