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

feat(fxFlex): compute immediate parent flex-direction #220

Merged
merged 1 commit into from
Mar 16, 2017

Conversation

ThomasBurleson
Copy link
Contributor

@ThomasBurleson ThomasBurleson commented Mar 15, 2017

Support use of fxFlex directive on HTML element without a fxLayout parent
directive. fxFlex requires a flex-direction to be specific on the immediate parent:

  • use defined flex-direction or getComputed.
  • auto-inject fxLayout styles to parent if parent flex-direction not already provided.

@ThomasBurleson ThomasBurleson force-pushed the thomas/fix-flex-computed-directions branch 6 times, most recently from 8d9c596 to 2ea5dfd Compare March 15, 2017 14:57
@ThomasBurleson ThomasBurleson changed the title fix(fxFlex): compute immediate parent flex-direction feat(fxFlex): compute immediate parent flex-direction Mar 15, 2017
@ThomasBurleson ThomasBurleson force-pushed the thomas/fix-flex-computed-directions branch 3 times, most recently from 8f319a8 to fc823bb Compare March 16, 2017 01:15
Support use of **fxFlex** directive on HTML element without a **fxLayout**  parent
directive. Previous versions of **fxFlex** required a `flex-direction` to be specific on the immediate parent.
Current version improved:

  * use defined flex-direction or getComputed.
  * auto-inject **fxLayout** styles to parent if parent `flex-direction` not already provided.
  * fix ngOnDestory() with fxLayoutWrap
@ThomasBurleson ThomasBurleson force-pushed the thomas/fix-flex-computed-directions branch from fc823bb to 3bfeddd Compare March 16, 2017 01:18
@ThomasBurleson ThomasBurleson added this to the v2.0.0-rc.2 milestone Mar 16, 2017
@ThomasBurleson
Copy link
Contributor Author

@mmalerba - ready for presubmit. Thx.

@mmalerba mmalerba merged commit ba0d85d into master Mar 16, 2017
@ThomasBurleson ThomasBurleson deleted the thomas/fix-flex-computed-directions branch March 30, 2017 23:28
karlhaas pushed a commit to karlhaas/flex-layout that referenced this pull request May 3, 2017
Support use of **fxFlex** directive on HTML element without a **fxLayout**  parent
directive. Previous versions of **fxFlex** required a `flex-direction` to be specific on the immediate parent.
Current version improved:

  * use defined flex-direction or getComputed.
  * auto-inject **fxLayout** styles to parent if parent `flex-direction` not already provided.
  * fix ngOnDestory() with fxLayoutWrap
@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
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants