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

Fixup margin: auto and justification behavior for overflowed containers #1646

Closed
wants to merge 1 commit into from

Conversation

NickGerleman
Copy link
Contributor

Summary:
Fixes #978

  1. Don't allow auto margin spaces to become a negative length
  2. Replicate fallback alignment behavior specified by box-alignment spec that we are using for align-content.

Differential Revision: D56091577

Copy link

vercel bot commented Apr 13, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
yoga-website ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 17, 2024 6:15pm

@facebook-github-bot
Copy link
Contributor

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

…rs (facebook#1646)

Summary:

X-link: facebook/react-native#44069

Fixes facebook#978

1. Don't allow auto margin spaces to become a negative length
2. Replicate fallback alignment behavior specified by box-alignment spec that we are using for align-content.

Reviewed By: joevilches

Differential Revision: D56091577
NickGerleman added a commit to NickGerleman/react-native that referenced this pull request Apr 17, 2024
…rs (facebook#44069)

Summary:
X-link: facebook/yoga#1646


Fixes facebook/yoga#978

1. Don't allow auto margin spaces to become a negative length
2. Replicate fallback alignment behavior specified by box-alignment spec that we are using for align-content.

Reviewed By: joevilches

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

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

facebook-github-bot pushed a commit to facebook/litho that referenced this pull request Apr 18, 2024
Summary:
X-link: facebook/yoga#1646

X-link: facebook/react-native#44069

Fixes facebook/yoga#978

1. Don't allow auto margin spaces to become a negative length
2. Replicate fallback alignment behavior specified by box-alignment spec that we are using for align-content.

Reviewed By: joevilches

Differential Revision: D56091577

fbshipit-source-id: 3c02f81f969bb947cdc5c80b15faaa0b0d39c0c2
facebook-github-bot pushed a commit to facebook/react-native that referenced this pull request Apr 18, 2024
…rs (#44069)

Summary:
X-link: facebook/yoga#1646

Pull Request resolved: #44069

Fixes facebook/yoga#978

1. Don't allow auto margin spaces to become a negative length
2. Replicate fallback alignment behavior specified by box-alignment spec that we are using for align-content.

Reviewed By: joevilches

Differential Revision: D56091577

fbshipit-source-id: 3c02f81f969bb947cdc5c80b15faaa0b0d39c0c2
@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 932361c.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Margin auto allows negative values
2 participants