Skip to content

Commit b972158

Browse files
committed
fix: race condition
1 parent 79bb158 commit b972158

File tree

2 files changed

+4
-3
lines changed

2 files changed

+4
-3
lines changed

src/app/Components/GlobalSearchInput/GlobalSearchInput.tsx

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@ import { Flex, RoundSearchInput, Touchable } from "@artsy/palette-mobile"
33
import { GlobalSearchInputOverlay } from "app/Components/GlobalSearchInput/GlobalSearchInputOverlay"
44
import { useDismissSearchOverlayOnTabBarPress } from "app/Components/GlobalSearchInput/utils/useDismissSearchOverlayOnTabBarPress"
55
import { ICON_HIT_SLOP } from "app/Components/constants"
6+
import { useDebouncedValue } from "app/utils/hooks/useDebouncedValue"
67
import { forwardRef, Fragment, useImperativeHandle, useState } from "react"
78
import { useTracking } from "react-tracking"
89

@@ -16,13 +17,14 @@ export type GlobalSearchInput = {
1617
export const GlobalSearchInput = forwardRef<GlobalSearchInput, GlobalSearchInputProps>(
1718
({ ownerType }, ref) => {
1819
const [isVisible, setIsVisible] = useState(false)
20+
const debouncedIsVisible = useDebouncedValue({ value: isVisible })
1921
const tracking = useTracking()
2022

2123
useDismissSearchOverlayOnTabBarPress({ isVisible, ownerType, setIsVisible })
2224

2325
useImperativeHandle(ref, () => ({
2426
focus: () => {
25-
if (!isVisible) {
27+
if (!debouncedIsVisible.debouncedValue) {
2628
setIsVisible(true)
2729
}
2830
},

src/app/Scenes/Search/Search.tsx

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -64,12 +64,11 @@ export const Search: React.FC = () => {
6464

6565
useRefetchWhenQueryChanged({ query: searchQuery, refetch })
6666

67-
// Focus input and open keyboard on bottom nav Search tab double-tab
6867
const scrollableRef = useBottomTabsScrollToTop(() => {
6968
if (enableNewSearchModal) {
69+
// Focus input and open keyboard on bottom nav Search tab double-tab
7070
searchInputRef.current?.focus()
7171
} else {
72-
// Focus input and open keyboard on bottom nav Search tab double-tab
7372
searchProviderValues.inputRef.current?.focus()
7473
}
7574
})

0 commit comments

Comments
 (0)