Skip to content
This repository has been archived by the owner on Feb 22, 2018. It is now read-only.

feat(ngModel): Treat the values of number and range inputs as numbers #527

Closed

Conversation

teropa
Copy link
Contributor

@teropa teropa commented Feb 8, 2014

Add a InputNumberLikeDirective that binds the model as a numeric
value, so that it can be bound to num types on scopes. Select the
directive for number and range input types.

Add a InputNumberLikeDirective that binds the model as a numeric
value, so that it can be bound to num types on scopes. Select the
directive for number and range input types.
@teropa
Copy link
Contributor Author

teropa commented Feb 8, 2014

I was getting type errors when trying to bind a numeric value on a controller to an input type="number". It was coming back as a String, and that was because it was handled by InputTextLikeDirective. Also, input type="range" was not being handled at all.

This commit adds a new InputNumberLikeDirective for those input types.

@mhevery mhevery self-assigned this Feb 11, 2014
mhevery pushed a commit to mhevery/angular.dart that referenced this pull request Feb 11, 2014
Add a InputNumberLikeDirective that binds the model as a numeric
value, so that it can be bound to num types on scopes. Select the
directive for number and range input types.

Closes dart-archive#527
@mhevery mhevery closed this in e703bd1 Feb 12, 2014
mhevery pushed a commit to mhevery/angular.dart that referenced this pull request Feb 15, 2014
Updating analyzer version to 0.11

Updated analyzer has a number of convenience methods that make it easier with code gen. No code changes needed.

Closes dart-archive#527
mhevery pushed a commit to mhevery/angular.dart that referenced this pull request Feb 15, 2014
Updating analyzer version to 0.11

Updated analyzer has a number of convenience methods that make it easier with code gen. No code changes needed.

Closes dart-archive#527
mhevery pushed a commit to mhevery/angular.dart that referenced this pull request Feb 19, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Development

Successfully merging this pull request may close these issues.

2 participants