Skip to content

Commit 96498bf

Browse files
author
Marvin Tam
committed
Allow entering the minus sign first
1 parent a27889a commit 96498bf

File tree

4 files changed

+19
-7
lines changed

4 files changed

+19
-7
lines changed

angular-money-directive.js

+8-4
Original file line numberDiff line numberDiff line change
@@ -35,10 +35,14 @@ angular.module('fiestah.money', [])
3535
}
3636

3737
// Allow "-" inputs only when min < 0
38-
if (value.indexOf('-') === 0 && min >= 0) {
39-
value = null;
40-
ngModelCtrl.$setViewValue('');
41-
ngModelCtrl.$render();
38+
if (value.indexOf('-') === 0) {
39+
if (min >= 0) {
40+
value = null;
41+
ngModelCtrl.$setViewValue('');
42+
ngModelCtrl.$render();
43+
} else if (value === '-') {
44+
value = '';
45+
}
4246
}
4347

4448
var empty = ngModelCtrl.$isEmpty(value);

bower.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "angular-money-directive",
3-
"version": "1.0.0",
3+
"version": "1.0.1",
44
"main": "angular-money-directive.js",
55
"ignore": [
66
"**/.*",

package.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "angular-money-directive",
3-
"version": "1.0.0",
3+
"version": "1.0.1",
44
"description": "AngularJS directive to validate money inputs",
55
"dependencies": {},
66
"devDependencies": {

test/angular-money-directive.spec.js

+9-1
Original file line numberDiff line numberDiff line change
@@ -91,8 +91,16 @@ describe('angular-money-directive', function () {
9191
});
9292

9393
describe('when min < 0', function () {
94+
beforeEach(function () {
95+
setupDirective('min="-10"');
96+
});
97+
98+
it('allows the negative sign', function () {
99+
setValue('-');
100+
expect(scope.model.price).to.not.be.ok;
101+
expect(form.price.$valid).to.be.true;
102+
});
94103
it('allows negative values', function () {
95-
setupDirective('min="-10"')
96104
setValue('-5.4');
97105
expect(scope.model.price).to.equal(-5.4);
98106
expect(form.price.$valid).to.be.true;

0 commit comments

Comments
 (0)