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

data-ng-model or x-data-model not recognized by the test gear #1020

Closed
zartc opened this issue Jun 6, 2012 · 3 comments
Closed

data-ng-model or x-data-model not recognized by the test gear #1020

zartc opened this issue Jun 6, 2012 · 3 comments

Comments

@zartc
Copy link

zartc commented Jun 6, 2012

If one change <ng-model> to <data-ng-model> or <x-data-model> (to please automatic inspector that flag the proprietary attribute as an error -- in Eclipse or IDEA) then tests fail!
Angular version: v1.0.0rc6

@mhevery
Copy link
Contributor

mhevery commented Jun 6, 2012

could you include a test which fails? as far as I know this works, so a repro is needed.

@zartc
Copy link
Author

zartc commented Jun 6, 2012

I discovered the problem when going through the tutorial:
http://docs.angularjs.org/tutorial/step_03
Simply change directives to the form and all tests
in test/e2e/scenarios.js fail.

I have put a comment on this issue on the bottom of the tutorial page.
regards.

On mercredi 6 juin 2012 19:03:17, Miško Hevery wrote:

could you include a test which fails? as far as I know this works, so a repro is needed.


Reply to this email directly or view it on GitHub:
#1020 (comment)

@nicjohnson
Copy link

Any plans to address this?

adam33 added a commit to adam33/angular.js that referenced this issue Oct 30, 2012
Prefixed attributes like data-ng-model and x-ng-model were not being
found by the Selector. It was only looking at ng: and ng- prefixed
attributes.
Added a few tests as well to ensure the aforementioned prefixed
attributes are being matched properly.

Closes angular#1020
IgorMinar pushed a commit to IgorMinar/angular.js that referenced this issue Oct 31, 2012
Prefixed attributes like data-ng-model and x-ng-model were not being
found by the Selector. It was only looking at ng: and ng- prefixed
attributes.
Added a few tests as well to ensure the aforementioned prefixed
attributes are being matched properly.

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

Successfully merging a pull request may close this issue.

4 participants