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

[flex] Don't apply transferred min/max sizes. #48452

Merged
merged 1 commit into from
Oct 3, 2024

Conversation

chromium-wpt-export-bot
Copy link
Collaborator

@chromium-wpt-export-bot chromium-wpt-export-bot commented Oct 3, 2024

https://chromium.googlesource.com/chromium/src/+/f357fe9f912bd1723fb73fdb693c6825dae23390

... introduced a change where we applied the transferred min/max sizes
to the inline min/max sizes.

This was unintentional. The spec is unclear on this situation (after
checking with the spec author), so filed an issue here:
w3c/csswg-drafts#10997

We may change back again, but at least this brings us in-line with
other browsers for the moment.

Fixed: 368562235
Change-Id: I6f353e7d2e1ef0e1077af2b3b76c5f42632cc6b6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5907910
Reviewed-by: David Grogan <dgrogan@chromium.org>
Commit-Queue: Ian Kilpatrick <ikilpatrick@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1363796}

https://chromium.googlesource.com/chromium/src/+/f357fe9f912bd1723fb73fdb693c6825dae23390

... introduced a change where we applied the transferred min/max sizes
to the inline min/max sizes.

This was unintentional. The spec is unclear on this situation (after
checking with the spec author), so filed an issue here:
w3c/csswg-drafts#10997

We may change back again, but at least this brings us in-line with
other browsers for the moment.

Fixed: 368562235
Change-Id: I6f353e7d2e1ef0e1077af2b3b76c5f42632cc6b6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5907910
Reviewed-by: David Grogan <dgrogan@chromium.org>
Commit-Queue: Ian Kilpatrick <ikilpatrick@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1363796}
Copy link
Collaborator

@wpt-pr-bot wpt-pr-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The review process for this patch is being conducted in the Chromium project.

@chromium-wpt-export-bot chromium-wpt-export-bot merged commit 2ef4a5b into master Oct 3, 2024
20 checks passed
@chromium-wpt-export-bot chromium-wpt-export-bot deleted the chromium-export-cl-5907910 branch October 3, 2024 20:22
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.

3 participants