diff --git a/packages/react-native/ReactCommon/yoga/yoga/algorithm/CalculateLayout.cpp b/packages/react-native/ReactCommon/yoga/yoga/algorithm/CalculateLayout.cpp index dc9f785eac..feed51708c 100644 --- a/packages/react-native/ReactCommon/yoga/yoga/algorithm/CalculateLayout.cpp +++ b/packages/react-native/ReactCommon/yoga/yoga/algorithm/CalculateLayout.cpp @@ -88,8 +88,8 @@ static void setChildTrailingPosition( const float size = child->getLayout().measuredDimension(dimension(axis)); child->setLayoutPosition( node->getLayout().measuredDimension(dimension(axis)) - size - - child->getLayout().position[leadingEdge(axis)], - trailingEdge(axis)); + child->getLayout().position[flexStartEdge(axis)], + flexEndEdge(axis)); } static void constrainMaxSizeForMode( @@ -456,7 +456,7 @@ static void layoutAbsoluteChild( mainAxis, direction, isMainAxisRow ? width : height) - child->getTrailingPosition( mainAxis, isMainAxisRow ? width : height), - leadingEdge(mainAxis)); + flexStartEdge(mainAxis)); } else if ( !child->isLeadingPositionDefined(mainAxis) && node->getStyle().justifyContent() == Justify::Center) { @@ -464,14 +464,14 @@ static void layoutAbsoluteChild( (node->getLayout().measuredDimension(dimension(mainAxis)) - child->getLayout().measuredDimension(dimension(mainAxis))) / 2.0f, - leadingEdge(mainAxis)); + flexStartEdge(mainAxis)); } else if ( !child->isLeadingPositionDefined(mainAxis) && node->getStyle().justifyContent() == Justify::FlexEnd) { child->setLayoutPosition( (node->getLayout().measuredDimension(dimension(mainAxis)) - child->getLayout().measuredDimension(dimension(mainAxis))), - leadingEdge(mainAxis)); + flexStartEdge(mainAxis)); } else if ( node->getConfig()->isExperimentalFeatureEnabled( ExperimentalFeature::AbsolutePercentageAgainstPaddingEdge) && @@ -485,7 +485,7 @@ static void layoutAbsoluteChild( mainAxis, direction, node->getLayout().measuredDimension(dimension(mainAxis))), - leadingEdge(mainAxis)); + flexStartEdge(mainAxis)); } if (child->isTrailingPosDefined(crossAxis) && @@ -498,7 +498,7 @@ static void layoutAbsoluteChild( crossAxis, direction, isMainAxisRow ? height : width) - child->getTrailingPosition( crossAxis, isMainAxisRow ? height : width), - leadingEdge(crossAxis)); + flexStartEdge(crossAxis)); } else if ( !child->isLeadingPositionDefined(crossAxis) && @@ -507,7 +507,7 @@ static void layoutAbsoluteChild( (node->getLayout().measuredDimension(dimension(crossAxis)) - child->getLayout().measuredDimension(dimension(crossAxis))) / 2.0f, - leadingEdge(crossAxis)); + flexStartEdge(crossAxis)); } else if ( !child->isLeadingPositionDefined(crossAxis) && ((resolveChildAlignment(node, child) == Align::FlexEnd) ^ @@ -515,7 +515,7 @@ static void layoutAbsoluteChild( child->setLayoutPosition( (node->getLayout().measuredDimension(dimension(crossAxis)) - child->getLayout().measuredDimension(dimension(crossAxis))), - leadingEdge(crossAxis)); + flexStartEdge(crossAxis)); } else if ( node->getConfig()->isExperimentalFeatureEnabled( ExperimentalFeature::AbsolutePercentageAgainstPaddingEdge) && @@ -529,7 +529,7 @@ static void layoutAbsoluteChild( crossAxis, direction, node->getLayout().measuredDimension(dimension(crossAxis))), - leadingEdge(crossAxis)); + flexStartEdge(crossAxis)); } } @@ -1308,7 +1308,7 @@ static void justifyMainAxis( node->getLeadingBorder(mainAxis) + child->getLeadingMargin( mainAxis, direction, availableInnerWidth), - leadingEdge(mainAxis)); + flexStartEdge(mainAxis)); } } else { // Now that we placed the element, we need to update the variables. @@ -1322,9 +1322,9 @@ static void justifyMainAxis( if (performLayout) { child->setLayoutPosition( - childLayout.position[leadingEdge(mainAxis)] + + childLayout.position[flexStartEdge(mainAxis)] + flexLine.layout.mainDim, - leadingEdge(mainAxis)); + flexStartEdge(mainAxis)); } if (child != flexLine.itemsInFlow.back()) { @@ -1378,9 +1378,9 @@ static void justifyMainAxis( } } else if (performLayout) { child->setLayoutPosition( - childLayout.position[leadingEdge(mainAxis)] + + childLayout.position[flexStartEdge(mainAxis)] + node->getLeadingBorder(mainAxis) + leadingMainDim, - leadingEdge(mainAxis)); + flexStartEdge(mainAxis)); } } } @@ -1856,18 +1856,18 @@ static void calculateLayoutImpl( node->getLeadingBorder(crossAxis) + child->getLeadingMargin( crossAxis, direction, availableInnerWidth), - leadingEdge(crossAxis)); + flexStartEdge(crossAxis)); } // If leading position is not defined or calculations result in Nan, // default to border + margin if (!isChildLeadingPosDefined || yoga::isUndefined( - child->getLayout().position[leadingEdge(crossAxis)])) { + child->getLayout().position[flexStartEdge(crossAxis)])) { child->setLayoutPosition( node->getLeadingBorder(crossAxis) + child->getLeadingMargin( crossAxis, direction, availableInnerWidth), - leadingEdge(crossAxis)); + flexStartEdge(crossAxis)); } } else { float leadingCrossDim = leadingPaddingAndBorderCross; @@ -1975,9 +1975,9 @@ static void calculateLayoutImpl( } // And we apply the position child->setLayoutPosition( - child->getLayout().position[leadingEdge(crossAxis)] + + child->getLayout().position[flexStartEdge(crossAxis)] + totalLineCrossDim + leadingCrossDim, - leadingEdge(crossAxis)); + flexStartEdge(crossAxis)); } } } @@ -2092,7 +2092,7 @@ static void calculateLayoutImpl( currentLead + child->getLeadingMargin( crossAxis, direction, availableInnerWidth), - leadingEdge(crossAxis)); + flexStartEdge(crossAxis)); break; } case Align::FlexEnd: { @@ -2102,7 +2102,7 @@ static void calculateLayoutImpl( crossAxis, direction, availableInnerWidth) - child->getLayout().measuredDimension( dimension(crossAxis)), - leadingEdge(crossAxis)); + flexStartEdge(crossAxis)); break; } case Align::Center: { @@ -2111,7 +2111,7 @@ static void calculateLayoutImpl( child->setLayoutPosition( currentLead + (lineHeight - childHeight) / 2, - leadingEdge(crossAxis)); + flexStartEdge(crossAxis)); break; } case Align::Stretch: { @@ -2119,7 +2119,7 @@ static void calculateLayoutImpl( currentLead + child->getLeadingMargin( crossAxis, direction, availableInnerWidth), - leadingEdge(crossAxis)); + flexStartEdge(crossAxis)); // Remeasure child with the line height as it as been only // measured with the owners height yet. @@ -2275,9 +2275,9 @@ static void calculateLayoutImpl( if (child->getStyle().positionType() != PositionType::Absolute) { child->setLayoutPosition( node->getLayout().measuredDimension(dimension(crossAxis)) - - child->getLayout().position[leadingEdge(crossAxis)] - + child->getLayout().position[flexStartEdge(crossAxis)] - child->getLayout().measuredDimension(dimension(crossAxis)), - leadingEdge(crossAxis)); + flexStartEdge(crossAxis)); } } } diff --git a/packages/react-native/ReactCommon/yoga/yoga/algorithm/FlexDirection.h b/packages/react-native/ReactCommon/yoga/yoga/algorithm/FlexDirection.h index 7dd2b4cce7..c6cd95f20e 100644 --- a/packages/react-native/ReactCommon/yoga/yoga/algorithm/FlexDirection.h +++ b/packages/react-native/ReactCommon/yoga/yoga/algorithm/FlexDirection.h @@ -47,7 +47,7 @@ inline FlexDirection resolveCrossDirection( : FlexDirection::Column; } -inline YGEdge leadingEdge(const FlexDirection flexDirection) { +inline YGEdge flexStartEdge(const FlexDirection flexDirection) { switch (flexDirection) { case FlexDirection::Column: return YGEdgeTop; @@ -62,7 +62,7 @@ inline YGEdge leadingEdge(const FlexDirection flexDirection) { fatalWithMessage("Invalid FlexDirection"); } -inline YGEdge trailingEdge(const FlexDirection flexDirection) { +inline YGEdge flexEndEdge(const FlexDirection flexDirection) { switch (flexDirection) { case FlexDirection::Column: return YGEdgeBottom; @@ -77,7 +77,7 @@ inline YGEdge trailingEdge(const FlexDirection flexDirection) { fatalWithMessage("Invalid FlexDirection"); } -inline YGEdge leadingLayoutEdge( +inline YGEdge inlineStartEdge( const FlexDirection flexDirection, const Direction direction) { if (isRow(flexDirection)) { @@ -87,7 +87,7 @@ inline YGEdge leadingLayoutEdge( return YGEdgeTop; } -inline YGEdge trailingLayoutEdge( +inline YGEdge inlineEndEdge( const FlexDirection flexDirection, const Direction direction) { if (isRow(flexDirection)) { diff --git a/packages/react-native/ReactCommon/yoga/yoga/node/Node.cpp b/packages/react-native/ReactCommon/yoga/yoga/node/Node.cpp index c7cee82930..3ea8f9bd3d 100644 --- a/packages/react-native/ReactCommon/yoga/yoga/node/Node.cpp +++ b/packages/react-native/ReactCommon/yoga/yoga/node/Node.cpp @@ -87,31 +87,31 @@ YGEdge Node::getLeadingLayoutEdgeUsingErrata( FlexDirection flexDirection, Direction direction) const { return hasErrata(Errata::StartingEndingEdgeFromFlexDirection) - ? leadingEdge(flexDirection) - : leadingLayoutEdge(flexDirection, direction); + ? flexStartEdge(flexDirection) + : inlineStartEdge(flexDirection, direction); } YGEdge Node::getTrailingLayoutEdgeUsingErrata( FlexDirection flexDirection, Direction direction) const { return hasErrata(Errata::StartingEndingEdgeFromFlexDirection) - ? trailingEdge(flexDirection) - : trailingLayoutEdge(flexDirection, direction); + ? flexEndEdge(flexDirection) + : inlineEndEdge(flexDirection, direction); } bool Node::isLeadingPositionDefined(FlexDirection axis) const { auto leadingPosition = isRow(axis) ? computeEdgeValueForRow( - style_.position(), YGEdgeStart, leadingEdge(axis)) - : computeEdgeValueForColumn(style_.position(), leadingEdge(axis)); + style_.position(), YGEdgeStart, flexStartEdge(axis)) + : computeEdgeValueForColumn(style_.position(), flexStartEdge(axis)); return !leadingPosition.isUndefined(); } bool Node::isTrailingPosDefined(FlexDirection axis) const { auto trailingPosition = isRow(axis) - ? computeEdgeValueForRow(style_.position(), YGEdgeEnd, trailingEdge(axis)) - : computeEdgeValueForColumn(style_.position(), trailingEdge(axis)); + ? computeEdgeValueForRow(style_.position(), YGEdgeEnd, flexEndEdge(axis)) + : computeEdgeValueForColumn(style_.position(), flexEndEdge(axis)); return !trailingPosition.isUndefined(); } @@ -119,16 +119,16 @@ bool Node::isTrailingPosDefined(FlexDirection axis) const { float Node::getLeadingPosition(FlexDirection axis, float axisSize) const { auto leadingPosition = isRow(axis) ? computeEdgeValueForRow( - style_.position(), YGEdgeStart, leadingEdge(axis)) - : computeEdgeValueForColumn(style_.position(), leadingEdge(axis)); + style_.position(), YGEdgeStart, flexStartEdge(axis)) + : computeEdgeValueForColumn(style_.position(), flexStartEdge(axis)); return resolveValue(leadingPosition, axisSize).unwrapOrDefault(0.0f); } float Node::getTrailingPosition(FlexDirection axis, float axisSize) const { auto trailingPosition = isRow(axis) - ? computeEdgeValueForRow(style_.position(), YGEdgeEnd, trailingEdge(axis)) - : computeEdgeValueForColumn(style_.position(), trailingEdge(axis)); + ? computeEdgeValueForRow(style_.position(), YGEdgeEnd, flexEndEdge(axis)) + : computeEdgeValueForColumn(style_.position(), flexEndEdge(axis)); return resolveValue(trailingPosition, axisSize).unwrapOrDefault(0.0f); } @@ -159,32 +159,34 @@ float Node::getTrailingMargin( float Node::getLeadingBorder(FlexDirection axis) const { YGValue leadingBorder = isRow(axis) - ? computeEdgeValueForRow(style_.border(), YGEdgeStart, leadingEdge(axis)) - : computeEdgeValueForColumn(style_.border(), leadingEdge(axis)); + ? computeEdgeValueForRow( + style_.border(), YGEdgeStart, flexStartEdge(axis)) + : computeEdgeValueForColumn(style_.border(), flexStartEdge(axis)); return maxOrDefined(leadingBorder.value, 0.0f); } float Node::getTrailingBorder(FlexDirection axis) const { YGValue trailingBorder = isRow(axis) - ? computeEdgeValueForRow(style_.border(), YGEdgeEnd, trailingEdge(axis)) - : computeEdgeValueForColumn(style_.border(), trailingEdge(axis)); + ? computeEdgeValueForRow(style_.border(), YGEdgeEnd, flexEndEdge(axis)) + : computeEdgeValueForColumn(style_.border(), flexEndEdge(axis)); return maxOrDefined(trailingBorder.value, 0.0f); } float Node::getLeadingPadding(FlexDirection axis, float widthSize) const { auto leadingPadding = isRow(axis) - ? computeEdgeValueForRow(style_.padding(), YGEdgeStart, leadingEdge(axis)) - : computeEdgeValueForColumn(style_.padding(), leadingEdge(axis)); + ? computeEdgeValueForRow( + style_.padding(), YGEdgeStart, flexStartEdge(axis)) + : computeEdgeValueForColumn(style_.padding(), flexStartEdge(axis)); return maxOrDefined(resolveValue(leadingPadding, widthSize).unwrap(), 0.0f); } float Node::getTrailingPadding(FlexDirection axis, float widthSize) const { auto trailingPadding = isRow(axis) - ? computeEdgeValueForRow(style_.padding(), YGEdgeEnd, trailingEdge(axis)) - : computeEdgeValueForColumn(style_.padding(), trailingEdge(axis)); + ? computeEdgeValueForRow(style_.padding(), YGEdgeEnd, flexEndEdge(axis)) + : computeEdgeValueForColumn(style_.padding(), flexEndEdge(axis)); return maxOrDefined(resolveValue(trailingPadding, widthSize).unwrap(), 0.0f); } @@ -418,7 +420,7 @@ YGValue Node::marginLeadingValue(FlexDirection axis) const { if (isRow(axis) && !style_.margin()[YGEdgeStart].isUndefined()) { return style_.margin()[YGEdgeStart]; } else { - return style_.margin()[leadingEdge(axis)]; + return style_.margin()[flexStartEdge(axis)]; } } @@ -426,7 +428,7 @@ YGValue Node::marginTrailingValue(FlexDirection axis) const { if (isRow(axis) && !style_.margin()[YGEdgeEnd].isUndefined()) { return style_.margin()[YGEdgeEnd]; } else { - return style_.margin()[trailingEdge(axis)]; + return style_.margin()[flexEndEdge(axis)]; } }