From 2a1edab6d4a8e8b4197b10868153a525f918509c Mon Sep 17 00:00:00 2001 From: dan Date: Fri, 13 Oct 2023 15:24:28 +0100 Subject: Don't re-render bars when showing/hiding them (#1691) * Don't re-render bars when showing/hiding them * Fix more cases * Use autorun instead of reaction to fix first render --- src/view/com/util/fab/FABInner.tsx | 17 ++++++++++------- 1 file changed, 10 insertions(+), 7 deletions(-) (limited to 'src/view/com/util/fab/FABInner.tsx') diff --git a/src/view/com/util/fab/FABInner.tsx b/src/view/com/util/fab/FABInner.tsx index 6c96eef2c..97eeba358 100644 --- a/src/view/com/util/fab/FABInner.tsx +++ b/src/view/com/util/fab/FABInner.tsx @@ -1,5 +1,6 @@ import React, {ComponentProps} from 'react' import {observer} from 'mobx-react-lite' +import {autorun} from 'mobx' import {Animated, StyleSheet, TouchableWithoutFeedback} from 'react-native' import LinearGradient from 'react-native-linear-gradient' import {gradients} from 'lib/styles' @@ -25,13 +26,15 @@ export const FABInner = observer(function FABInnerImpl({ const store = useStores() const interp = useAnimatedValue(0) React.useEffect(() => { - Animated.timing(interp, { - toValue: store.shell.minimalShellMode ? 0 : 1, - duration: 100, - useNativeDriver: true, - isInteraction: false, - }).start() - }, [interp, store.shell.minimalShellMode]) + return autorun(() => { + Animated.timing(interp, { + toValue: store.shell.minimalShellMode ? 0 : 1, + duration: 100, + useNativeDriver: true, + isInteraction: false, + }).start() + }) + }, [interp, store]) const transform = isTablet ? undefined : { -- cgit 1.4.1