Reputation: 671
I'm learning angularjs and I want to be able let the user enter many inputs. When these inputs are entered the list
array elements should change accordingly. I wanted to try using ngRepeat directive but I read that since it creates a new scope I cannot databind:
<div ng-repeat="item in list">
<label>Input {{$index+1}}:</label>
<input ng-model="item" type="text"/>
</div>
I was wondering if I should be using a custom directive to do this or approach it differently.
Upvotes: 31
Views: 58725
Reputation: 181
I found an interesting way of doing this, and it's allowing me to work on an array of primitives.
I'm using AngularJS 1.2.1, which is the only version I can make this work in.
HTML:
<div ng-repeat="item in list">
<label>Input {{$index+1}}:</label>
<input ng-model="item" type="text" ng-blur="editItem($index, item)"/>
</div>
JavaScript:
$scope.editItem = function(idx, eItem) {
$scope.list[idx] = eItem;
};
Link: http://jsfiddle.net/bxD2P/10/ (thank you to Gloopy for the starter Fiddle)
I'm sure there are easy ways of poking holes in how this works, and I would love to hear them. It would allow me to shore up my code.
Upvotes: 3
Reputation: 12231
Consider using the ngList
directive https://docs.angularjs.org/api/ng/directive/ngList
Upvotes: 1
Reputation: 1319
Here's a way to do it. I used textareas and a different structure to my repeaters, but the main concept is:
It is essentially, faking the binding.
Working fiddle - http://jsfiddle.net/VvnWY/4/
The html:
<script type="text/ng-template" id="textareas.html">
<textarea ng-if="strings" ng-repeat="str in strings" ng-blur="blur( $event, $index )">{{strings[$index]}}</textarea>
</script>
<div ng-controller="MyCtrl">
Here's a few strings: <br />
<div ng-repeat="str in strings">{{strings[$index]}}</div>
Here's the strings as editable (twice so that you can see the updates from a model change): <br />
<form-textareas strings="strings"></form-textareas>
<form-textareas strings="strings"></form-textareas>
</div>
The JS:
var myApp = angular.module('myApp',[]);
angular.module('myApp', [])
.controller('MyCtrl', ['$scope', function($scope) {
$scope.strings = [ "foo", "bar", "cow" ];
}])
.directive('formTextareas', function() {
return {
restrict: "E",
scope: {
strings: '='
},
templateUrl: "textareas.html",
link: function( $scope ){
$scope.blur = function( $event, $index ){
$scope.strings[ $index ] = $event.currentTarget.value;
};
}
};
})
;
Upvotes: 1
Reputation: 364697
The reason that databinding to a primitive "item" doesn't work is because of the way ng-repeat creates the child scopes for each item. For each item, ng-repeat has the new child scope prototypically inherit from the parent scope (see dashed lines in picture below), and then it assigns the item's value to a new property on the child scope (red items in picture below). The name of the new property is the loop variable's name. From the ng-repeat source code:
childScope = scope.$new();
...
childScope[valueIdent] = value;
If item is a primitive, the new child scope property is essentially assigned a copy of the primitive's value. This child scope property is not visible to the parent scope, and changes you make to the input field are stored in this child scope property. E.g., suppose we have in the parent scope
$scope.list = [ 'value 1', 'value 2', 'value 3' ];
And in the HTML:
<div ng-repeat="item in list">
Then, the first child scope would have the following item
property, with a primitive value (value 1
):
item: "value 1"
Because of the ng-model databinding, changes you make to the form's input field are stored in that child scope property.
You can verify this by logging the child scope to the console. Add to your HTML, inside the ng-repeat:
<a ng-click="showScope($event)">show scope</a>
Add to your controller:
$scope.showScope = function(e) {
console.log(angular.element(e.srcElement).scope());
}
childScope[valueIdent] = value;
results in the item property's value being set to a reference to one of the array objects (not a copy).
Using the showScope() technique, you'll see that the child scope item
property's value references one of the array objects -- it is no longer a primitive value.
See also don't bind to primitives in ng-repeat child scopes and
What are the nuances of scope prototypal / prototypical inheritance in AngularJS? (which contains pictures of what the scopes look like when using ng-repeat).
Upvotes: 88
Reputation: 37785
You'll have better luck if your list
is an array of objects (as opposed to an array of primitives). This works fine even though a new scope is created with ng-repeat
:
<div ng-repeat="item in list">
<label>Input {{$index+1}}:</label>
<input ng-model="item.value" type="text"/>
</div>
with a controller of:
function TestController($scope) {
$scope.list = [ { value: 'value 1' }, { value: 'value 2' }, { value: 'value 3' } ];
}
See this fiddle as an example.
On the other hand if you are trying to bind to an array of strings the new scope will cause a problem as the values you are modifying will not be tied to the original array string primitives (as in this fiddle example).
Upvotes: 37