Skip to content
This repository has been archived by the owner on Mar 12, 2020. It is now read-only.

Non type:submit input/button #165

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

alexalouit
Copy link

Add trigger for non type:submit input/button (useful for multi-step forms).

Add trigger for non type:submit input/button
@ghost
Copy link

ghost commented Aug 21, 2015

This commit sadly conflicts with the current branch (breaks validator), but I would totally be happy about a feature like that.

Any insights on how to achieve that?

Thanks

Conflicts:
	dist/validator.js
	dist/validator.min.js
	js/validator.js
Add trigger for non type:submit input/button
@alexalouit
Copy link
Author

This is fixed, validator don't report error for me.

$ grunt test
Running "jshint:gruntfile" (jshint) task
>> 1 file lint free.

Running "jshint:src" (jshint) task
>> 1 file lint free.

Running "jshint:test" (jshint) task
>> 2 files lint free.

Running "qunit:files" (qunit) task
Testing js/tests/index.html         

Starting test suite
================================================

✔ All tests passed in "validator" module
OK
>> 51 assertions passed (1465ms)

Done, without errors.
$ grunt uglify
Running "uglify:min" (uglify) task
File "dist/validator.min.js" created.
Original: 10710 bytes.
Minified: 6159 bytes.

Done, without errors.

@uniacid
Copy link

uniacid commented Dec 22, 2016

FYI I updated this based on the latest version.

uniacid#2

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants