Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update upstream #88

Merged
merged 1 commit into from
Nov 22, 2017
Merged

Commits on Nov 21, 2017

  1. feat(ngModelOptions): add debounce catch-all + allow debouncing 'defa…

    …ult' only
    
    Closes #15411
    Closes #16335
    
    BREAKING CHANGE:
    
    the 'default' key in 'debounce' now only debounces the default event, i.e. the event
    that is added as an update trigger by the different input directives automatically.
    
    Previously, it also applied to other update triggers defined in 'updateOn' that
    did not have a corresponding key in the 'debounce'.
    
    This behavior is now supported via a special wildcard / catch-all key: '*'.
    
    See the following example:
    
    Pre-1.7:
    'mouseup' is also debounced by 500 milliseconds because 'default' is applied:
    ```
    ng-model-options="{
      updateOn: 'default blur mouseup',
      debounce: { 'default': 500, 'blur': 0 }
    }
    ```
    
    1.7:
    The pre-1.7 behavior can be re-created by setting '*' as a catch-all debounce value:
    ```
    ng-model-options="{
      updateOn: 'default blur mouseup',
      debounce: { '*': 500, 'blur': 0 }
    }
    ```
    
    In contrast, when only 'default' is used, 'blur' and 'mouseup' are not debounced:
    ```
    ng-model-options="{
      updateOn: 'default blur mouseup',
      debounce: { 'default': 500 }
    }
    ```
    Narretz committed Nov 21, 2017
    Configuration menu
    Copy the full SHA
    55ba449 View commit details
    Browse the repository at this point in the history