[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