Reset callback

[lucax88x]

Added a reset callback with documentation with also some gernal documentation on general callbacks which weren't present in the API documentation

[hueitan]

In what case people need resetCallback?

We can do it by ng-click="reset(Form)"

$scope.reset = function (form) {
    $validationProvider.reset(form);
    // callback here
}

[hueitan]

Also this resetCallback will works in every inputs.

[lucax88x]

it's used to perform graphical variations on html elements when a reset is performed.

Are you sure about your last statement?

[hueitan]

https://github.com/lucax88x/angular-validation/blob/blessed/src/validator.directive.js#L229

This works on every on - reset event.

<form name="Form1">
 <input ... />
 <input ... />
 <input ... />
 <button ng-click="reset(Form1)"></button>
</form>

then resetCallback will execute 3 times.

[lucax88x]

yeah, this is correct, as you can see, the callbacks need a element parameter.

for example:

this.$validation.validCallback = function(element) {
   $(element).removeClass('has-error').addClass('has-success-tick');
};
this.$validation.invalidCallback = function(element) {
   $(element).removeClass('has-success-tick').addClass('has-error');
};
this.$validation.resetCallback = function(element) {
   $(element).removeClass('has-error').removeClass('has-success-tick');
};

[hueitan]

Oh I got your point.

resetCallback has the same purpose as validCallback, invalidCallback

[lucax88x]

yes, but used when a reset is performed

[hueitan]

Thanks @lucax88x This looks reasonable. 🍻

@Nazanin1369 @lucax88x

We now have a lot of callback within every section.

maybe it's time we need to define the callback SPEC it is easy to confuse which is global callback and each callback.

[Nazanin1369]

Agree.
On Dec 15, 2015 7:38 PM, "Huei Tan" [email protected] wrote:

Thanks @lucax88x https://github.com/lucax88x This looks reasonable. [image:
🍻]

@Nazanin1369 https://github.com/Nazanin1369 @lucax88x
https://github.com/lucax88x

We now have a lot of callback within every section.

maybe it's time we need to define the callback SPEC.

—
Reply to this email directly or view it on GitHub
#172 (comment)
.

[hueitan]

@lucax88x I'm going to merge this feature. Do you want to create a test case for this or I can do it.

[lucax88x]

@huei90 what if you create a spec for valid callback & invalid callback and I add one for reset?

[hueitan]

Ok thanks @lucax88x I will create another Issue for the spec.

and I merge the reset feature now.