fix(forms): use change event for select multiple (#9713)

robwormald · web-flow · commit 3cbded669488 · 2016-06-30T20:24:39.000-07:00
diff --git a/modules/@angular/common/src/forms-deprecated/directives/select_multiple_control_value_accessor.ts b/modules/@angular/common/src/forms-deprecated/directives/select_multiple_control_value_accessor.ts
@@ -47,7 +47,7 @@ abstract class HTMLCollection {
  */
 @Directive({
   selector: 'select[multiple][ngControl],select[multiple][ngFormControl],select[multiple][ngModel]',
-  host: {'(input)': 'onChange($event.target)', '(blur)': 'onTouched()'},
+  host: {'(change)': 'onChange($event.target)', '(blur)': 'onTouched()'},
   providers: [SELECT_MULTIPLE_VALUE_ACCESSOR]
 })
 export class SelectMultipleControlValueAccessor implements ControlValueAccessor {
@@ -182,4 +182,4 @@ export class NgSelectMultipleOption implements OnDestroy {
   }
 }
 
-export const SELECT_DIRECTIVES = [SelectMultipleControlValueAccessor, NgSelectMultipleOption];
+export const SELECT_DIRECTIVES = [SelectMultipleControlValueAccessor, NgSelectMultipleOption];
diff --git a/modules/@angular/forms/src/directives/select_multiple_control_value_accessor.ts b/modules/@angular/forms/src/directives/select_multiple_control_value_accessor.ts
@@ -50,7 +50,7 @@ abstract class HTMLCollection {
 @Directive({
   selector:
       'select[multiple][formControlName],select[multiple][formControl],select[multiple][ngModel]',
-  host: {'(input)': 'onChange($event.target)', '(blur)': 'onTouched()'},
+  host: {'(change)': 'onChange($event.target)', '(blur)': 'onTouched()'},
   providers: [SELECT_MULTIPLE_VALUE_ACCESSOR]
 })
 export class SelectMultipleControlValueAccessor implements ControlValueAccessor {
@@ -185,4 +185,4 @@ export class NgSelectMultipleOption implements OnDestroy {
   }
 }
 
-export const SELECT_DIRECTIVES = [SelectMultipleControlValueAccessor, NgSelectMultipleOption];
+export const SELECT_DIRECTIVES = [SelectMultipleControlValueAccessor, NgSelectMultipleOption];

Original file line number	Diff line number	Diff line change
`@@ -47,7 +47,7 @@ abstract class HTMLCollection {`
`47`	`47`	`*/`
`48`	`48`	`@Directive({`
`49`	`49`	`selector: 'select[multiple][ngControl],select[multiple][ngFormControl],select[multiple][ngModel]',`
`50`		`- host: {'(input)': 'onChange($event.target)', '(blur)': 'onTouched()'},`
	`50`	`+ host: {'(change)': 'onChange($event.target)', '(blur)': 'onTouched()'},`
`51`	`51`	`providers: [SELECT_MULTIPLE_VALUE_ACCESSOR]`
`52`	`52`	`})`
`53`	`53`	`export class SelectMultipleControlValueAccessor implements ControlValueAccessor {`
`@@ -182,4 +182,4 @@ export class NgSelectMultipleOption implements OnDestroy {`
`182`	`182`	`}`
`183`	`183`	`}`
`184`	`184`
`185`		`-export const SELECT_DIRECTIVES = [SelectMultipleControlValueAccessor, NgSelectMultipleOption];`
	`185`	`+export const SELECT_DIRECTIVES = [SelectMultipleControlValueAccessor, NgSelectMultipleOption];`
Original file line number	Diff line number	Diff line change
`@@ -50,7 +50,7 @@ abstract class HTMLCollection {`
`50`	`50`	`@Directive({`
`51`	`51`	`selector:`
`52`	`52`	`'select[multiple][formControlName],select[multiple][formControl],select[multiple][ngModel]',`
`53`		`- host: {'(input)': 'onChange($event.target)', '(blur)': 'onTouched()'},`
	`53`	`+ host: {'(change)': 'onChange($event.target)', '(blur)': 'onTouched()'},`
`54`	`54`	`providers: [SELECT_MULTIPLE_VALUE_ACCESSOR]`
`55`	`55`	`})`
`56`	`56`	`export class SelectMultipleControlValueAccessor implements ControlValueAccessor {`
`@@ -185,4 +185,4 @@ export class NgSelectMultipleOption implements OnDestroy {`
`185`	`185`	`}`
`186`	`186`	`}`
`187`	`187`
`188`		`-export const SELECT_DIRECTIVES = [SelectMultipleControlValueAccessor, NgSelectMultipleOption];`
	`188`	`+export const SELECT_DIRECTIVES = [SelectMultipleControlValueAccessor, NgSelectMultipleOption];`