Fixes regression of prop parsing for elevation (facebook#38959)

Summary: Pull Request resolved: facebook#38959 When refactoring ViewProps in D47492635, `elevation` prop parsing was dropped. This restores it. ## Changelog: [General] [Fixed] - Fabric regression for elevation prop in Android Reviewed By: mdvacca Differential Revision: D48269510 fbshipit-source-id: 26dbc7dbdcaadb3dad537226656abd9aee4d05a9
rozele · Aug 11, 2023 · ec42ae0 · ec42ae0
1 parent d655d44
commit ec42ae0
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 2 deletions.
diff --git a/packages/react-native/ReactCommon/react/renderer/components/view/BaseViewProps.h b/packages/react-native/ReactCommon/react/renderer/components/view/BaseViewProps.h
@@ -72,8 +72,6 @@ class BaseViewProps : public YogaStylableProps, public AccessibilityProps {
 
   LayoutConformance experimental_layoutConformance{};
 
-  Float elevation{}; /* Android-only */
-
 #pragma mark - Convenience Methods
 
   BorderMetrics resolveBorderMetrics(LayoutMetrics const &layoutMetrics) const;

diff --git a/...components/view/platform/android/react/renderer/components/view/HostPlatformViewProps.cpp b/...components/view/platform/android/react/renderer/components/view/HostPlatformViewProps.cpp
@@ -23,6 +23,14 @@ HostPlatformViewProps::HostPlatformViewProps(
     RawProps const &rawProps,
     bool shouldSetRawProps)
     : BaseViewProps(context, sourceProps, rawProps, shouldSetRawProps),
+      elevation(
+          CoreFeatures::enablePropIteratorSetter ? sourceProps.elevation
+                                                 : convertRawProp(
+                                                       context,
+                                                       rawProps,
+                                                       "elevation",
+                                                       sourceProps.elevation,
+                                                       {})),
       nativeBackground(
           CoreFeatures::enablePropIteratorSetter
               ? sourceProps.nativeBackground

diff --git a/...r/components/view/platform/android/react/renderer/components/view/HostPlatformViewProps.h b/...r/components/view/platform/android/react/renderer/components/view/HostPlatformViewProps.h
@@ -40,6 +40,8 @@ class HostPlatformViewProps : public BaseViewProps {
 
 #pragma mark - Props
 
+  Float elevation{};
+
   std::optional<NativeDrawable> nativeBackground{};
   std::optional<NativeDrawable> nativeForeground{};