Rename AbsolutePositioning errata #1558
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Ignored Deployment
|
|
This pull request was exported from Phabricator. Differential Revision: D52820117 |
joevilches
added a commit
to joevilches/react-native
that referenced
this pull request
Jan 17, 2024
Summary: X-link: facebook/yoga#1558 AbsolutePositioning -> AbsolutePositioningCatchAll A bit more clear. This errata is for various issues with positioning absolute nodes. There really isn't a clear description as to what specifically this enables/disables, so I just opted to say "catch all" to indicate that this controls various bugs Differential Revision: D52820117
|
This pull request was exported from Phabricator. Differential Revision: D52820117 |
joevilches
added a commit
to joevilches/yoga
that referenced
this pull request
Jan 17, 2024
Summary: Pull Request resolved: facebook#1558 X-link: facebook/react-native#42318 AbsolutePositioning -> AbsolutePositioningCatchAll A bit more clear. This errata is for various issues with positioning absolute nodes. There really isn't a clear description as to what specifically this enables/disables, so I just opted to say "catch all" to indicate that this controls various bugs Differential Revision: D52820117 fbshipit-source-id: 7f306a32022de8cbf38734597a68effe358293b6
joevilches
force-pushed
the
export-D52820117
branch
from
54cbcb4
to
4cdf15b
Compare
joevilches
added a commit
to joevilches/react-native
that referenced
this pull request
Jan 18, 2024
Summary: X-link: facebook/yoga#1558 AbsolutePositioning -> AbsolutePositioningCatchAll A bit more clear. This errata is for various issues with positioning absolute nodes. There really isn't a clear description as to what specifically this enables/disables, so I just opted to say "catch all" to indicate that this controls various bugs Reviewed By: NickGerleman Differential Revision: D52820117
joevilches
added a commit
to joevilches/react-native
that referenced
this pull request
Jan 18, 2024
Summary: X-link: facebook/yoga#1558 AbsolutePositioning -> AbsolutePositioningCatchAll A bit more clear. This errata is for various issues with positioning absolute nodes. There really isn't a clear description as to what specifically this enables/disables, so I just opted to say "catch all" to indicate that this controls various bugs Reviewed By: NickGerleman Differential Revision: D52820117
joevilches
added a commit
to joevilches/react-native
that referenced
this pull request
Jan 18, 2024
Summary: X-link: facebook/yoga#1558 AbsolutePositioning -> AbsolutePositioningCatchAll A bit more clear. This errata is for various issues with positioning absolute nodes. There really isn't a clear description as to what specifically this enables/disables, so I just opted to say "catch all" to indicate that this controls various bugs Reviewed By: NickGerleman Differential Revision: D52820117
|
This pull request was exported from Phabricator. Differential Revision: D52820117 |
joevilches
added a commit
to joevilches/yoga
that referenced
this pull request
Jan 18, 2024
Summary: Pull Request resolved: facebook#1558 X-link: facebook/react-native#42318 AbsolutePositioning -> AbsolutePositioningCatchAll A bit more clear. This errata is for various issues with positioning absolute nodes. There really isn't a clear description as to what specifically this enables/disables, so I just opted to say "catch all" to indicate that this controls various bugs Reviewed By: NickGerleman Differential Revision: D52820117 fbshipit-source-id: 21a70973155bb37e437ca4754e746fdae928af21
joevilches
force-pushed
the
export-D52820117
branch
from
4cdf15b
to
b50300c
Compare
|
This pull request was exported from Phabricator. Differential Revision: D52820117 |
joevilches
added a commit
to joevilches/yoga
that referenced
this pull request
Jan 18, 2024
Summary: Pull Request resolved: facebook#1558 X-link: facebook/react-native#42318 AbsolutePositioning -> AbsolutePositioningCatchAll A bit more clear. This errata is for various issues with positioning absolute nodes. There really isn't a clear description as to what specifically this enables/disables, so I just opted to say "catch all" to indicate that this controls various bugs Reviewed By: NickGerleman Differential Revision: D52820117 fbshipit-source-id: f4d82790d3c9aa555f28882fb0e2939c88ad6069
joevilches
force-pushed
the
export-D52820117
branch
from
b50300c
to
8ee9129
Compare
joevilches
added a commit
to joevilches/react-native
that referenced
this pull request
Jan 18, 2024
Summary: X-link: facebook/yoga#1558 AbsolutePositioning -> AbsolutePositioningCatchAll A bit more clear. This errata is for various issues with positioning absolute nodes. There really isn't a clear description as to what specifically this enables/disables, so I just opted to say "catch all" to indicate that this controls various bugs bypass-github-export-checks Reviewed By: NickGerleman Differential Revision: D52820117
joevilches
added a commit
to joevilches/react-native
that referenced
this pull request
Jan 18, 2024
Summary: X-link: facebook/yoga#1558 AbsolutePositioning -> AbsolutePositioningCatchAll A bit more clear. This errata is for various issues with positioning absolute nodes. There really isn't a clear description as to what specifically this enables/disables, so I just opted to say "catch all" to indicate that this controls various bugs bypass-github-export-checks Reviewed By: NickGerleman Differential Revision: D52820117
Differential Revision: https://www.internalfb.com/diff/D52705765?entry_point=27 fbshipit-source-id: 20be9ce21beb11f4afd7a643e226261ec5c89e70
Summary: Pull Request resolved: facebook#1556 X-link: facebook/react-native#42315 Since we aim to ship static to all users of yoga (not just XPR), we need to remove the errata that is gating most of the features. This should be a non breaking change. To ensure that, I added a new errata which, if on, will use the inner size of the containing node as the containing block. This is how it has been for a while and resolving this is risky and time consuming so for the time being we will stick with that. Differential Revision: https://www.internalfb.com/diff/D52706161?entry_point=27 fbshipit-source-id: a2b822adcbb5834758f5f509ea66878b0453246c
Summary: Pull Request resolved: facebook#1558 X-link: facebook/react-native#42318 AbsolutePositioning -> AbsolutePositioningCatchAll A bit more clear. This errata is for various issues with positioning absolute nodes. There really isn't a clear description as to what specifically this enables/disables, so I just opted to say "catch all" to indicate that this controls various bugs bypass-github-export-checks Reviewed By: NickGerleman Differential Revision: D52820117 fbshipit-source-id: e6046c33ad89575c517bc71cff265f4e6b50fc89
|
This pull request was exported from Phabricator. Differential Revision: D52820117 |
facebook-github-bot
pushed a commit
to facebook/litho
that referenced
this pull request
Jan 19, 2024
Summary: X-link: facebook/yoga#1558 X-link: facebook/react-native#42318 AbsolutePositioning -> AbsolutePositioningCatchAll A bit more clear. This errata is for various issues with positioning absolute nodes. There really isn't a clear description as to what specifically this enables/disables, so I just opted to say "catch all" to indicate that this controls various bugs Reviewed By: NickGerleman Differential Revision: D52820117 fbshipit-source-id: 80b77832baf65e68e57ca523c418422dd346ef0f
|
This pull request has been merged in 1541f9d. |
facebook-github-bot
pushed a commit
to facebook/react-native
that referenced
this pull request
Jan 19, 2024
Summary: X-link: facebook/yoga#1558 Pull Request resolved: #42318 AbsolutePositioning -> AbsolutePositioningCatchAll A bit more clear. This errata is for various issues with positioning absolute nodes. There really isn't a clear description as to what specifically this enables/disables, so I just opted to say "catch all" to indicate that this controls various bugs Reviewed By: NickGerleman Differential Revision: D52820117 fbshipit-source-id: 80b77832baf65e68e57ca523c418422dd346ef0f
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary:
X-link: facebook/react-native#42318
AbsolutePositioning -> AbsolutePositioningCatchAll
A bit more clear. This errata is for various issues with positioning absolute nodes. There really isn't a clear description as to what specifically this enables/disables, so I just opted to say "catch all" to indicate that this controls various bugs
Differential Revision: D52820117