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

Removed need for empty dropdown attributes. #3460

Closed
wants to merge 1 commit into from
Closed

Removed need for empty dropdown attributes. #3460

wants to merge 1 commit into from

Conversation

palaslet
Copy link
Contributor

By adding "restrict: 'EAC'" to the directives, we no longer need to add the empty dropdown and dropdown-toggle directives.
This need for duplicity (both class and attribute) contributes to more clutter in the html.

By adding "restrict: 'EAC'" to the directives, we no longer need to add the empty dropdown and dropdown-toggle directives.
This need for duplicity (both class and attribute) contributes to more clutter in the html.
@chrisirhc
Copy link
Contributor

We've removed class restrict intentionally. See #2156 for discussion. The class attribute causes unintentional conflict. It's also too generic as a class. Users may have code that just uses the dropdown class and don't expect this directive to link on those elements.

@wesleycho
Copy link
Contributor

Closing this, due to the explicit removal of support as mentioned in the above referenced issue.

@wesleycho wesleycho closed this Jul 19, 2015
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.

4 participants