Skip to content

Commit

Permalink
Delete CoreFeatures::excludeYogaFromRawProps (facebook#45627)
Browse files Browse the repository at this point in the history
Summary:
Pull Request resolved: facebook#45627

I'm deleting CoreFeatures::excludeYogaFromRawProps in favor of ReactNativeFeatureFlags::excludeYogaFromRawProps();

changelog: [internal] internal

Reviewed By: rubennorte

Differential Revision: D60124448
  • Loading branch information
mdvacca committed Jul 24, 2024
1 parent 7ffe488 commit e5423c4
Show file tree
Hide file tree
Showing 4 changed files with 1 addition and 7 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -394,8 +394,6 @@ void Binding::installFabricUIManager(

CoreFeatures::enablePropIteratorSetter =
ReactNativeFeatureFlags::enableCppPropsIteratorSetter();
CoreFeatures::excludeYogaFromRawProps =
ReactNativeFeatureFlags::excludeYogaFromRawProps();

auto toolbox = SchedulerToolbox{};
toolbox.contextContainer = contextContainer;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -785,7 +785,7 @@ Rect YogaLayoutableShadowNode::getContentBounds() const {
}

/*static*/ void YogaLayoutableShadowNode::filterRawProps(RawProps& rawProps) {
if (CoreFeatures::excludeYogaFromRawProps) {
if (ReactNativeFeatureFlags::excludeYogaFromRawProps()) {
// TODO: this shouldn't live in RawProps
rawProps.filterYogaStylePropsInDynamicConversion();
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,5 @@ namespace facebook::react {

bool CoreFeatures::enablePropIteratorSetter = false;
bool CoreFeatures::enableGranularScrollViewStateUpdatesIOS = false;
bool CoreFeatures::excludeYogaFromRawProps = false;

} // namespace facebook::react
3 changes: 0 additions & 3 deletions packages/react-native/ReactCommon/react/utils/CoreFeatures.h
Original file line number Diff line number Diff line change
Expand Up @@ -22,9 +22,6 @@ class CoreFeatures {
// When enabled, RCTScrollViewComponentView will trigger ShadowTree state
// updates for all changes in scroll position.
static bool enableGranularScrollViewStateUpdatesIOS;

// When enabled, rawProps in Props will not include Yoga specific props.
static bool excludeYogaFromRawProps;
};

} // namespace facebook::react

0 comments on commit e5423c4

Please sign in to comment.