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

fix(mdToolbar): Better fix for ng-if that allows ng-controller. #4423

Closed
wants to merge 1 commit into from

Conversation

topherfangio
Copy link
Contributor

After modifying the <md-toolbar> directive to allow both
md-scroll-shrink and ng-if, we accidentally broke support
for also using ng-controller as well as some layout problems
due to the transclusion and a wrapping <div>. This reverts
those changes but still allows for ng-if to work.

Also updated docs with some extra info.

Fixes #4414.

After modifying the `<md-toolbar>` directive to allow both
`md-scroll-shrink` and `ng-if`, we accidentally broke support
for also using `ng-controller` as well as some layout problems
due to the transclusion and a wrapping `<div>`. This reverts
those changes but still allows for `ng-if` to work.

Fixes #4144.
@topherfangio topherfangio deleted the fix-toolbar-scroll-shrink-again-4414 branch September 10, 2015 20:41
kennethcachia pushed a commit to kennethcachia/material that referenced this pull request Sep 23, 2015
After modifying the `<md-toolbar>` directive to allow both
`md-scroll-shrink` and `ng-if`, we accidentally broke support
for also using `ng-controller` as well as some layout problems
due to the transclusion and a wrapping `<div>`. This reverts
those changes but still allows for `ng-if` to work.

Fixes angular#4144. Closes angular#4423.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant