Revert "completely rearchitect toasts"

This reverts commit 2e2fcaeeed.
This commit is contained in:
Hailey
2025-05-02 11:41:55 -07:00
parent da7503fee3
commit 267cc37e60
5 changed files with 117 additions and 121 deletions
@@ -9,7 +9,7 @@ class ExpoScrollForwarderView: ExpoView, UIGestureRecognizerDelegate {
} }
} }
private var rctScrollView: RCTScrollView? private var scrollView: UIScrollView?
private var rctRefreshCtrl: RCTRefreshControl? private var rctRefreshCtrl: RCTRefreshControl?
private var cancelGestureRecognizers: [UIGestureRecognizer]? private var cancelGestureRecognizers: [UIGestureRecognizer]?
private var animTimer: Timer? private var animTimer: Timer?
@@ -68,7 +68,7 @@ class ExpoScrollForwarderView: ExpoView, UIGestureRecognizerDelegate {
} }
@IBAction func callOnPan(_ sender: UIPanGestureRecognizer) { @IBAction func callOnPan(_ sender: UIPanGestureRecognizer) {
guard let rctsv = self.rctScrollView, let sv = rctsv.scrollView else { guard let sv = self.scrollView else {
return return
} }
@@ -113,7 +113,7 @@ class ExpoScrollForwarderView: ExpoView, UIGestureRecognizerDelegate {
} }
func startDecayAnimation(_ translation: CGFloat, _ velocity: CGFloat) { func startDecayAnimation(_ translation: CGFloat, _ velocity: CGFloat) {
guard let sv = self.rctScrollView?.scrollView else { guard let sv = self.scrollView else {
return return
} }
@@ -161,31 +161,48 @@ class ExpoScrollForwarderView: ExpoView, UIGestureRecognizerDelegate {
return offset return offset
} }
func tryFindScrollView() { private func findScrollView(in view: UIView, foundCount: Int) -> UIScrollView? {
guard let scrollViewTag = scrollViewTag else { var foundCount = foundCount
return if let sv = view as? UIScrollView { return sv }
for child in view.subviews {
if let found = findScrollView(in: child, foundCount: foundCount) {
if foundCount == 1 {
print("found sv: \(found)")
// return found
} else {
print("found sv: \(found)")
foundCount += 1
}
}
}
return nil
} }
func tryFindScrollView() {
// Before we switch to a different scrollview, we always want to remove the cancel gesture recognizer. // Before we switch to a different scrollview, we always want to remove the cancel gesture recognizer.
// Otherwise we might end up with duplicates when we switch back to that scrollview. // Otherwise we might end up with duplicates when we switch back to that scrollview.
self.removeCancelGestureRecognizers() self.removeCancelGestureRecognizers()
self.rctScrollView = self.appContext? guard let sv = self.findScrollView(in: self.superview!.superview!.superview!, foundCount: 0) else {
.findView(withTag: scrollViewTag, ofType: RCTScrollView.self) print("⚠️ ExpoScrollForwarder: couldnt find UIScrollView under tag \(tag)")
self.rctRefreshCtrl = self.rctScrollView?.scrollView.refreshControl as? RCTRefreshControl return
}
self.scrollView = sv
self.rctRefreshCtrl = sv.refreshControl as? RCTRefreshControl
self.addCancelGestureRecognizers() self.addCancelGestureRecognizers()
} }
func addCancelGestureRecognizers() { func addCancelGestureRecognizers() {
self.cancelGestureRecognizers?.forEach { r in self.cancelGestureRecognizers?.forEach { r in
self.rctScrollView?.scrollView?.addGestureRecognizer(r) self.scrollView?.addGestureRecognizer(r)
} }
} }
func removeCancelGestureRecognizers() { func removeCancelGestureRecognizers() {
self.cancelGestureRecognizers?.forEach { r in self.cancelGestureRecognizers?.forEach { r in
self.rctScrollView?.scrollView?.removeGestureRecognizer(r) self.scrollView?.removeGestureRecognizer(r)
} }
} }
@@ -202,7 +219,7 @@ class ExpoScrollForwarderView: ExpoView, UIGestureRecognizerDelegate {
} }
func scrollToOffset(_ offset: Int, animated: Bool = true) { func scrollToOffset(_ offset: Int, animated: Bool = true) {
self.rctScrollView?.scroll(toOffset: CGPoint(x: 0, y: offset), animated: animated) self.scrollView?.scrollRectToVisible(CGRect(x: 0, y: offset, width: 0, height: 0), animated: animated)
} }
func stopTimer() { func stopTimer() {
-2
View File
@@ -60,7 +60,6 @@ import {Provider as HiddenRepliesProvider} from '#/state/threadgate-hidden-repli
import {Provider as TrendingConfigProvider} from '#/state/trending-config' import {Provider as TrendingConfigProvider} from '#/state/trending-config'
import {TestCtrls} from '#/view/com/testing/TestCtrls' import {TestCtrls} from '#/view/com/testing/TestCtrls'
import {Provider as VideoVolumeProvider} from '#/view/com/util/post-embeds/VideoVolumeContext' import {Provider as VideoVolumeProvider} from '#/view/com/util/post-embeds/VideoVolumeContext'
import {ToastContainer} from '#/view/com/util/Toast'
import * as Toast from '#/view/com/util/Toast' import * as Toast from '#/view/com/util/Toast'
import {Shell} from '#/view/shell' import {Shell} from '#/view/shell'
import {ThemeProvider as Alf} from '#/alf' import {ThemeProvider as Alf} from '#/alf'
@@ -157,7 +156,6 @@ function InnerApp() {
<TestCtrls /> <TestCtrls />
<Shell /> <Shell />
<NuxDialogs /> <NuxDialogs />
<ToastContainer />
</IntentDialogProvider> </IntentDialogProvider>
</GestureHandlerRootView> </GestureHandlerRootView>
</TrendingConfigProvider> </TrendingConfigProvider>
+1 -1
View File
@@ -51,7 +51,7 @@ import {Provider as TrendingConfigProvider} from '#/state/trending-config'
import {Provider as ActiveVideoProvider} from '#/view/com/util/post-embeds/ActiveVideoWebContext' import {Provider as ActiveVideoProvider} from '#/view/com/util/post-embeds/ActiveVideoWebContext'
import {Provider as VideoVolumeProvider} from '#/view/com/util/post-embeds/VideoVolumeContext' import {Provider as VideoVolumeProvider} from '#/view/com/util/post-embeds/VideoVolumeContext'
import * as Toast from '#/view/com/util/Toast' import * as Toast from '#/view/com/util/Toast'
import {ToastContainer} from '#/view/com/util/Toast' import {ToastContainer} from '#/view/com/util/Toast.web'
import {Shell} from '#/view/shell/index' import {Shell} from '#/view/shell/index'
import {ThemeProvider as Alf} from '#/alf' import {ThemeProvider as Alf} from '#/alf'
import {useColorModeTheme} from '#/alf/util/useColorModeTheme' import {useColorModeTheme} from '#/alf/util/useColorModeTheme'
+42 -63
View File
@@ -1,7 +1,13 @@
import {useEffect, useMemo, useRef, useState} from 'react' import {useEffect, useMemo, useRef, useState} from 'react'
import {AccessibilityInfo, View} from 'react-native' import {AccessibilityInfo, View} from 'react-native'
import {Gesture, GestureDetector} from 'react-native-gesture-handler' import {
Gesture,
GestureDetector,
GestureHandlerRootView,
} from 'react-native-gesture-handler'
import Animated, { import Animated, {
FadeInUp,
FadeOutUp,
runOnJS, runOnJS,
useAnimatedReaction, useAnimatedReaction,
useAnimatedStyle, useAnimatedStyle,
@@ -9,13 +15,12 @@ import Animated, {
withDecay, withDecay,
withSpring, withSpring,
} from 'react-native-reanimated' } from 'react-native-reanimated'
import RootSiblings from 'react-native-root-siblings'
import {useSafeAreaInsets} from 'react-native-safe-area-context' import {useSafeAreaInsets} from 'react-native-safe-area-context'
import { import {
FontAwesomeIcon, FontAwesomeIcon,
type Props as FontAwesomeProps, type Props as FontAwesomeProps,
} from '@fortawesome/react-native-fontawesome' } from '@fortawesome/react-native-fontawesome'
import EventEmitter from 'eventemitter3'
import {nanoid} from 'nanoid/non-secure'
import {useNonReactiveCallback} from '#/lib/hooks/useNonReactiveCallback' import {useNonReactiveCallback} from '#/lib/hooks/useNonReactiveCallback'
import {atoms as a, useTheme} from '#/alf' import {atoms as a, useTheme} from '#/alf'
@@ -23,37 +28,6 @@ import {Text} from '#/components/Typography'
const TIMEOUT = 2e3 const TIMEOUT = 2e3
type ToastProps = {
message: string
icon: FontAwesomeProps['icon']
}
const SHOW_TOAST = 'show-toast'
const ToastEventEmitter = new EventEmitter<typeof SHOW_TOAST>()
export function ToastContainer() {
const [toasts, setToasts] = useState<(ToastProps & {key: string})[]>([])
useEffect(() => {
const listener = ({message, icon}: ToastProps) => {
setToasts(prev => [...prev, {message, icon, key: nanoid()}])
}
ToastEventEmitter.on(SHOW_TOAST, listener)
return () => {
ToastEventEmitter.off(SHOW_TOAST, listener)
}
}, [])
return toasts.map(toast => (
<Toast
{...toast}
key={toast.key}
destroy={() => setToasts(prev => prev.filter(t => t !== toast))}
/>
))
}
export function show( export function show(
message: string, message: string,
icon: FontAwesomeProps['icon'] = 'check', icon: FontAwesomeProps['icon'] = 'check',
@@ -62,7 +36,9 @@ export function show(
return return
} }
AccessibilityInfo.announceForAccessibility(message) AccessibilityInfo.announceForAccessibility(message)
ToastEventEmitter.emit(SHOW_TOAST, {message, icon}) const item = new RootSiblings(
<Toast message={message} icon={icon} destroy={() => item.destroy()} />,
)
} }
function Toast({ function Toast({
@@ -77,23 +53,19 @@ function Toast({
const t = useTheme() const t = useTheme()
const {top} = useSafeAreaInsets() const {top} = useSafeAreaInsets()
const isPanning = useSharedValue(false) const isPanning = useSharedValue(false)
const animationTranslateY = useSharedValue(-200)
const dismissSwipeTranslateY = useSharedValue(0) const dismissSwipeTranslateY = useSharedValue(0)
const [cardHeight, setCardHeight] = useState(0) const [cardHeight, setCardHeight] = useState(0)
// for the exit animation to work on iOS the animated component
// must not be the root component
// so we need to wrap it in a view and unmount the toast ahead of time
const [alive, setAlive] = useState(true)
const hideAndDestroyImmediately = () => { const hideAndDestroyImmediately = () => {
'worklet' setAlive(false)
animationTranslateY.set( setTimeout(() => {
withSpring( destroy()
-200, }, 1e3)
{
damping: 100,
stiffness: 800,
restDisplacementThreshold: 0.01,
},
() => runOnJS(destroy)(),
),
)
} }
const destroyTimeoutRef = useRef<ReturnType<typeof setTimeout>>() const destroyTimeoutRef = useRef<ReturnType<typeof setTimeout>>()
@@ -106,15 +78,8 @@ function Toast({
}) })
useEffect(() => { useEffect(() => {
animationTranslateY.set(
withSpring(0, {
damping: 100,
stiffness: 800,
restDisplacementThreshold: 0.01,
}),
)
hideAndDestroyAfterTimeout() hideAndDestroyAfterTimeout()
}, [hideAndDestroyAfterTimeout, animationTranslateY]) }, [hideAndDestroyAfterTimeout])
const panGesture = useMemo(() => { const panGesture = useMemo(() => {
return Gesture.Pan() return Gesture.Pan()
@@ -123,15 +88,18 @@ function Toast({
.maxPointers(1) .maxPointers(1)
.onStart(() => { .onStart(() => {
'worklet' 'worklet'
if (!alive) return
isPanning.set(true) isPanning.set(true)
runOnJS(pauseDestroy)() runOnJS(pauseDestroy)()
}) })
.onUpdate(e => { .onUpdate(e => {
'worklet' 'worklet'
if (!alive) return
dismissSwipeTranslateY.value = e.translationY dismissSwipeTranslateY.value = e.translationY
}) })
.onEnd(e => { .onEnd(e => {
'worklet' 'worklet'
if (!alive) return
runOnJS(hideAndDestroyAfterTimeout)() runOnJS(hideAndDestroyAfterTimeout)()
isPanning.set(false) isPanning.set(false)
if (e.velocityY < -100) { if (e.velocityY < -100) {
@@ -147,15 +115,15 @@ function Toast({
}) })
} else { } else {
dismissSwipeTranslateY.value = withSpring(0, { dismissSwipeTranslateY.value = withSpring(0, {
damping: 75, stiffness: 500,
stiffness: 1000, damping: 50,
restDisplacementThreshold: 0.01,
}) })
} }
}) })
}, [ }, [
dismissSwipeTranslateY, dismissSwipeTranslateY,
isPanning, isPanning,
alive,
hideAndDestroyAfterTimeout, hideAndDestroyAfterTimeout,
pauseDestroy, pauseDestroy,
]) ])
@@ -175,7 +143,7 @@ function Toast({
) )
const animatedStyle = useAnimatedStyle(() => { const animatedStyle = useAnimatedStyle(() => {
const translation = dismissSwipeTranslateY.get() + animationTranslateY.get() const translation = dismissSwipeTranslateY.get()
return { return {
transform: [ transform: [
{ {
@@ -186,6 +154,14 @@ function Toast({
}) })
return ( return (
<GestureHandlerRootView
style={[a.absolute, {top: topOffset, left: 16, right: 16}]}
pointerEvents="box-none">
{alive && (
<Animated.View
entering={FadeInUp}
exiting={FadeOutUp}
style={[a.flex_1]}>
<Animated.View <Animated.View
onLayout={evt => setCardHeight(evt.nativeEvent.layout.height)} onLayout={evt => setCardHeight(evt.nativeEvent.layout.height)}
accessibilityRole="alert" accessibilityRole="alert"
@@ -194,8 +170,6 @@ function Toast({
accessibilityHint="" accessibilityHint=""
onAccessibilityEscape={hideAndDestroyImmediately} onAccessibilityEscape={hideAndDestroyImmediately}
style={[ style={[
a.absolute,
{top: topOffset, left: 16, right: 16},
a.flex_1, a.flex_1,
t.name === 'dark' ? t.atoms.bg_contrast_25 : t.atoms.bg, t.name === 'dark' ? t.atoms.bg_contrast_25 : t.atoms.bg,
a.shadow_lg, a.shadow_lg,
@@ -215,7 +189,9 @@ function Toast({
a.justify_center, a.justify_center,
{ {
backgroundColor: backgroundColor:
t.name === 'dark' ? t.palette.black : t.palette.primary_50, t.name === 'dark'
? t.palette.black
: t.palette.primary_50,
}, },
]}> ]}>
<FontAwesomeIcon <FontAwesomeIcon
@@ -232,5 +208,8 @@ function Toast({
</View> </View>
</GestureDetector> </GestureDetector>
</Animated.View> </Animated.View>
</Animated.View>
)}
</GestureHandlerRootView>
) )
} }
+3 -1
View File
@@ -9,6 +9,7 @@ import {
type FontAwesomeIconStyle, type FontAwesomeIconStyle,
type Props as FontAwesomeProps, type Props as FontAwesomeProps,
} from '@fortawesome/react-native-fontawesome' } from '@fortawesome/react-native-fontawesome'
import type React from 'react';
const DURATION = 3500 const DURATION = 3500
@@ -25,7 +26,8 @@ let toastTimeout: NodeJS.Timeout | undefined
// components // components
// = // =
export function ToastContainer() { type ToastContainerProps = {}
export const ToastContainer: React.FC<ToastContainerProps> = ({}) => {
const [activeToast, setActiveToast] = useState<ActiveToast | undefined>() const [activeToast, setActiveToast] = useState<ActiveToast | undefined>()
useEffect(() => { useEffect(() => {
globalSetActiveToast = (t: ActiveToast | undefined) => { globalSetActiveToast = (t: ActiveToast | undefined) => {