Implement youtube embeds

[OliverUv]

Currently there are YouTube previews, but no embedded player. Implement an opt-in for this to compete effectively.

There should be a warning about risks involved in running YouTube's JS in the app page.

iframe possible to cordon off access to parent JS environment?

[MTRNord]

Maybe use a dep like https://github.com/troybetz/react-youtube ?

[BloodyIron]

Being able to watch youtube and other videos in the in-line chat window is a very big convenience experienced by others using Discord and other such alternatives. I really want to see this in Riot as I'm looking to set this up for us (on-prem).

[t3chguy]

/me adds to Todo List

[BloodyIron]

Any word on this? :S

Perhaps can we get this set to a milestone?

[t3chguy]

I'll revive this tomorrow I already had it working.

[BloodyIron]

Pssst, any word @t3chguy ? :)

[t3chguy]

I have to merge it with Dave's old efforts to form some generic behemoth - it'll be after exams (3-4 weeks or so)

[fuzzy76]

How about oembed (YouTube supports it)? That would open up preview for a lot of sites.

[Arritmic]

Sorry, do we know anything about the youtube preview support? Thanks.

[BloodyIron]

I too am curious.

[dotMavriQ]

I'd love to know this as well

[t3chguy]

My work on this bitrotted too much

[BloodyIron]

@t3chguy surely something can be salvaged?

[turt2live]

It's currently being pursued as https://github.com/vector-im/riot-web/issues/10060 (in fact, the demo was with youtube embeds)

[t3chguy]

That doesn't sound like the best UX though, means you have to use a compatible client and send a specially formatted message instead of just a plaintext YouTube URL

[turt2live]

What makes you say that?

• https://github.com/matrix-org/matrix-react-sdk/pull/3252/files#diff-dac7e2daba0bfb4d51105608b4bc2007R54-R76
• https://github.com/matrix-org/matrix-react-sdk/pull/3252/files#diff-fa2e6c7a24d3ee8c284aa720b8f399edR1097-R1101

[turt2live]

(there's no reason why we can't convert events for better UX like this)

[t3chguy]

ah okay, I was just speculating based on the intro of the MSC, did not look through the code