Problem/Motivation

This is a follow-up to #2964640: Provide a media type for remote video in Standard where we added the configuration for the remote video media type.
We missed adding a media library display for the "Remote video" type. So currently the actual video is displayed in the media library instead of a thumbnail.

Proposed resolution

  • Add the display config.
CommentFileSizeAuthor
#7 media-overview.png694.54 KBseanb
#2 2982279.patch1.02 KBchr.fritsch

Comments

chr.fritsch created an issue. See original summary.

chr.fritsch’s picture

chr.fritsch
he/him
Primary language German
Location 🇩🇪🇪🇺🌍
commented
Status: Active » Needs review
StatusFileSize
new 2982279.patch1.02 KB

Here is the missing config

marcoscano’s picture

marcoscano
Primary language Portuguese, Brazil
Location Barcelona, Spain
commented

Looks good to me, but I wonder if core/profiles/standard/tests/src/Functional/StandardTest (or similar) shouldn't ensure that the types we ship in standard show up as expected in the media library?

chr.fritsch’s picture

chr.fritsch
he/him
Primary language German
Location 🇩🇪🇪🇺🌍
commented

These configs are currently part of the media_library module. I think it's because media_library is currently marked as experimental.

marcoscano’s picture

marcoscano
Primary language Portuguese, Brazil
Location Barcelona, Spain
commented
Status: Needs review » Reviewed & tested by the community

Right! Hadn't paid attention to that detail. I don't see why this shouldn't get in then.

Thanks!

phenaproxima’s picture

phenaproxima
he/him
Primary language English
Location Massachusetts
commented
Priority: Normal » Major
Issue tags: +Media Initiative

Escalating to major. Will try to get eyes on this today.

seanb’s picture

seanb
Primary language Dutch
Location Netherlands
commented
StatusFileSize
new media-overview.png694.54 KB

RTBC +1, just installed a fresh D8 and added some youtube video's.

Media library remote video

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Reviewed & tested by the community » Fixed

I think whilst it is experimental it is okay to ship this without a test. Committed 257a881 and pushed to 8.6.x. Thanks!

  • alexpott committed 257a881 on 8.6.x 
    Issue #2982279 by chr.fritsch, seanB, marcoscano: Provide a media...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.