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

md-tabs fail when used with ng-include #1788

Closed
hodeyp opened this issue Mar 4, 2015 · 6 comments
Closed

md-tabs fail when used with ng-include #1788

hodeyp opened this issue Mar 4, 2015 · 6 comments
Assignees
Milestone

Comments

@hodeyp
Copy link

hodeyp commented Mar 4, 2015

I have a page that adds a number of templates using ng-include. One of the templates includes an md-tabs component.

The tabs are not switching correctly, when selecting the second tab, the first tab remains visible. I have tried the template outside of an ng-include and it works as to should do.

@bkeil
Copy link

bkeil commented Mar 10, 2015

I am having a similar issue, which may be related:

I also have an md-tabs included by an ng-include. The first tab looks fine. When I switch to the second tab, however, none of the {{ }} templates have been evaluated. They appear raw, curly braces and all.

Bug is present in 0.8.2 and 0.8.3

@ThomasBurleson ThomasBurleson modified the milestone: 0.9.0 Mar 11, 2015
@robertmesserle robertmesserle self-assigned this Mar 12, 2015
@robertmesserle
Copy link
Contributor

@hodeyp @bkeil I suspect this may be resolved by #1944. Can one of you provide some sort of online sample demonstrating the issue so that I can confirm?

@hodeyp
Copy link
Author

hodeyp commented Mar 19, 2015

@bkeil
Copy link

bkeil commented Mar 20, 2015

Here's a nice simple example with my problem: http://plnkr.co/edit/A7UaSk1vvIo9svw9JkIt

All of the tabs are working except for "Outer B » Inner B".

robertmesserle added a commit that referenced this issue Mar 20, 2015
Closes #1087
Closes #1107
Closes #1140
Closes #1247
Closes #1261
Closes #1380
Closes #1387
Closes #1403
Closes #1443
Closes #1505
Closes #1506
Closes #1516
Closes #1518
Closes #1564
Closes #1570
Closes #1620
Closes #1626
Closes #1698
Closes #1777
Closes #1788
Closes #1850
Closes #1959
@robertmesserle
Copy link
Contributor

This is resolved in the latest tabs rewrite - should be in master sometime next week.

@hodeyp
Copy link
Author

hodeyp commented Mar 20, 2015

awesome, thanks!

robertmesserle added a commit that referenced this issue Mar 20, 2015
Closes #1087
Closes #1107
Closes #1140
Closes #1247
Closes #1261
Closes #1380
Closes #1387
Closes #1403
Closes #1443
Closes #1505
Closes #1506
Closes #1516
Closes #1518
Closes #1564
Closes #1570
Closes #1620
Closes #1626
Closes #1698
Closes #1777
Closes #1788
Closes #1850
Closes #1959
Closes #1986
robertmesserle added a commit that referenced this issue Mar 24, 2015
BREAKING CHANGE: Generated HTML structure has changed, so custom styles
will need to be updated to match the new HTML structure.

Closes #1087
Closes #1107
Closes #1140
Closes #1247
Closes #1261
Closes #1380
Closes #1387
Closes #1403
Closes #1443
Closes #1505
Closes #1506
Closes #1516
Closes #1518
Closes #1564
Closes #1570
Closes #1620
Closes #1626
Closes #1698
Closes #1777
Closes #1788
Closes #1850
Closes #1959
Closes #1986
robertmesserle added a commit that referenced this issue Mar 24, 2015
BREAKING CHANGE: Generated HTML structure has changed, so custom styles
will need to be updated to match the new HTML structure.

Closes #1087
Closes #1107
Closes #1140
Closes #1247
Closes #1261
Closes #1380
Closes #1387
Closes #1403
Closes #1443
Closes #1505
Closes #1506
Closes #1516
Closes #1518
Closes #1564
Closes #1570
Closes #1620
Closes #1626
Closes #1698
Closes #1777
Closes #1788
Closes #1850
Closes #1959
Closes #1986
robertmesserle added a commit that referenced this issue Mar 25, 2015
BREAKING CHANGE: Generated HTML structure has changed, so custom styles
will need to be updated to match the new HTML structure.

Closes #1087
Closes #1107
Closes #1140
Closes #1247
Closes #1261
Closes #1380
Closes #1387
Closes #1403
Closes #1443
Closes #1505
Closes #1506
Closes #1516
Closes #1518
Closes #1564
Closes #1570
Closes #1620
Closes #1626
Closes #1698
Closes #1777
Closes #1788
Closes #1850
Closes #1959
Closes #1986
robertmesserle added a commit that referenced this issue Mar 25, 2015
BREAKING CHANGE: Generated HTML structure has changed, so custom styles
will need to be updated to match the new HTML structure.

Closes #1087
Closes #1107
Closes #1140
Closes #1247
Closes #1261
Closes #1380
Closes #1387
Closes #1403
Closes #1443
Closes #1505
Closes #1506
Closes #1516
Closes #1518
Closes #1564
Closes #1570
Closes #1620
Closes #1626
Closes #1698
Closes #1777
Closes #1788
Closes #1850
Closes #1959
Closes #1986
Closes #2020
robertmesserle added a commit that referenced this issue Mar 26, 2015
BREAKING CHANGE: Generated HTML structure has changed, so custom styles
will need to be updated to match the new HTML structure.

Closes #1087
Closes #1107
Closes #1140
Closes #1247
Closes #1261
Closes #1380
Closes #1387
Closes #1403
Closes #1443
Closes #1505
Closes #1506
Closes #1516
Closes #1518
Closes #1564
Closes #1570
Closes #1620
Closes #1626
Closes #1698
Closes #1777
Closes #1788
Closes #1850
Closes #1959
Closes #1986
Closes #2020
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 a pull request may close this issue.

5 participants