Skip to content
This repository was archived by the owner on Jan 6, 2025. It is now read-only.

fix(fxFlex): fxFlex=auto with overlapping breakpoints activated #183

Merged
merged 1 commit into from
Feb 21, 2017

Conversation

ThomasBurleson
Copy link
Contributor

@ThomasBurleson ThomasBurleson commented Feb 14, 2017

Layout changes from gt-sm will not restore default layout with auto

fix_135

fixes #135.

@ThomasBurleson
Copy link
Contributor Author

@d3viant0ne - plz review.

@ThomasBurleson ThomasBurleson added this to the v2.0.0-rc.1 milestone Feb 14, 2017
.gitignore Outdated
@@ -18,6 +18,8 @@ build/Release

# Users Environment Variables
.lock-wscript
yarn.lock
Copy link
Contributor

Choose a reason for hiding this comment

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

As opposed to ignoring the lock file, I would suggest adding a .gitattributes file and then treating it as a binary. Removes the merge conflict issue while still providing the ability to easily see any shifting dependencies upstream.

i.e

# .gitattributes
yarn.lock -diff
* text=auto
bin/* eol=lf

@ThomasBurleson ThomasBurleson force-pushed the fix/issue_135 branch 2 times, most recently from 61e9b38 to 116ad70 Compare February 15, 2017 00:51
@ThomasBurleson ThomasBurleson added pr: lgtm This PR has been approved by the reviewer pr: needs presubmit pr: merge #3 and removed pr: needs review labels Feb 15, 2017
import {DemoIssue5345} from "./issue.5345.demo";
import {DemoIssue9897} from "./issue.9897.demo";
import {DemoIssue181} from './issue.181.demo';
import {DemoIssue5345} from "./issue.5345.demo";
Copy link
Contributor

Choose a reason for hiding this comment

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

these first 2 are double imported

Layout changes from gt-sm will not restore default layout with auto

fixes #135.
@mmalerba mmalerba merged commit cb614ed into master Feb 21, 2017
@ThomasBurleson ThomasBurleson deleted the fix/issue_135 branch March 4, 2017 22:54
karlhaas pushed a commit to karlhaas/flex-layout that referenced this pull request May 3, 2017
…lar#183)

Layout changes from gt-sm will not restore default layout with auto

fixes angular#135.
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Sep 5, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla: yes pr: lgtm This PR has been approved by the reviewer pr: needs presubmit
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Layout with fxFlex="auto" not restoring max-height values properly
4 participants