[native-stack] Focused search bar causes new screens to have incorrect header
nandorojo opened this issue · 8 comments
Description
I'm using the iOS search bar to search artists. Works as expected.
When you click on an artist, it opens the Artist
screen.
The Artist
screen has headerShown: false
, since I have a custom header on that screen.
// this is the screen that search navigates to
<Screen
name="Artist"
component={ArtistScreen}
options={{
headerShown: false,
}}
getId={({ params }) => {
return params?.slug
}}
/>
However, if I open the artist screen while the search bar is focused, it adds a header to the Artist
screen, and creates a glitchy experience when I go back.
Screenshots
Works
When the search bar is not focused, and I click an artist, it works normally. It navigates to the Artist
screen, and there's no header.
Video: https://www.loom.com/share/174403203eb941f9a5731ba62253ab11
Doesn't work
However, when the search bar is focused, it incorrectly adds a header to the Artist
screen. Then, going back causes a weird jump.
Video: https://www.loom.com/share/8f8037145df649f880b891f52b099520
Steps To Reproduce
- Create a native stack screen with a search bar
- Navigate to a screen that has
headerShown: false
while the search bar is focused. - Try going back, too
Expected behavior
Having a searchBar
should not add a header when opening a screen that has headerShown: false
.
Actual behavior
A focused searchBar
causes screens opened to have a header. It also causes a layout glitch when you go back. I have contentInsetAdjustmentBehavior="automatic"
on my ScrollView
like I saw in the TestExample
file.
Snack or minimal code example
Expo Snack doesn't support 3.4.0. Let me know if you need me to make a full reproduction.
Package versions
- React: 16.8.3
- React Native: 0.63.4
- React Native Screens: 3.4.0
Setting searchBar.hideNavigationBar: false
makes the header bug go away. However, going back still causes the layout shift if the search bar was focused before opening the artist screen.
Also, I'd prefer to have hideNavigationBar: true
. But I'll leave it false for now as a workaround.
123925662-55fec500-d9b5-11eb-9cf9-f3a1f4670c64.mp4
i'm having same problem. Is it caused by createNativeStackNavigator() ?
@nandorojo Thanks for finding it. Unfortunately this is a native iOS issue stack overflow.
There is no easy fix on our side and we don't want to introduce very complex native logic to create workaround.
After inspecting this issue I came up with a JS workaround that should work for you.
https://gist.github.com/Ubax/086c18d07044ede728aa0ea8aa56b7d5
If it works for you, please let us know.
What you can also do is wrap content in <ScrollView contentInsetAdjustmentBehavior="automatic">
. Then set headerTranslucent: true
and optionally searchBar.hideWhenScrolling: false
. This should fix the layout shift.
I’ll try this, thanks!
If it works for you, please let us know.
Additionally #1153 is fixing the header bug, so you can test it too
@nandorojo I merged #1153 and therefore I am closing the issue.
If there are still some problems please let us know.