Conversation
- Introduced a ref to conditionally measure header height only when necessary. - Updated onHeaderLayout to prevent unnecessary state updates. - Adjusted onLayout prop for Animated.View to utilize the new measurement logic.
|
Hi @freely322 , |
|
@nitzanyiz @Inbal-Tish @ethanshar Take a look please |
|
Here's a patch for this fix. |
|
Wow, this is a painful bug that wasted a bunch of time... When will this be merged in? I manually patched. |
+1 the above fixed for me... only thing that I had to change was that I always had to add So, pretty much what @ajp8164 had but with +5... so looks like this |
|
Will also fix #2722 |
|
@Inbal-Tish @nitzanyiz - please have a look at this PR 🙏🏽 |
|
@ethanshar @Inbal-Tish @nitzanyiz can you take a look at this? |
|
also having this issue, @ethanshar @Inbal-Tish @nitzanyiz will anyone take a look at this? |
|
I'm migrating to expo v54 + react native v0.81 and this issue is back even after applying the patch #2653 (comment) Did anyone managed to solve this issue? If so, how? |
|
Not me, I had to roll back to v1.1289.0 |
|
I’ve found that using nativewind 4.2.0 together with tailwind 3.4.17 works fine in my project — not sure if that’ll work in your case though. |
Refactor header layout and height calculations
|
Was this included in the update? I'm using |
|
@joshbedo the changelog suggests that this bug did not get fixed. Do you mean that you still have the error or that you can't reproduce it on the latest version? |




The issue related to dynamic headerHeight was not set.
Related issues:
#2625
#2693
#2681
#2670
#2657
#2571
#2722