[wp-trac] [WordPress Trac] #32417: Add new core media widget

WordPress Trac noreply at wordpress.org
Thu Oct 15 21:17:07 UTC 2015


#32417: Add new core media widget
----------------------------------------+---------------------------------
 Reporter:  melchoyce                   |       Owner:  wonderboymusic
     Type:  feature request             |      Status:  assigned
 Priority:  normal                      |   Milestone:  4.4
Component:  Widgets                     |     Version:  4.3
 Severity:  normal                      |  Resolution:
 Keywords:  needs-unit-tests has-patch  |     Focuses:  ui, administration
----------------------------------------+---------------------------------

Comment (by Fab1en):

 Thanks @DrewAPicture for your comments.

 Issue 1. was due to the preview container : using <p> was preventing the
 audio and video players to be appended inside this container.
 Issue 6. was due to click event bound to the image only at startup, and
 not after changing the image.

 This media widget has been designed to support just one single media. The
 fields that are saved in the widget are
 * widget title
 * attachment id (single)
 * size ("thumbnail", "large", etc, for images)
 * link ("file", "post" or custom link)
 * Open link in a new tab or window

 If we want to use caption, galleries and playlist, we need to completely
 change this mechanism to store a shortcode and/or a bunch of HTML. I would
 be happy to go this way, but I just want to be sure that this is what
 everybody wants. That would mean to thrown away a lot of current patch
 code, and probably generate the shortcode preview via ajax.

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


More information about the wp-trac mailing list