Fix issue where we were not centering absolute nodes correctly when justifying (#1489)
Summary: Pull Request resolved: https://github.com/facebook/yoga/pull/1489 X-link: https://github.com/facebook/react-native/pull/41690 Centering involves centering the margin box in the content box of the parent, and then getting the distance from the flex start edge of the parent to the child Reviewed By: NickGerleman Differential Revision: D51383625 fbshipit-source-id: 6bbbace95689ef39c35303bea4b99505952df457
This commit is contained in:
Joe Vilches
committed by
Facebook GitHub Bot
Facebook GitHub Bot
parent
f8d048bb1a
commit
897f9b7423
@@ -4555,3 +4555,245 @@ test('static_position_both_insets_set_amalgamation', () => {
|
||||
config.free();
|
||||
}
|
||||
});
|
||||
test('static_position_justify_center_amalgamation', () => {
|
||||
const config = Yoga.Config.create();
|
||||
let root;
|
||||
|
||||
config.setExperimentalFeatureEnabled(ExperimentalFeature.AbsolutePercentageAgainstPaddingEdge, true);
|
||||
|
||||
try {
|
||||
root = Yoga.Node.create(config);
|
||||
root.setPositionType(PositionType.Absolute);
|
||||
|
||||
const root_child0 = Yoga.Node.create(config);
|
||||
root_child0.setMargin(Edge.Left, 4);
|
||||
root_child0.setMargin(Edge.Top, 5);
|
||||
root_child0.setMargin(Edge.Right, 9);
|
||||
root_child0.setMargin(Edge.Bottom, 1);
|
||||
root_child0.setPadding(Edge.Left, 2);
|
||||
root_child0.setPadding(Edge.Top, 9);
|
||||
root_child0.setPadding(Edge.Right, 11);
|
||||
root_child0.setPadding(Edge.Bottom, 13);
|
||||
root_child0.setBorder(Edge.Left, 5);
|
||||
root_child0.setBorder(Edge.Top, 6);
|
||||
root_child0.setBorder(Edge.Right, 7);
|
||||
root_child0.setBorder(Edge.Bottom, 8);
|
||||
root.insertChild(root_child0, 0);
|
||||
|
||||
const root_child0_child0 = Yoga.Node.create(config);
|
||||
root_child0_child0.setJustifyContent(Justify.Center);
|
||||
root_child0_child0.setPositionType(PositionType.Static);
|
||||
root_child0_child0.setMargin(Edge.Left, 8);
|
||||
root_child0_child0.setMargin(Edge.Top, 6);
|
||||
root_child0_child0.setMargin(Edge.Right, 3);
|
||||
root_child0_child0.setMargin(Edge.Bottom, 9);
|
||||
root_child0_child0.setPadding(Edge.Left, 1);
|
||||
root_child0_child0.setPadding(Edge.Top, 7);
|
||||
root_child0_child0.setPadding(Edge.Right, 9);
|
||||
root_child0_child0.setPadding(Edge.Bottom, 4);
|
||||
root_child0_child0.setBorder(Edge.Left, 8);
|
||||
root_child0_child0.setBorder(Edge.Top, 10);
|
||||
root_child0_child0.setBorder(Edge.Right, 2);
|
||||
root_child0_child0.setBorder(Edge.Bottom, 1);
|
||||
root_child0.insertChild(root_child0_child0, 0);
|
||||
|
||||
const root_child0_child0_child0 = Yoga.Node.create(config);
|
||||
root_child0_child0_child0.setPositionType(PositionType.Absolute);
|
||||
root_child0_child0_child0.setMargin(Edge.Left, 9);
|
||||
root_child0_child0_child0.setMargin(Edge.Top, 12);
|
||||
root_child0_child0_child0.setMargin(Edge.Right, 4);
|
||||
root_child0_child0_child0.setMargin(Edge.Bottom, 7);
|
||||
root_child0_child0_child0.setPadding(Edge.Left, 5);
|
||||
root_child0_child0_child0.setPadding(Edge.Top, 3);
|
||||
root_child0_child0_child0.setPadding(Edge.Right, 8);
|
||||
root_child0_child0_child0.setPadding(Edge.Bottom, 10);
|
||||
root_child0_child0_child0.setBorder(Edge.Left, 2);
|
||||
root_child0_child0_child0.setBorder(Edge.Top, 1);
|
||||
root_child0_child0_child0.setBorder(Edge.Right, 5);
|
||||
root_child0_child0_child0.setBorder(Edge.Bottom, 9);
|
||||
root_child0_child0_child0.setWidth("21%");
|
||||
root_child0_child0.insertChild(root_child0_child0_child0, 0);
|
||||
|
||||
const root_child0_child0_child0_child0 = Yoga.Node.create(config);
|
||||
root_child0_child0_child0_child0.setMargin(Edge.Left, 9);
|
||||
root_child0_child0_child0_child0.setMargin(Edge.Top, 12);
|
||||
root_child0_child0_child0_child0.setMargin(Edge.Right, 4);
|
||||
root_child0_child0_child0_child0.setMargin(Edge.Bottom, 7);
|
||||
root_child0_child0_child0_child0.setPadding(Edge.Left, 5);
|
||||
root_child0_child0_child0_child0.setPadding(Edge.Top, 3);
|
||||
root_child0_child0_child0_child0.setPadding(Edge.Right, 8);
|
||||
root_child0_child0_child0_child0.setPadding(Edge.Bottom, 10);
|
||||
root_child0_child0_child0_child0.setBorder(Edge.Left, 2);
|
||||
root_child0_child0_child0_child0.setBorder(Edge.Top, 1);
|
||||
root_child0_child0_child0_child0.setBorder(Edge.Right, 5);
|
||||
root_child0_child0_child0_child0.setBorder(Edge.Bottom, 9);
|
||||
root_child0_child0_child0_child0.setWidth(100);
|
||||
root_child0_child0_child0_child0.setHeight(50);
|
||||
root_child0_child0_child0.insertChild(root_child0_child0_child0_child0, 0);
|
||||
|
||||
const root_child0_child0_child1 = Yoga.Node.create(config);
|
||||
root_child0_child0_child1.setMargin(Edge.Left, 9);
|
||||
root_child0_child0_child1.setMargin(Edge.Top, 12);
|
||||
root_child0_child0_child1.setMargin(Edge.Right, 4);
|
||||
root_child0_child0_child1.setMargin(Edge.Bottom, 7);
|
||||
root_child0_child0_child1.setPadding(Edge.Left, 5);
|
||||
root_child0_child0_child1.setPadding(Edge.Top, 3);
|
||||
root_child0_child0_child1.setPadding(Edge.Right, 8);
|
||||
root_child0_child0_child1.setPadding(Edge.Bottom, 10);
|
||||
root_child0_child0_child1.setBorder(Edge.Left, 2);
|
||||
root_child0_child0_child1.setBorder(Edge.Top, 1);
|
||||
root_child0_child0_child1.setBorder(Edge.Right, 5);
|
||||
root_child0_child0_child1.setBorder(Edge.Bottom, 9);
|
||||
root_child0_child0_child1.setWidth("10%");
|
||||
root_child0_child0.insertChild(root_child0_child0_child1, 1);
|
||||
|
||||
const root_child0_child0_child1_child0 = Yoga.Node.create(config);
|
||||
root_child0_child0_child1_child0.setMargin(Edge.Left, 9);
|
||||
root_child0_child0_child1_child0.setMargin(Edge.Top, 12);
|
||||
root_child0_child0_child1_child0.setMargin(Edge.Right, 4);
|
||||
root_child0_child0_child1_child0.setMargin(Edge.Bottom, 7);
|
||||
root_child0_child0_child1_child0.setPadding(Edge.Left, 5);
|
||||
root_child0_child0_child1_child0.setPadding(Edge.Top, 3);
|
||||
root_child0_child0_child1_child0.setPadding(Edge.Right, 8);
|
||||
root_child0_child0_child1_child0.setPadding(Edge.Bottom, 10);
|
||||
root_child0_child0_child1_child0.setBorder(Edge.Left, 2);
|
||||
root_child0_child0_child1_child0.setBorder(Edge.Top, 1);
|
||||
root_child0_child0_child1_child0.setBorder(Edge.Right, 5);
|
||||
root_child0_child0_child1_child0.setBorder(Edge.Bottom, 9);
|
||||
root_child0_child0_child1_child0.setWidth(100);
|
||||
root_child0_child0_child1_child0.setHeight(50);
|
||||
root_child0_child0_child1.insertChild(root_child0_child0_child1_child0, 0);
|
||||
|
||||
const root_child0_child0_child2 = Yoga.Node.create(config);
|
||||
root_child0_child0_child2.setMargin(Edge.Left, 9);
|
||||
root_child0_child0_child2.setMargin(Edge.Top, 12);
|
||||
root_child0_child0_child2.setMargin(Edge.Right, 4);
|
||||
root_child0_child0_child2.setMargin(Edge.Bottom, 7);
|
||||
root_child0_child0_child2.setPadding(Edge.Left, 5);
|
||||
root_child0_child0_child2.setPadding(Edge.Top, 3);
|
||||
root_child0_child0_child2.setPadding(Edge.Right, 8);
|
||||
root_child0_child0_child2.setPadding(Edge.Bottom, 10);
|
||||
root_child0_child0_child2.setBorder(Edge.Left, 2);
|
||||
root_child0_child0_child2.setBorder(Edge.Top, 1);
|
||||
root_child0_child0_child2.setBorder(Edge.Right, 5);
|
||||
root_child0_child0_child2.setBorder(Edge.Bottom, 9);
|
||||
root_child0_child0_child2.setWidth("10%");
|
||||
root_child0_child0.insertChild(root_child0_child0_child2, 2);
|
||||
|
||||
const root_child0_child0_child2_child0 = Yoga.Node.create(config);
|
||||
root_child0_child0_child2_child0.setMargin(Edge.Left, 9);
|
||||
root_child0_child0_child2_child0.setMargin(Edge.Top, 12);
|
||||
root_child0_child0_child2_child0.setMargin(Edge.Right, 4);
|
||||
root_child0_child0_child2_child0.setMargin(Edge.Bottom, 7);
|
||||
root_child0_child0_child2_child0.setPadding(Edge.Left, 5);
|
||||
root_child0_child0_child2_child0.setPadding(Edge.Top, 3);
|
||||
root_child0_child0_child2_child0.setPadding(Edge.Right, 8);
|
||||
root_child0_child0_child2_child0.setPadding(Edge.Bottom, 10);
|
||||
root_child0_child0_child2_child0.setBorder(Edge.Left, 2);
|
||||
root_child0_child0_child2_child0.setBorder(Edge.Top, 1);
|
||||
root_child0_child0_child2_child0.setBorder(Edge.Right, 5);
|
||||
root_child0_child0_child2_child0.setBorder(Edge.Bottom, 9);
|
||||
root_child0_child0_child2_child0.setWidth(100);
|
||||
root_child0_child0_child2_child0.setHeight(50);
|
||||
root_child0_child0_child2.insertChild(root_child0_child0_child2_child0, 0);
|
||||
root.calculateLayout(undefined, undefined, Direction.LTR);
|
||||
|
||||
expect(root.getComputedLeft()).toBe(0);
|
||||
expect(root.getComputedTop()).toBe(0);
|
||||
expect(root.getComputedWidth()).toBe(215);
|
||||
expect(root.getComputedHeight()).toBe(301);
|
||||
|
||||
expect(root_child0.getComputedLeft()).toBe(4);
|
||||
expect(root_child0.getComputedTop()).toBe(5);
|
||||
expect(root_child0.getComputedWidth()).toBe(202);
|
||||
expect(root_child0.getComputedHeight()).toBe(295);
|
||||
|
||||
expect(root_child0_child0.getComputedLeft()).toBe(15);
|
||||
expect(root_child0_child0.getComputedTop()).toBe(21);
|
||||
expect(root_child0_child0.getComputedWidth()).toBe(166);
|
||||
expect(root_child0_child0.getComputedHeight()).toBe(244);
|
||||
|
||||
expect(root_child0_child0_child0.getComputedLeft()).toBe(18);
|
||||
expect(root_child0_child0_child0.getComputedTop()).toBe(85);
|
||||
expect(root_child0_child0_child0.getComputedWidth()).toBe(40);
|
||||
expect(root_child0_child0_child0.getComputedHeight()).toBe(92);
|
||||
|
||||
expect(root_child0_child0_child0_child0.getComputedLeft()).toBe(16);
|
||||
expect(root_child0_child0_child0_child0.getComputedTop()).toBe(16);
|
||||
expect(root_child0_child0_child0_child0.getComputedWidth()).toBe(100);
|
||||
expect(root_child0_child0_child0_child0.getComputedHeight()).toBe(50);
|
||||
|
||||
expect(root_child0_child0_child1.getComputedLeft()).toBe(18);
|
||||
expect(root_child0_child0_child1.getComputedTop()).toBe(29);
|
||||
expect(root_child0_child0_child1.getComputedWidth()).toBe(20);
|
||||
expect(root_child0_child0_child1.getComputedHeight()).toBe(92);
|
||||
|
||||
expect(root_child0_child0_child1_child0.getComputedLeft()).toBe(16);
|
||||
expect(root_child0_child0_child1_child0.getComputedTop()).toBe(16);
|
||||
expect(root_child0_child0_child1_child0.getComputedWidth()).toBe(100);
|
||||
expect(root_child0_child0_child1_child0.getComputedHeight()).toBe(50);
|
||||
|
||||
expect(root_child0_child0_child2.getComputedLeft()).toBe(18);
|
||||
expect(root_child0_child0_child2.getComputedTop()).toBe(140);
|
||||
expect(root_child0_child0_child2.getComputedWidth()).toBe(20);
|
||||
expect(root_child0_child0_child2.getComputedHeight()).toBe(92);
|
||||
|
||||
expect(root_child0_child0_child2_child0.getComputedLeft()).toBe(16);
|
||||
expect(root_child0_child0_child2_child0.getComputedTop()).toBe(16);
|
||||
expect(root_child0_child0_child2_child0.getComputedWidth()).toBe(100);
|
||||
expect(root_child0_child0_child2_child0.getComputedHeight()).toBe(50);
|
||||
|
||||
root.calculateLayout(undefined, undefined, Direction.RTL);
|
||||
|
||||
expect(root.getComputedLeft()).toBe(0);
|
||||
expect(root.getComputedTop()).toBe(0);
|
||||
expect(root.getComputedWidth()).toBe(215);
|
||||
expect(root.getComputedHeight()).toBe(301);
|
||||
|
||||
expect(root_child0.getComputedLeft()).toBe(4);
|
||||
expect(root_child0.getComputedTop()).toBe(5);
|
||||
expect(root_child0.getComputedWidth()).toBe(202);
|
||||
expect(root_child0.getComputedHeight()).toBe(295);
|
||||
|
||||
expect(root_child0_child0.getComputedLeft()).toBe(15);
|
||||
expect(root_child0_child0.getComputedTop()).toBe(21);
|
||||
expect(root_child0_child0.getComputedWidth()).toBe(166);
|
||||
expect(root_child0_child0.getComputedHeight()).toBe(244);
|
||||
|
||||
expect(root_child0_child0_child0.getComputedLeft()).toBe(111);
|
||||
expect(root_child0_child0_child0.getComputedTop()).toBe(85);
|
||||
expect(root_child0_child0_child0.getComputedWidth()).toBe(40);
|
||||
expect(root_child0_child0_child0.getComputedHeight()).toBe(92);
|
||||
|
||||
expect(root_child0_child0_child0_child0.getComputedLeft()).toBe(-77);
|
||||
expect(root_child0_child0_child0_child0.getComputedTop()).toBe(16);
|
||||
expect(root_child0_child0_child0_child0.getComputedWidth()).toBe(100);
|
||||
expect(root_child0_child0_child0_child0.getComputedHeight()).toBe(50);
|
||||
|
||||
expect(root_child0_child0_child1.getComputedLeft()).toBe(131);
|
||||
expect(root_child0_child0_child1.getComputedTop()).toBe(29);
|
||||
expect(root_child0_child0_child1.getComputedWidth()).toBe(20);
|
||||
expect(root_child0_child0_child1.getComputedHeight()).toBe(92);
|
||||
|
||||
expect(root_child0_child0_child1_child0.getComputedLeft()).toBe(-97);
|
||||
expect(root_child0_child0_child1_child0.getComputedTop()).toBe(16);
|
||||
expect(root_child0_child0_child1_child0.getComputedWidth()).toBe(100);
|
||||
expect(root_child0_child0_child1_child0.getComputedHeight()).toBe(50);
|
||||
|
||||
expect(root_child0_child0_child2.getComputedLeft()).toBe(131);
|
||||
expect(root_child0_child0_child2.getComputedTop()).toBe(140);
|
||||
expect(root_child0_child0_child2.getComputedWidth()).toBe(20);
|
||||
expect(root_child0_child0_child2.getComputedHeight()).toBe(92);
|
||||
|
||||
expect(root_child0_child0_child2_child0.getComputedLeft()).toBe(-97);
|
||||
expect(root_child0_child0_child2_child0.getComputedTop()).toBe(16);
|
||||
expect(root_child0_child0_child2_child0.getComputedWidth()).toBe(100);
|
||||
expect(root_child0_child0_child2_child0.getComputedHeight()).toBe(50);
|
||||
} finally {
|
||||
if (typeof root !== 'undefined') {
|
||||
root.freeRecursive();
|
||||
}
|
||||
|
||||
config.free();
|
||||
}
|
||||
});
|
||||
|
||||
Reference in New Issue
Block a user