angular-autodisable

angular-autodisable copied to clipboard

Allows multiple buttons with `type="submit"`. Fixes `$ is not defined` error with webpack etc. by using `angular.element` and `$document.find`.

kuhnroyal

handling ng-disabled

1

One might want to disable / enable form inputs depending on some state. Currently, ng-autodisable and ng-disabled don't work well together, and we could have the situation where ng-autodisable has...

lbesson

not work with ngTouch module.

1

cause ngTouch module's ngClick directive using touch event to trigger click event. and disabled element can trigger touch event. so this won't work. I can send a PR to fix...

lili21

Not working with ng-submit

1

Here is my jade template ``` form(name='signInForm' ng-submit="signIn()" ng-autodisable) input(type="email" ng-model="email" placeholder='Email' ng-required='true') input(type="password" ng-model="password" placeholder='Password' ng-required='true') button(type="submit" ng-disabled="signInForm.$invalid") Sign In ```

dflourusso

Added support for using require('angular-autodisable'). Added support for npm run build. Added dev dependencies. Inspired by the pull request from @lili21

stnor

jqLite's find() is limited to lookups by tag name

1

therefore, this snippet never finds any buttons to disable ``` handler = handlerInstance(element.find('button[type=submit]'), SUBMIT_EVENT, getLoadingClass(attrs), getCallbacks(attrs[SUBMIT_ATTR])); ``` maybe you could use something like `element[0].querySelectorAll('button[type=submit]')` instead?

juliekoubova

The prefix ng-\* in directive might conflict with future native directives.

floftar

autodisable form submit button

4

I'm trying to use this directive on a form with a `button[type="submit"]`. The problem is... The child button is generated in a directive itself. (`Submit

jarrodpayne