Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rename LinkView to LinkComponent and deprecate the former #11394

Merged
merged 3 commits into from
Jun 10, 2015

Conversation

cibernox
Copy link
Contributor

Since Views are being deprecated and Ember.LinkView actually extends Ember.Component since early March.

Ember.View is still available but throws a deprecation warning when used.

@rwjblue
Copy link
Member

rwjblue commented Jun 10, 2015

Thank you for tackling, this is a great start!

I think we need to ensure that calling Ember.LinkView.reopen (which is mentioned in the guides and API docs) also triggers a deprecation.

@cibernox
Copy link
Contributor Author

Added deprecation on reopen too (not sure if there is a more idiomatic way of doing that)

@mixonic
Copy link
Member

mixonic commented Jun 10, 2015

@cibernox a followup that add a URL and entry in the deprecation guide would be great, but I'll do it this AM if you don't have time. We will be looking at this stuff all day anyway.

Thank you!

mixonic added a commit that referenced this pull request Jun 10, 2015
…nent

Rename LinkView to LinkComponent and deprecate the former
@mixonic mixonic merged commit 2aefbac into emberjs:master Jun 10, 2015
@cibernox cibernox deleted the rename_link_view_to_link_comonent branch June 10, 2015 13:23
@mixonic
Copy link
Member

mixonic commented Jun 10, 2015

@rwjblue This should go into [BUGFIX beta]. Sorry, will try to be more strict about this.

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

Successfully merging this pull request may close these issues.

3 participants