[wp-trac] [WordPress Trac] #49852: Use feature image of video attachment as preview instead of icon
WordPress Trac
noreply at wordpress.org
Mon Apr 20 17:35:02 UTC 2020
#49852: Use feature image of video attachment as preview instead of icon
-------------------------------------------------+-------------------------
Reporter: spacedmonkey | Owner: (none)
Type: enhancement | Status: new
Priority: normal | Milestone: Awaiting
| Review
Component: Media | Version:
Severity: normal | Resolution:
Keywords: good-first-bug needs-refresh has- | Focuses:
patch |
-------------------------------------------------+-------------------------
Changes (by johnbillion):
* keywords: needs-patch good-first-bug => good-first-bug needs-refresh
has-patch
Comment:
Thanks for the patch @samful and welcome to WordPress Trac!
Some feedback on your patch, as it's your first one:
* It looks like the formatting of your patch file isn't right, here's a
guide for creating a patch file correctly:
https://make.wordpress.org/core/handbook/tutorials/trac/submitting-a-patch/
* Not all videos will have a featured image, so we'll need some logic
which only uses the featured image ID if one exists, and still uses the ID
of the video attachment if not so the correct icon is displayed.
* Audio files can actually have a featured image too, although this is not
a commonly used feature. They're used if you create an audio playlist in
the classic editor. We might as well add support for featured images for
audio too.
Feel free to resubmit your patch using the same file name, and just ask
here if you have any questions.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/49852#comment:2>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list