Skip to content

Commit

Permalink
Removed outdated assertion in RCTShadowView related to breaking chang…
Browse files Browse the repository at this point in the history
…e in Yoga

Summary: It's been more than two years; I think everyone already migrated and learned the new behavior.

Reviewed By: emilsjolander

Differential Revision: D6829885

fbshipit-source-id: a86d56fb7235a137e9ce6e360d7ae2224b047313
  • Loading branch information
shergin authored and facebook-github-bot committed Jan 29, 2018
1 parent ed5872e commit e3ff3cf
Showing 1 changed file with 0 additions and 10 deletions.
10 changes: 0 additions & 10 deletions React/Views/RCTShadowView.m
Original file line number Diff line number Diff line change
Expand Up @@ -203,16 +203,6 @@ - (void)applyLayoutNode:(YGNodeRef)node
return;
}

#if RCT_DEBUG
// This works around a breaking change in Yoga layout where setting flexBasis needs to be set explicitly, instead of relying on flex to propagate.
// We check for it by seeing if a width/height is provided along with a flexBasis of 0 and the width/height is laid out as 0.
if (YGNodeStyleGetFlexBasis(node).unit == YGUnitPoint && YGNodeStyleGetFlexBasis(node).value == 0 &&
((YGNodeStyleGetWidth(node).unit == YGUnitPoint && YGNodeStyleGetWidth(node).value > 0 && YGNodeLayoutGetWidth(node) == 0) ||
(YGNodeStyleGetHeight(node).unit == YGUnitPoint && YGNodeStyleGetHeight(node).value > 0 && YGNodeLayoutGetHeight(node) == 0))) {
RCTLogError(@"View was rendered with explicitly set width/height but with a 0 flexBasis. (This might be fixed by changing flex: to flexGrow:) View: %@", self);
}
#endif

CGRect frame = CGRectMake(YGNodeLayoutGetLeft(node), YGNodeLayoutGetTop(node), YGNodeLayoutGetWidth(node), YGNodeLayoutGetHeight(node));

// Even if `YGNodeLayoutGetDirection` can return `YGDirectionInherit` here, it actually means
Expand Down

0 comments on commit e3ff3cf

Please sign in to comment.