Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make position static behave like position static #41685

Closed
wants to merge 2 commits into from

Conversation

joevilches
Copy link
Contributor

Summary: This is the final step (that I know of) to get the core features of static working. Here we turn on all of the tests and pass down the correct owner size for the call to calculateLayoutInternal that is in layoutAbsoluteChild

Reviewed By: NickGerleman

Differential Revision: D51293606

@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. p: Facebook Partner: Facebook Partner labels Nov 29, 2023
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D51293606

joevilches added a commit to joevilches/react-native that referenced this pull request Nov 29, 2023
Summary:

This is the final step (that I know of) to get the core features of static working. Here we turn on all of the tests and pass down the correct owner size for the call to `calculateLayoutInternal` that is in `layoutAbsoluteChild`

Reviewed By: NickGerleman

Differential Revision: D51293606
joevilches added a commit to joevilches/react-native that referenced this pull request Nov 29, 2023
Summary:

This is the final step (that I know of) to get the core features of static working. Here we turn on all of the tests and pass down the correct owner size for the call to `calculateLayoutInternal` that is in `layoutAbsoluteChild`

Reviewed By: NickGerleman

Differential Revision: D51293606
joevilches added a commit to joevilches/react-native that referenced this pull request Nov 29, 2023
Summary:

This is the final step (that I know of) to get the core features of static working. Here we turn on all of the tests and pass down the correct owner size for the call to `calculateLayoutInternal` that is in `layoutAbsoluteChild`

Reviewed By: NickGerleman

Differential Revision: D51293606
joevilches added a commit to joevilches/react-native that referenced this pull request Nov 29, 2023
Summary:

This is the final step (that I know of) to get the core features of static working. Here we turn on all of the tests and pass down the correct owner size for the call to `calculateLayoutInternal` that is in `layoutAbsoluteChild`

Reviewed By: NickGerleman

Differential Revision: D51293606
joevilches added a commit to joevilches/react-native that referenced this pull request Nov 29, 2023
Summary:

This is the final step (that I know of) to get the core features of static working. Here we turn on all of the tests and pass down the correct owner size for the call to `calculateLayoutInternal` that is in `layoutAbsoluteChild`

Reviewed By: NickGerleman

Differential Revision: D51293606
joevilches added a commit to joevilches/yoga that referenced this pull request Nov 29, 2023
Summary:
X-link: facebook/react-native#41685

This is the final step (that I know of) to get the core features of static working. Here we turn on all of the tests and pass down the correct owner size for the call to `calculateLayoutInternal` that is in `layoutAbsoluteChild`

Reviewed By: NickGerleman

Differential Revision: D51293606

fbshipit-source-id: 9e88116761406f9300c9bd260cd7662b9b25ab5f
@analysis-bot
Copy link

analysis-bot commented Nov 29, 2023

Platform Engine Arch Size (bytes) Diff
android hermes arm64-v8a 16,514,812 +1,152
android hermes armeabi-v7a n/a --
android hermes x86 n/a --
android hermes x86_64 n/a --
android jsc arm64-v8a 19,889,495 +5,246
android jsc armeabi-v7a n/a --
android jsc x86 n/a --
android jsc x86_64 n/a --

Base commit: 8081265
Branch: main

joevilches added a commit to joevilches/react-native that referenced this pull request Dec 4, 2023
Summary:
X-link: facebook/yoga#1482


This is the final step (that I know of) to get the core features of static working. Here we turn on all of the tests and pass down the correct owner size for the call to `calculateLayoutInternal` that is in `layoutAbsoluteChild`

Reviewed By: NickGerleman

Differential Revision: D51293606
joevilches added a commit to joevilches/react-native that referenced this pull request Dec 4, 2023
Summary:
X-link: facebook/yoga#1482


This is the final step (that I know of) to get the core features of static working. Here we turn on all of the tests and pass down the correct owner size for the call to `calculateLayoutInternal` that is in `layoutAbsoluteChild`

Reviewed By: NickGerleman

Differential Revision: D51293606
joevilches added a commit to joevilches/react-native that referenced this pull request Dec 4, 2023
Summary:
X-link: facebook/yoga#1482


This is the final step (that I know of) to get the core features of static working. Here we turn on all of the tests and pass down the correct owner size for the call to `calculateLayoutInternal` that is in `layoutAbsoluteChild`

Reviewed By: NickGerleman

Differential Revision: D51293606
joevilches added a commit to joevilches/react-native that referenced this pull request Dec 4, 2023
Summary:
X-link: facebook/yoga#1482


This is the final step (that I know of) to get the core features of static working. Here we turn on all of the tests and pass down the correct owner size for the call to `calculateLayoutInternal` that is in `layoutAbsoluteChild`

Reviewed By: NickGerleman

Differential Revision: D51293606
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D51293606

joevilches added a commit to joevilches/react-native that referenced this pull request Dec 4, 2023
Summary:
X-link: facebook/yoga#1482


This is the final step (that I know of) to get the core features of static working. Here we turn on all of the tests and pass down the correct owner size for the call to `calculateLayoutInternal` that is in `layoutAbsoluteChild`

Reviewed By: NickGerleman

Differential Revision: D51293606
joevilches added a commit to joevilches/react-native that referenced this pull request Dec 4, 2023
Summary:
X-link: facebook/yoga#1482


This is the final step (that I know of) to get the core features of static working. Here we turn on all of the tests and pass down the correct owner size for the call to `calculateLayoutInternal` that is in `layoutAbsoluteChild`

Reviewed By: NickGerleman

Differential Revision: D51293606
joevilches added a commit to joevilches/yoga that referenced this pull request Dec 4, 2023
Summary:
Pull Request resolved: facebook#1482

X-link: facebook/react-native#41685

This is the final step (that I know of) to get the core features of static working. Here we turn on all of the tests and pass down the correct owner size for the call to `calculateLayoutInternal` that is in `layoutAbsoluteChild`

Reviewed By: NickGerleman

Differential Revision: D51293606

fbshipit-source-id: b90de742dae3c635ff8e30702073c87f0bd2e9a9
joevilches added a commit to joevilches/react-native that referenced this pull request Dec 4, 2023
Summary:
X-link: facebook/yoga#1482


This is the final step (that I know of) to get the core features of static working. Here we turn on all of the tests and pass down the correct owner size for the call to `calculateLayoutInternal` that is in `layoutAbsoluteChild`

Reviewed By: NickGerleman

Differential Revision: D51293606
joevilches added a commit to joevilches/react-native that referenced this pull request Dec 5, 2023
Summary:
X-link: facebook/yoga#1482


This is the final step (that I know of) to get the core features of static working. Here we turn on all of the tests and pass down the correct owner size for the call to `calculateLayoutInternal` that is in `layoutAbsoluteChild`

Reviewed By: NickGerleman

Differential Revision: D51293606
joevilches added a commit to joevilches/yoga that referenced this pull request Dec 5, 2023
Summary:

X-link: facebook/react-native#41685

This is the final step (that I know of) to get the core features of static working. Here we turn on all of the tests and pass down the correct owner size for the call to `calculateLayoutInternal` that is in `layoutAbsoluteChild`

Reviewed By: NickGerleman

Differential Revision: D51293606
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D51293606

joevilches added a commit to joevilches/yoga that referenced this pull request Dec 5, 2023
Summary:

X-link: facebook/react-native#41685

This is the final step (that I know of) to get the core features of static working. Here we turn on all of the tests and pass down the correct owner size for the call to `calculateLayoutInternal` that is in `layoutAbsoluteChild`

Reviewed By: NickGerleman

Differential Revision: D51293606
joevilches added a commit to joevilches/react-native that referenced this pull request Dec 5, 2023
Summary:
X-link: facebook/yoga#1482


This is the final step (that I know of) to get the core features of static working. Here we turn on all of the tests and pass down the correct owner size for the call to `calculateLayoutInternal` that is in `layoutAbsoluteChild`

Reviewed By: NickGerleman

Differential Revision: D51293606
joevilches added a commit to joevilches/yoga that referenced this pull request Dec 5, 2023
Summary:

X-link: facebook/react-native#41685

This is the final step (that I know of) to get the core features of static working. Here we turn on all of the tests and pass down the correct owner size for the call to `calculateLayoutInternal` that is in `layoutAbsoluteChild`

Reviewed By: NickGerleman

Differential Revision: D51293606
joevilches added a commit to joevilches/react-native that referenced this pull request Dec 7, 2023
Summary:
X-link: facebook/yoga#1482


This is the final step (that I know of) to get the core features of static working. Here we turn on all of the tests and pass down the correct owner size for the call to `calculateLayoutInternal` that is in `layoutAbsoluteChild`

Reviewed By: NickGerleman

Differential Revision: D51293606
joevilches added a commit to joevilches/yoga that referenced this pull request Dec 7, 2023
Summary:
Pull Request resolved: facebook#1482

X-link: facebook/react-native#41685

This is the final step (that I know of) to get the core features of static working. Here we turn on all of the tests and pass down the correct owner size for the call to `calculateLayoutInternal` that is in `layoutAbsoluteChild`

Differential Revision: https://www.internalfb.com/diff/D51293606?entry_point=27

fbshipit-source-id: 21c7ae7edec1a08b09c7c948d4ef1ee412df23ab
joevilches added a commit to joevilches/yoga that referenced this pull request Dec 7, 2023
Summary:
Pull Request resolved: facebook#1482

X-link: facebook/react-native#41685

This is the final step (that I know of) to get the core features of static working. Here we turn on all of the tests and pass down the correct owner size for the call to `calculateLayoutInternal` that is in `layoutAbsoluteChild`

Differential Revision: https://www.internalfb.com/diff/D51293606?entry_point=27

fbshipit-source-id: 1634e5963603360b351da9531f3bbe7bd814cf9b
joevilches added a commit to joevilches/yoga that referenced this pull request Dec 7, 2023
Summary:
Pull Request resolved: facebook#1482

X-link: facebook/react-native#41685

This is the final step (that I know of) to get the core features of static working. Here we turn on all of the tests and pass down the correct owner size for the call to `calculateLayoutInternal` that is in `layoutAbsoluteChild`

Differential Revision: https://www.internalfb.com/diff/D51293606?entry_point=27

fbshipit-source-id: 4684055f44344f1608e615968b19e82dde7f39de
joevilches added a commit to joevilches/yoga that referenced this pull request Dec 7, 2023
Summary:
Pull Request resolved: facebook#1482

X-link: facebook/react-native#41685

This is the final step (that I know of) to get the core features of static working. Here we turn on all of the tests and pass down the correct owner size for the call to `calculateLayoutInternal` that is in `layoutAbsoluteChild`

Differential Revision: https://www.internalfb.com/diff/D51293606?entry_point=27

fbshipit-source-id: ff285316796d1db811d4dcfaf100c1cb73d4c463
joevilches added a commit to joevilches/yoga that referenced this pull request Dec 7, 2023
Summary:
Pull Request resolved: facebook#1482

X-link: facebook/react-native#41685

This is the final step (that I know of) to get the core features of static working. Here we turn on all of the tests and pass down the correct owner size for the call to `calculateLayoutInternal` that is in `layoutAbsoluteChild`

Differential Revision: https://www.internalfb.com/diff/D51293606?entry_point=27

fbshipit-source-id: aad97dc8ee797582b73544857fd398acaf488b11
joevilches added a commit to joevilches/yoga that referenced this pull request Dec 7, 2023
Summary:
Pull Request resolved: facebook#1482

X-link: facebook/react-native#41685

This is the final step (that I know of) to get the core features of static working. Here we turn on all of the tests and pass down the correct owner size for the call to `calculateLayoutInternal` that is in `layoutAbsoluteChild`

Differential Revision: https://www.internalfb.com/diff/D51293606?entry_point=27

fbshipit-source-id: bea02c6dd891e6be421c07de038755d455c54eff
Joe Vilches and others added 2 commits December 7, 2023 14:39
…ng absolute node's position (facebook#41684)

Summary:
X-link: facebook/yoga#1481


Absolute nodes can be laid out by themselves and do not have to care about what is happening to their siblings. Because of this we can make `positionAbsoluteChild` the sole place where we handle this logic. Right now that is scattered around algorithm with many `if (child is absolute)` cases everywhere. This makes implementing position static a lot harder since we are relying on the CB to do all this work, not the parent.

With this change the only time we set position for an absolute node and it matter (i.e. not overwritten) is in `positionAbsoluteChild`

Reviewed By: NickGerleman

Differential Revision: D51290723
Summary:
X-link: facebook/yoga#1482


This is the final step (that I know of) to get the core features of static working. Here we turn on all of the tests and pass down the correct owner size for the call to `calculateLayoutInternal` that is in `layoutAbsoluteChild`

Reviewed By: NickGerleman

Differential Revision: D51293606
joevilches added a commit to joevilches/yoga that referenced this pull request Dec 7, 2023
Summary:

X-link: facebook/react-native#41685

This is the final step (that I know of) to get the core features of static working. Here we turn on all of the tests and pass down the correct owner size for the call to `calculateLayoutInternal` that is in `layoutAbsoluteChild`

Reviewed By: NickGerleman

Differential Revision: D51293606
joevilches added a commit to joevilches/react-native that referenced this pull request Dec 7, 2023
Summary:
X-link: facebook/yoga#1482


This is the final step (that I know of) to get the core features of static working. Here we turn on all of the tests and pass down the correct owner size for the call to `calculateLayoutInternal` that is in `layoutAbsoluteChild`

Reviewed By: NickGerleman

Differential Revision: D51293606
joevilches added a commit to joevilches/yoga that referenced this pull request Dec 7, 2023
Summary:

X-link: facebook/react-native#41685

This is the final step (that I know of) to get the core features of static working. Here we turn on all of the tests and pass down the correct owner size for the call to `calculateLayoutInternal` that is in `layoutAbsoluteChild`

Reviewed By: NickGerleman

Differential Revision: D51293606
joevilches added a commit to joevilches/react-native that referenced this pull request Dec 7, 2023
Summary:
X-link: facebook/yoga#1482


This is the final step (that I know of) to get the core features of static working. Here we turn on all of the tests and pass down the correct owner size for the call to `calculateLayoutInternal` that is in `layoutAbsoluteChild`

Reviewed By: NickGerleman

Differential Revision: D51293606
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D51293606

joevilches added a commit to joevilches/react-native that referenced this pull request Dec 7, 2023
Summary:
X-link: facebook/yoga#1482


This is the final step (that I know of) to get the core features of static working. Here we turn on all of the tests and pass down the correct owner size for the call to `calculateLayoutInternal` that is in `layoutAbsoluteChild`

Reviewed By: NickGerleman

Differential Revision: D51293606
joevilches added a commit to joevilches/react-native that referenced this pull request Dec 7, 2023
Summary:
X-link: facebook/yoga#1482


This is the final step (that I know of) to get the core features of static working. Here we turn on all of the tests and pass down the correct owner size for the call to `calculateLayoutInternal` that is in `layoutAbsoluteChild`

Reviewed By: NickGerleman

Differential Revision: D51293606
joevilches added a commit to joevilches/react-native that referenced this pull request Dec 7, 2023
Summary:
X-link: facebook/yoga#1482


This is the final step (that I know of) to get the core features of static working. Here we turn on all of the tests and pass down the correct owner size for the call to `calculateLayoutInternal` that is in `layoutAbsoluteChild`

Reviewed By: NickGerleman

Differential Revision: D51293606
joevilches added a commit to joevilches/react-native that referenced this pull request Dec 7, 2023
Summary:
X-link: facebook/yoga#1482


This is the final step (that I know of) to get the core features of static working. Here we turn on all of the tests and pass down the correct owner size for the call to `calculateLayoutInternal` that is in `layoutAbsoluteChild`

Reviewed By: NickGerleman

Differential Revision: D51293606
joevilches added a commit to joevilches/yoga that referenced this pull request Dec 7, 2023
Summary:
Pull Request resolved: facebook#1482

X-link: facebook/react-native#41685

This is the final step (that I know of) to get the core features of static working. Here we turn on all of the tests and pass down the correct owner size for the call to `calculateLayoutInternal` that is in `layoutAbsoluteChild`

Differential Revision: https://www.internalfb.com/diff/D51293606?entry_point=27

fbshipit-source-id: e09fe3951b5a70668ab700c1a11f872a143ea5dd
joevilches added a commit to joevilches/yoga that referenced this pull request Dec 7, 2023
Summary:
Pull Request resolved: facebook#1482

X-link: facebook/react-native#41685

This is the final step (that I know of) to get the core features of static working. Here we turn on all of the tests and pass down the correct owner size for the call to `calculateLayoutInternal` that is in `layoutAbsoluteChild`

Differential Revision: https://www.internalfb.com/diff/D51293606?entry_point=27

fbshipit-source-id: cb3174fe68a652732323881f619fb4e91ae7eb4e
joevilches added a commit to joevilches/yoga that referenced this pull request Dec 7, 2023
Summary:
Pull Request resolved: facebook#1482

X-link: facebook/react-native#41685

This is the final step (that I know of) to get the core features of static working. Here we turn on all of the tests and pass down the correct owner size for the call to `calculateLayoutInternal` that is in `layoutAbsoluteChild`

Differential Revision: https://www.internalfb.com/diff/D51293606?entry_point=27

fbshipit-source-id: 6837a5e6057f0bbce401f91c23989f5bc602800d
joevilches added a commit to joevilches/yoga that referenced this pull request Dec 7, 2023
Summary:
Pull Request resolved: facebook#1482

X-link: facebook/react-native#41685

This is the final step (that I know of) to get the core features of static working. Here we turn on all of the tests and pass down the correct owner size for the call to `calculateLayoutInternal` that is in `layoutAbsoluteChild`

Differential Revision: https://www.internalfb.com/diff/D51293606?entry_point=27

fbshipit-source-id: e4b0b46c984210c0c4b066046917aa67c63e7bce
facebook-github-bot pushed a commit to facebook/litho that referenced this pull request Dec 8, 2023
Summary:
X-link: facebook/yoga#1482

X-link: facebook/react-native#41685

This is the final step (that I know of) to get the core features of static working. Here we turn on all of the tests and pass down the correct owner size for the call to `calculateLayoutInternal` that is in `layoutAbsoluteChild`

Reviewed By: NickGerleman

Differential Revision: D51293606

fbshipit-source-id: 972259e7ebecb19b55aef2ef866bd7cb57aaf0ca
@facebook-github-bot facebook-github-bot added the Merged This PR has been merged. label Dec 8, 2023
@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 475036d.

facebook-github-bot pushed a commit to facebook/yoga that referenced this pull request Dec 8, 2023
Summary:
Pull Request resolved: #1482

X-link: facebook/react-native#41685

This is the final step (that I know of) to get the core features of static working. Here we turn on all of the tests and pass down the correct owner size for the call to `calculateLayoutInternal` that is in `layoutAbsoluteChild`

Reviewed By: NickGerleman

Differential Revision: D51293606

fbshipit-source-id: 972259e7ebecb19b55aef2ef866bd7cb57aaf0ca
Othinn pushed a commit to Othinn/react-native that referenced this pull request Jan 9, 2024
Summary:
X-link: facebook/yoga#1482

Pull Request resolved: facebook#41685

This is the final step (that I know of) to get the core features of static working. Here we turn on all of the tests and pass down the correct owner size for the call to `calculateLayoutInternal` that is in `layoutAbsoluteChild`

Reviewed By: NickGerleman

Differential Revision: D51293606

fbshipit-source-id: 972259e7ebecb19b55aef2ef866bd7cb57aaf0ca
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported Merged This PR has been merged. p: Facebook Partner: Facebook Partner
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants