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.
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

chr.fritsch created an issue. See original summary.

chr.fritsch’s picture

Status: Active » Needs review
FileSize
1.02 KB

Here is the missing config

marcoscano’s picture

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

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

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

Priority: Normal » Major
Issue tags: +Media Initiative

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

seanB’s picture

FileSize
694.54 KB

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

Media library remote video

alexpott’s picture

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.