Skip to content
This repository has been archived by the owner on May 29, 2019. It is now read-only.

refactor(tab): change to use centralized active state #5425

Closed
wants to merge 1 commit into from

Conversation

wesleycho
Copy link
Contributor

  • Change to store active index on tabset controller
  • Set index on tab by tab basis

BREAKING CHANGE: The tab API is changed - please refer to the
documentation on new usage of tabs

This implements the hardest part of #4836.

- Change to store active index on tabset controller
- Set index on tab by tab basis

BREAKING CHANGE: The tab API is changed - please refer to the
documentation on new usage of tabs. Note that if you are using custom
templates, the templates will need to be changed slightly.
@@ -37,6 +42,9 @@ AngularJS version of the tabs directive.
* `heading` -
Heading text.

* `index` -
Tab index
Copy link
Contributor

Choose a reason for hiding this comment

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

missing a period.

@Foxandxss
Copy link
Contributor

Apart from those two points, works for me.

@wesleycho wesleycho added this to the 1.2.0 milestone Feb 8, 2016
@wesleycho wesleycho closed this in 5ec19b6 Feb 8, 2016
@wesleycho wesleycho deleted the refactor/tab-api branch February 8, 2016 14:23
This was referenced Feb 16, 2016
deeg pushed a commit that referenced this pull request Feb 17, 2016
This change makes the active binding added under #5425 optional.

Closes #5489
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.

2 participants