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

Poor deprecation notice for using classBinding in a template. #11263

Closed
rwjblue opened this issue May 24, 2015 · 1 comment · Fixed by #11271
Closed

Poor deprecation notice for using classBinding in a template. #11263

rwjblue opened this issue May 24, 2015 · 1 comment · Fixed by #11271
Labels
Milestone

Comments

@rwjblue
Copy link
Member

rwjblue commented May 24, 2015

DEPRECATION: You're using legacy class binding syntax: classBinding=":input type socialMediaValidClass" @ 4:60 in (inline). Please replace with class=""
        at /Users/rwjblue/src/ember-land/ember-canary/dist/ember-template-compiler.js:10393:26
        at each (/Users/rwjblue/src/ember-land/ember-canary/dist/ember-template-compiler.js:10460:7)
        at /Users/rwjblue/src/ember-land/ember-canary/dist/ember-template-compiler.js:10378:7
        at Walker.visit (/Users/rwjblue/src/ember-land/ember-canary/dist/ember-template-compiler.js:15645:7)
        at visitors.ElementNode (/Users/rwjblue/src/ember-land/ember-canary/dist/ember-template-compiler.js:15661:16)
        at Walker.children (/Users/rwjblue/src/ember-land/ember-canary/dist/ember-template-compiler.js:15678:7)

Needs to include moduleName and line number if possible, along with the corrected syntax.

Demo JSBin: http://emberjs.jsbin.com/rwjblue/542/edit?html,js

@rwjblue rwjblue added the Bug label May 24, 2015
@rwjblue rwjblue added this to the 1.13.0 milestone May 24, 2015
@ivpusic
Copy link

ivpusic commented May 24, 2015

How I can get rid of classBinding deprecation in this case?

{{input classBinding=":validate errors:invalid"}}

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

Successfully merging a pull request may close this issue.

2 participants