[wp-trac] [WordPress Trac] #28905: Audio/Video MCE views should use iframe sandboxes

WordPress Trac noreply at wordpress.org
Tue Jul 15 22:17:59 UTC 2014


#28905: Audio/Video MCE views should use iframe sandboxes
----------------------------+-------------------------
 Reporter:  wonderboymusic  |       Owner:
     Type:  defect (bug)    |      Status:  new
 Priority:  normal          |   Milestone:  4.0
Component:  Media           |     Version:  trunk
 Severity:  normal          |  Resolution:
 Keywords:  has-patch       |     Focuses:  javascript
----------------------------+-------------------------

Comment (by wonderboymusic):

 In [changeset:"29179"]:
 {{{
 #!CommitTicketReference repository="" revision="29179"
 Simplify creation of audio, video, and playlist MCE views by placing them
 in iframe sandboxes.

 Wins:
 * Eliminates duplication of code between PHP and JS
 * Views can load JS without messing with TinyMCE and scope
 * MEjs doesn't break when it loads a file plugin-mode. This allows any
 file type the MEjs supports to play in MCE views.
 * YouTube now works as the source for video.
 * Users can still style the views, editor stylesheets are included in
 these sandboxes.
 * Audio and Video URLs and `[embed]`s are no longer broken.
 * Remove the crazy compat code necessary to determine what file types play
 in what browser.
 * Remove unneeded Underscore templates.
 * Remove the compat code for playlists.

 See #28905.
 }}}

--
Ticket URL: <https://core.trac.wordpress.org/ticket/28905#comment:12>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list