Skip to content

Commit dd9eb82

Browse files
committed
Merge pull request seiyria#9 from yourowndisaster09/master
Require ngModel in directive so that it can listen to ngChange
2 parents b8dce7c + f4f2260 commit dd9eb82

File tree

1 file changed

+4
-5
lines changed

1 file changed

+4
-5
lines changed

slider.js

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -4,9 +4,8 @@ angular.module('ui.bootstrap-slider', [])
44
restrict: 'AE',
55
replace: true,
66
template: '<input type="text" />',
7-
link: function ($scope, element, attrs) {
8-
var model = $parse(attrs.ngModel);
9-
7+
require: 'ngModel',
8+
link: function ($scope, element, attrs, ngModelCtrl) {
109
$.fn.slider.Constructor.prototype.disable = function () {
1110
this.picker.off();
1211
}
@@ -58,10 +57,10 @@ angular.module('ui.bootstrap-slider', [])
5857
}
5958

6059
var slider = $(element[0]).slider(options);
61-
var updateEvent = attrs.updateEvent || 'slide';
60+
var updateEvent = attrs.updateEvent || 'slide';
6261

6362
slider.on(updateEvent, function(ev) {
64-
model.assign($scope, ev.value);
63+
ngModelCtrl.$setViewValue(ev.value);
6564
$timeout(function() {
6665
$scope.$apply();
6766
});

0 commit comments

Comments
 (0)