Skip to content
This repository was archived by the owner on Sep 5, 2024. It is now read-only.

fix(layout): fix use of flex-basis in layout modes #9572

Merged
merged 1 commit into from
Sep 22, 2016

Conversation

ThomasBurleson
Copy link
Contributor

Fix use of flex-basis when setting the max-width or max-height values.
This allows flex-offset to work flex items correctly stretched.

Fixes #5345.

@ThomasBurleson ThomasBurleson force-pushed the wip/layout-flex-basis-fix branch from 937f852 to 453517c Compare September 10, 2016 19:44
@ThomasBurleson ThomasBurleson added this to the 1.1.2 milestone Sep 10, 2016
@ThomasBurleson ThomasBurleson added ui: layout ui: responsive needs: review This PR is waiting on review from the team labels Sep 10, 2016
@EladBezalel
Copy link
Member

Manual testing anomalies

Home page

localhost

image

head

image

Checkbox syncing demo

localhost

image

head

image

Colors Theming Demo

localhost

image

head

image

Whiteframe Directive Interpolation Demo

localhost

BREAKING

head

image

Css Typography

localhost

image

head

image

Layout Children

localhost

image

head

image

Layout Troubleshooting

localhost

image

head

image

@crisbeto
Copy link
Member

I went through it as well. Basically got the same as @EladBezalel

@ThomasBurleson
Copy link
Contributor Author

@EladBezalel, @crisbeto - thx

@ThomasBurleson ThomasBurleson added needs: work in progress Mainly for in progress PRs, but may be used for issues that require multiple PRs and removed needs: review This PR is waiting on review from the team labels Sep 13, 2016
Fix use of `flex-basis` when setting the max-width or max-height values.
This allows `flex-offset` to work flex items correctly stretched.

Fixes #5345.

  Closes #9572
@ThomasBurleson ThomasBurleson force-pushed the wip/layout-flex-basis-fix branch from 453517c to af5d99b Compare September 16, 2016 12:10
@ThomasBurleson ThomasBurleson added needs: review This PR is waiting on review from the team and removed in progress Mainly for in progress PRs, but may be used for issues that require multiple PRs needs: work labels Sep 16, 2016
@ThomasBurleson
Copy link
Contributor Author

@EladBezalel - please review again. All above ^ should be fixed/resolved.

Copy link
Member

@EladBezalel EladBezalel left a comment

Choose a reason for hiding this comment

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

Tested locally, LGTM!

@ThomasBurleson ThomasBurleson added needs: presubmit and removed needs: review This PR is waiting on review from the team labels Sep 22, 2016
@kara kara added pr: merge ready This PR is ready for a caretaker to review and removed needs: presubmit labels Sep 22, 2016
@kara kara merged commit c6fb5a5 into master Sep 22, 2016
@EladBezalel EladBezalel deleted the wip/layout-flex-basis-fix branch October 2, 2016 08:42
Frank3K pushed a commit to Frank3K/material that referenced this pull request Oct 8, 2016
Fix use of `flex-basis` when setting the max-width or max-height values.
This allows `flex-offset` to work flex items correctly stretched.

Fixes angular#5345.

  Closes angular#9572
vologab added a commit to vologab/material that referenced this pull request Mar 3, 2017
@FDiskas
Copy link

FDiskas commented Mar 17, 2017

#10345 🎎

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
pr: merge ready This PR is ready for a caretaker to review ui: layout ui: responsive
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants