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

[FlexNG] Fix flex-basis for aspect ratio items. #21178

Merged
merged 1 commit into from Feb 12, 2020

Conversation

chromium-wpt-export-bot
Copy link
Collaborator

@chromium-wpt-export-bot chromium-wpt-export-bot commented Jan 15, 2020

This refactors flex base size calculation to more explicitly match the
parts of https://drafts.csswg.org/css-flexbox/#algo-main-item . As a
result it is pretty verbose.

Bug: 845235
Change-Id: I1a49c7f153b8e0779b1d1eaa1206bd5056e9da2e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1986054
Commit-Queue: David Grogan <dgrogan@chromium.org>
Reviewed-by: Ian Kilpatrick <ikilpatrick@chromium.org>
Cr-Commit-Position: refs/heads/master@{#740525}

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 force-pushed the chromium-export-cl-1986054 branch 4 times, most recently from 3b91737 to 9486ff7 Compare January 22, 2020 02:37
@gsnedders gsnedders closed this Jan 24, 2020
@gsnedders gsnedders deleted the chromium-export-cl-1986054 branch January 24, 2020 18:05
@gsnedders gsnedders restored the chromium-export-cl-1986054 branch January 24, 2020 18:52
@Hexcles Hexcles reopened this Jan 24, 2020
@chromium-wpt-export-bot chromium-wpt-export-bot force-pushed the chromium-export-cl-1986054 branch 3 times, most recently from b84c3fc to 759269a Compare January 25, 2020 00:33
@chromium-wpt-export-bot chromium-wpt-export-bot force-pushed the chromium-export-cl-1986054 branch 3 times, most recently from 7421522 to 3bd1e46 Compare February 11, 2020 23:58
This refactors flex base size calculation to more explicitly match the
parts of https://drafts.csswg.org/css-flexbox/#algo-main-item . As a
result it is pretty verbose.

Bug: 845235
Change-Id: I1a49c7f153b8e0779b1d1eaa1206bd5056e9da2e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1986054
Commit-Queue: David Grogan <dgrogan@chromium.org>
Reviewed-by: Ian Kilpatrick <ikilpatrick@chromium.org>
Cr-Commit-Position: refs/heads/master@{#740525}
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.

None yet

5 participants