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

Deprecate controller as syntax #13738

Open
wesleycho opened this issue Jan 11, 2016 · 3 comments
Open

Deprecate controller as syntax #13738

wesleycho opened this issue Jan 11, 2016 · 3 comments

Comments

@wesleycho
Copy link
Contributor

Given comment here & controller as being recommended against, would it be reasonable to deprecate support for it? It would simplify the regex, and overhead in the controller. The only piece I see this really affecting functionality-wise is use of ng-controller, but ng-controller is an anti-pattern.

@vivainio
Copy link

It may not be useful to deprecate anything in ng 1.x series, as that's supposed to be the boring series ;)

@wesleycho
Copy link
Contributor Author

Well, some things are deprecated such as replace in the directive definition object.

@Narretz
Copy link
Contributor

Narretz commented Jan 11, 2016

We could deprecate it, but we won't remove it. Basically deprecation to be scary, and add a best practice note so that people "do the right thing"

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants