Skip to content

Commit d92731b

Browse files
authored
[Video] Lexicon implementation (#4881)
* implement AppBskyEmbedVideo lexicon in player * add alt to native player * add prerelease package * update prerelease * add video embed view manually from record * fix type error on example video * black bg + use aspect ratio on web * add video to feeds * fix video overflowing aspect ratio * remove prerelease package --------- Co-authored-by: Samuel Newman <[email protected]>
1 parent b136c44 commit d92731b

File tree

9 files changed

+211
-91
lines changed

9 files changed

+211
-91
lines changed

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@
5252
"open-analyzer": "EXPO_PUBLIC_OPEN_ANALYZER=1 yarn build-web"
5353
},
5454
"dependencies": {
55-
"@atproto/api": "0.13.3",
55+
"@atproto/api": "0.13.5",
5656
"@bam.tech/react-native-image-resizer": "^3.0.4",
5757
"@braintree/sanitize-url": "^6.0.2",
5858
"@discord/bottom-sheet": "bluesky-social/react-native-bottom-sheet",

src/view/com/post-thread/PostThread.tsx

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -428,6 +428,7 @@ export function PostThread({uri}: {uri: string | undefined}) {
428428
(item.ctx.depth < 0 && !!item.parent) || item.ctx.depth > 1
429429
const hasUnrevealedParents =
430430
index === 0 && skeleton?.parents && maxParents < skeleton.parents.length
431+
431432
return (
432433
<View
433434
ref={item.ctx.isHighlightedPost ? highlightedPostRef : undefined}

src/view/com/posts/FeedItem.tsx

Lines changed: 22 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -17,37 +17,37 @@ import {msg, Trans} from '@lingui/macro'
1717
import {useLingui} from '@lingui/react'
1818
import {useQueryClient} from '@tanstack/react-query'
1919

20+
import {isReasonFeedSource, ReasonFeedSource} from '#/lib/api/feed/types'
21+
import {MAX_POST_LINES} from '#/lib/constants'
22+
import {usePalette} from '#/lib/hooks/usePalette'
23+
import {makeProfileLink} from '#/lib/routes/links'
2024
import {useGate} from '#/lib/statsig/statsig'
25+
import {sanitizeDisplayName} from '#/lib/strings/display-names'
26+
import {sanitizeHandle} from '#/lib/strings/handles'
27+
import {countLines} from '#/lib/strings/helpers'
28+
import {s} from '#/lib/styles'
2129
import {POST_TOMBSTONE, Shadow, usePostShadow} from '#/state/cache/post-shadow'
2230
import {useFeedFeedbackContext} from '#/state/feed-feedback'
31+
import {precacheProfile} from '#/state/queries/profile'
2332
import {useSession} from '#/state/session'
2433
import {useComposerControls} from '#/state/shell/composer'
2534
import {useMergedThreadgateHiddenReplies} from '#/state/threadgate-hidden-replies'
26-
import {isReasonFeedSource, ReasonFeedSource} from 'lib/api/feed/types'
27-
import {MAX_POST_LINES} from 'lib/constants'
28-
import {usePalette} from 'lib/hooks/usePalette'
29-
import {makeProfileLink} from 'lib/routes/links'
30-
import {sanitizeDisplayName} from 'lib/strings/display-names'
31-
import {sanitizeHandle} from 'lib/strings/handles'
32-
import {countLines} from 'lib/strings/helpers'
33-
import {s} from 'lib/styles'
34-
import {precacheProfile} from 'state/queries/profile'
35+
import {FeedNameText} from '#/view/com/util/FeedInfoText'
36+
import {PostCtrls} from '#/view/com/util/post-ctrls/PostCtrls'
37+
import {PostEmbeds} from '#/view/com/util/post-embeds'
38+
import {PostMeta} from '#/view/com/util/PostMeta'
39+
import {Text} from '#/view/com/util/text/Text'
40+
import {PreviewableUserAvatar} from '#/view/com/util/UserAvatar'
3541
import {atoms as a} from '#/alf'
3642
import {Repost_Stroke2_Corner2_Rounded as Repost} from '#/components/icons/Repost'
3743
import {ContentHider} from '#/components/moderation/ContentHider'
44+
import {LabelsOnMyPost} from '#/components/moderation/LabelsOnMe'
45+
import {PostAlerts} from '#/components/moderation/PostAlerts'
3846
import {AppModerationCause} from '#/components/Pills'
3947
import {ProfileHoverCard} from '#/components/ProfileHoverCard'
4048
import {RichText} from '#/components/RichText'
41-
import {LabelsOnMyPost} from '../../../components/moderation/LabelsOnMe'
42-
import {PostAlerts} from '../../../components/moderation/PostAlerts'
43-
import {FeedNameText} from '../util/FeedInfoText'
4449
import {Link, TextLink, TextLinkOnWebOnly} from '../util/Link'
45-
import {PostCtrls} from '../util/post-ctrls/PostCtrls'
46-
import {PostEmbeds} from '../util/post-embeds'
4750
import {VideoEmbed} from '../util/post-embeds/VideoEmbed'
48-
import {PostMeta} from '../util/PostMeta'
49-
import {Text} from '../util/text/Text'
50-
import {PreviewableUserAvatar} from '../util/UserAvatar'
5151
import {AviFollowButton} from './AviFollowButton'
5252

5353
interface FeedItemProps {
@@ -571,7 +571,11 @@ function VideoDebug() {
571571

572572
return (
573573
<VideoEmbed
574-
source={`https://lumi.jazco.dev/watch/did:plc:q6gjnaw2blty4crticxkmujt/Qmc8w93UpTa2adJHg4ZhnDPrBs1EsbzrekzPcqF5SwusuZ/playlist.m3u8?ignore_me_just_testing_frontend_stuff=${id}`}
574+
embed={{
575+
playlist: `https://lumi.jazco.dev/watch/did:plc:q6gjnaw2blty4crticxkmujt/Qmc8w93UpTa2adJHg4ZhnDPrBs1EsbzrekzPcqF5SwusuZ/playlist.m3u8?ignore_me_just_testing_frontend_stuff=${id}`,
576+
cid: 'Qmc8w93UpTa2adJHg4ZhnDPrBs1EsbzrekzPcqF5SwusuZ',
577+
aspectRatio: {height: 9, width: 16},
578+
}}
575579
/>
576580
)
577581
}

src/view/com/util/post-embeds/VideoEmbed.tsx

Lines changed: 47 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -1,21 +1,25 @@
11
import React, {useCallback, useState} from 'react'
22
import {View} from 'react-native'
3+
import {Image} from 'expo-image'
4+
import {AppBskyEmbedVideo} from '@atproto/api'
35
import {msg, Trans} from '@lingui/macro'
46
import {useLingui} from '@lingui/react'
57

6-
import {VideoEmbedInnerNative} from 'view/com/util/post-embeds/VideoEmbedInner/VideoEmbedInnerNative'
8+
import {clamp} from '#/lib/numbers'
9+
import {useGate} from '#/lib/statsig/statsig'
10+
import {VideoEmbedInnerNative} from '#/view/com/util/post-embeds/VideoEmbedInner/VideoEmbedInnerNative'
711
import {atoms as a, useTheme} from '#/alf'
8-
import {Button, ButtonIcon} from '#/components/Button'
12+
import {Button} from '#/components/Button'
913
import {Play_Filled_Corner2_Rounded as PlayIcon} from '#/components/icons/Play'
1014
import {VisibilityView} from '../../../../../modules/expo-bluesky-swiss-army'
1115
import {ErrorBoundary} from '../ErrorBoundary'
1216
import {useActiveVideoNative} from './ActiveVideoNativeContext'
1317
import * as VideoFallback from './VideoEmbedInner/VideoFallback'
1418

15-
export function VideoEmbed({source}: {source: string}) {
19+
export function VideoEmbed({embed}: {embed: AppBskyEmbedVideo.View}) {
1620
const t = useTheme()
1721
const {activeSource, setActiveSource} = useActiveVideoNative()
18-
const isActive = source === activeSource
22+
const isActive = embed.playlist === activeSource
1923
const {_} = useLingui()
2024

2125
const [key, setKey] = useState(0)
@@ -25,39 +29,61 @@ export function VideoEmbed({source}: {source: string}) {
2529
),
2630
[key],
2731
)
32+
const gate = useGate()
33+
34+
if (!gate('videos')) {
35+
return null
36+
}
37+
38+
let aspectRatio = 16 / 9
39+
40+
if (embed.aspectRatio) {
41+
const {width, height} = embed.aspectRatio
42+
aspectRatio = width / height
43+
aspectRatio = clamp(aspectRatio, 1 / 1, 3 / 1)
44+
}
2845

2946
return (
3047
<View
3148
style={[
3249
a.w_full,
3350
a.rounded_sm,
34-
{aspectRatio: 16 / 9},
3551
a.overflow_hidden,
36-
t.atoms.bg_contrast_25,
52+
{aspectRatio},
53+
{backgroundColor: t.palette.black},
3754
a.my_xs,
3855
]}>
3956
<ErrorBoundary renderError={renderError} key={key}>
4057
<VisibilityView
4158
enabled={true}
42-
onChangeStatus={isActive => {
43-
if (isActive) {
44-
setActiveSource(source)
59+
onChangeStatus={isVisible => {
60+
if (isVisible) {
61+
setActiveSource(embed.playlist)
4562
}
4663
}}>
4764
{isActive ? (
48-
<VideoEmbedInnerNative />
65+
<VideoEmbedInnerNative embed={embed} />
4966
) : (
50-
<Button
51-
style={[a.flex_1, t.atoms.bg_contrast_25]}
52-
onPress={() => {
53-
setActiveSource(source)
54-
}}
55-
label={_(msg`Play video`)}
56-
variant="ghost"
57-
color="secondary"
58-
size="large">
59-
<ButtonIcon icon={PlayIcon} />
60-
</Button>
67+
<>
68+
<Image
69+
source={{uri: embed.thumbnail}}
70+
alt={embed.alt}
71+
style={a.flex_1}
72+
contentFit="contain"
73+
accessibilityIgnoresInvertColors
74+
/>
75+
<Button
76+
style={[a.absolute, a.inset_0]}
77+
onPress={() => {
78+
setActiveSource(embed.playlist)
79+
}}
80+
label={_(msg`Play video`)}
81+
variant="ghost"
82+
color="secondary"
83+
size="large">
84+
<PlayIcon width={48} fill={t.palette.white} />
85+
</Button>
86+
</>
6187
)}
6288
</VisibilityView>
6389
</ErrorBoundary>

src/view/com/util/post-embeds/VideoEmbed.web.tsx

Lines changed: 22 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,19 +1,23 @@
11
import React, {useCallback, useEffect, useRef, useState} from 'react'
22
import {View} from 'react-native'
3+
import {AppBskyEmbedVideo} from '@atproto/api'
34
import {Trans} from '@lingui/macro'
45

6+
import {clamp} from '#/lib/numbers'
7+
import {useGate} from '#/lib/statsig/statsig'
58
import {
69
HLSUnsupportedError,
710
VideoEmbedInnerWeb,
8-
} from 'view/com/util/post-embeds/VideoEmbedInner/VideoEmbedInnerWeb'
11+
} from '#/view/com/util/post-embeds/VideoEmbedInner/VideoEmbedInnerWeb'
912
import {atoms as a, useTheme} from '#/alf'
1013
import {ErrorBoundary} from '../ErrorBoundary'
1114
import {useActiveVideoWeb} from './ActiveVideoWebContext'
1215
import * as VideoFallback from './VideoEmbedInner/VideoFallback'
1316

14-
export function VideoEmbed({source}: {source: string}) {
17+
export function VideoEmbed({embed}: {embed: AppBskyEmbedVideo.View}) {
1518
const t = useTheme()
1619
const ref = useRef<HTMLDivElement>(null)
20+
const gate = useGate()
1721
const {active, setActive, sendPosition, currentActiveView} =
1822
useActiveVideoWeb()
1923
const [onScreen, setOnScreen] = useState(false)
@@ -43,12 +47,25 @@ export function VideoEmbed({source}: {source: string}) {
4347
[key],
4448
)
4549

50+
if (!gate('videos')) {
51+
return null
52+
}
53+
54+
let aspectRatio = 16 / 9
55+
56+
if (embed.aspectRatio) {
57+
const {width, height} = embed.aspectRatio
58+
// min: 3/1, max: square
59+
aspectRatio = clamp(width / height, 1 / 1, 3 / 1)
60+
}
61+
4662
return (
4763
<View
4864
style={[
4965
a.w_full,
50-
{aspectRatio: 16 / 9},
51-
t.atoms.bg_contrast_25,
66+
{aspectRatio},
67+
{backgroundColor: t.palette.black},
68+
a.relative,
5269
a.rounded_sm,
5370
a.my_xs,
5471
]}>
@@ -61,7 +78,7 @@ export function VideoEmbed({source}: {source: string}) {
6178
sendPosition={sendPosition}
6279
isAnyViewActive={currentActiveView !== null}>
6380
<VideoEmbedInnerWeb
64-
source={source}
81+
embed={embed}
6582
active={active}
6683
setActive={setActive}
6784
onScreen={onScreen}

src/view/com/util/post-embeds/VideoEmbedInner/VideoEmbedInnerNative.tsx

Lines changed: 25 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2,12 +2,14 @@ import React, {useCallback, useEffect, useRef, useState} from 'react'
22
import {Pressable, View} from 'react-native'
33
import Animated, {FadeInDown} from 'react-native-reanimated'
44
import {VideoPlayer, VideoView} from 'expo-video'
5+
import {AppBskyEmbedVideo} from '@atproto/api'
56
import {msg} from '@lingui/macro'
67
import {useLingui} from '@lingui/react'
78
import {useIsFocused} from '@react-navigation/native'
89

910
import {HITSLOP_30} from '#/lib/constants'
1011
import {useAppState} from '#/lib/hooks/useAppState'
12+
import {clamp} from '#/lib/numbers'
1113
import {logger} from '#/logger'
1214
import {useActiveVideoNative} from 'view/com/util/post-embeds/ActiveVideoNativeContext'
1315
import {atoms as a, useTheme} from '#/alf'
@@ -19,7 +21,12 @@ import {
1921
} from '../../../../../../modules/expo-bluesky-swiss-army'
2022
import {TimeIndicator} from './TimeIndicator'
2123

22-
export function VideoEmbedInnerNative() {
24+
export function VideoEmbedInnerNative({
25+
embed,
26+
}: {
27+
embed: AppBskyEmbedVideo.View
28+
}) {
29+
const {_} = useLingui()
2330
const {player} = useActiveVideoNative()
2431
const ref = useRef<VideoView>(null)
2532
const isScreenFocused = useIsFocused()
@@ -47,13 +54,23 @@ export function VideoEmbedInnerNative() {
4754
ref.current?.enterFullscreen()
4855
}, [])
4956

57+
let aspectRatio = 16 / 9
58+
59+
if (embed.aspectRatio) {
60+
const {width, height} = embed.aspectRatio
61+
aspectRatio = width / height
62+
aspectRatio = clamp(aspectRatio, 1 / 1, 3 / 1)
63+
}
64+
5065
return (
51-
<View style={[a.flex_1, a.relative]}>
66+
<View style={[a.flex_1, a.relative, {aspectRatio}]}>
5267
<VideoView
5368
ref={ref}
5469
player={player}
5570
style={[a.flex_1, a.rounded_sm]}
71+
contentFit="contain"
5672
nativeControls={true}
73+
accessibilityIgnoresInvertColors
5774
onEnterFullscreen={() => {
5875
PlatformInfo.setAudioCategory(AudioCategory.Playback)
5976
PlatformInfo.setAudioActive(true)
@@ -65,13 +82,17 @@ export function VideoEmbedInnerNative() {
6582
player.muted = true
6683
if (!player.playing) player.play()
6784
}}
85+
accessibilityLabel={
86+
embed.alt ? _(msg`Video: ${embed.alt}`) : _(msg`Video`)
87+
}
88+
accessibilityHint=""
6889
/>
69-
<Controls player={player} enterFullscreen={enterFullscreen} />
90+
<VideoControls player={player} enterFullscreen={enterFullscreen} />
7091
</View>
7192
)
7293
}
7394

74-
function Controls({
95+
function VideoControls({
7596
player,
7697
enterFullscreen,
7798
}: {

0 commit comments

Comments
 (0)