Page MenuHomePhabricator

[native] Fix LoggedOutModal buttons not appearing on load in release build
ClosedPublic

Authored by ashoat on May 29 2024, 7:52 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Dec 16, 7:06 PM
Unknown Object (File)
Mon, Dec 16, 7:06 PM
Unknown Object (File)
Mon, Dec 16, 7:06 PM
Unknown Object (File)
Mon, Dec 16, 6:50 PM
Unknown Object (File)
Mon, Nov 25, 7:02 AM
Unknown Object (File)
Mon, Nov 25, 6:11 AM
Unknown Object (File)
Nov 23 2024, 11:06 AM
Unknown Object (File)
Nov 23 2024, 10:50 AM
Subscribers

Details

Summary

This resolves ENG-8321.

I made a mistake in D12208. In Reanimated 3 it's okay to pass a Reanimated value directly to a style prop of an Animated.View, but not in Reanimated 2. I had been reading the Reanimated 3 docs initially, and after learning about this I updated most places but seem to have missed this spot.

In Reanimated 2, all uses of Reanimated values must be wrapped in useAnimatedStyle.

Guessing it only shows up in a release build due to some race conditions.

Test Plan

Confirmed that it resolves the linked Linear issue by compiling a release build to my iOS simulator

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable