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

feat(accordion): remove replace: true usage #5985

Closed

Conversation

wesleycho
Copy link
Contributor

  • Remove usage of replace: true
  • Remove unnecessary panel-class feature

BREAKING CHANGE: This removes usage of replace: true in the accordion group, which results in a template change where the template no longer needs to contain the panel itself, but its contents. The accordion group will add the panel class by default, so the user just needs to add the appropriate classes to the accordion group element. This allows the user to use ng-class as well to fully control the panel related classes, so panel-class now is unnecessary

This finishes the first task of #5712.

- Remove usage of `replace: true`
- Remove unnecessary `panel-class` feature

BREAKING CHANGE: This removes usage of `replace: true` in the accordion group, which results in a template change where the template no longer needs to contain the panel itself, but its contents. The accordion group will add the `panel` class by default, so the user just needs to add the appropriate classes to the accordion group element. This allows the user to use ng-class as well to fully control the panel related classes, so `panel-class` now is unnecessary
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