diff options
author | Hailey <me@haileyok.com> | 2024-02-06 11:05:17 -0800 |
---|---|---|
committer | GitHub <noreply@github.com> | 2024-02-06 11:05:17 -0800 |
commit | 856f80fc6df731b1dbe9efa289ad6a4f728d4e0d (patch) | |
tree | 815d7ed5b083fe36cca321efa60042d14b22c16e /src/view/com/util/post-embeds/ExternalPlayerEmbed.tsx | |
parent | a9ab13e5a936c4d917b878bd53f4e536fa8c95f8 (diff) | |
download | voidsky-856f80fc6df731b1dbe9efa289ad6a4f728d4e0d.tar.zst |
fix some youtube videos not properly loading (#2726)
* add player iframe to bskyweb * iframe for youtube content * update tests * ts error
Diffstat (limited to 'src/view/com/util/post-embeds/ExternalPlayerEmbed.tsx')
-rw-r--r-- | src/view/com/util/post-embeds/ExternalPlayerEmbed.tsx | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/src/view/com/util/post-embeds/ExternalPlayerEmbed.tsx b/src/view/com/util/post-embeds/ExternalPlayerEmbed.tsx index 8b0858b69..d556e7669 100644 --- a/src/view/com/util/post-embeds/ExternalPlayerEmbed.tsx +++ b/src/view/com/util/post-embeds/ExternalPlayerEmbed.tsx @@ -78,9 +78,13 @@ function Player({ onLoad: () => void }) { // ensures we only load what's requested + // when it's a youtube video, we need to allow both bsky.app and youtube.com const onShouldStartLoadWithRequest = React.useCallback( - (event: ShouldStartLoadRequest) => event.url === params.playerUri, - [params.playerUri], + (event: ShouldStartLoadRequest) => + event.url === params.playerUri || + (params.source.startsWith('youtube') && + event.url.includes('www.youtube.com')), + [params.playerUri, params.source], ) // Don't show the player until it is active |