[wp-trac] Re: [WordPress Trac] #8850: WYSIWYG editor fails to correct invalid video embed code

[WordPress Trac]

#8850: WYSIWYG editor fails to correct invalid video embed code
--------------------------------------------------------+-------------------
 Reporter:  fd                                          |        Owner:  azaozz
     Type:  defect (bug)                                |       Status:  new   
 Priority:  normal                                      |    Milestone:  2.8   
Component:  TinyMCE                                     |      Version:  2.7   
 Severity:  normal                                      |   Resolution:        
 Keywords:  youtube embed tinymce editor video wysiwyg  |  
--------------------------------------------------------+-------------------
Comment (by azaozz):

 Replying to [comment:2 fd]:
 > It ''would'' be just an HTML validity issue if all it was doing was
 removing the <embed> tag. But it's doing a lot more than that.
 ...
 > Also, it has added data and type attributes to the (already valid)
 object tag. I didn't put them there and there is no reason for them to be
 there. They're optional attributes.
 >
 > It ''added'' a whole extra <param> with name="src" for no good reason.
 >
 > And, crucially I think, it ''deleted'' the (perfectly valid) <param>
 with name="movie" pointing to the actual source of the video.

 Unfortunately the `<object>` tag is not supported well in many browsers.
 Only a few seem to understand the `movie` param but most understand `src`.
 Also it seems some browsers need `data` and `type` to be able to show the
 video.

 > I'm going to stick with "mangling" when describing this behavior.

 It seems to produce a valid `<object>` tag from an invalid one pasted by
 the user (from YouTube or not). It also seems to preserve all attributes
 and makes it compatible with all popular browsers without using invalid
 HTML. The only bug seems to be the double `allowFullScreen` param that
 most likely will be fixed in the next version.

-- 
Ticket URL: <http://trac.wordpress.org/ticket/8850#comment:5>
WordPress Trac <http://trac.wordpress.org/>
WordPress blogging software