fix: CSelect: change input event to change, due to IE/edge support lack

woothu · woothu · commit ef84a23ca4a5 · 2020-05-28T08:10:33.000+02:00
diff --git a/src/components/form/CSelect.vue b/src/components/form/CSelect.vue
@@ -19,7 +19,7 @@
         v-on="$listeners"
         :id="safeId"
         :class="inputClasses"
-        @input="onSelect($event)"
+        @change="onSelect($event)"
       >
         <option 
           v-if="placeholder" 
diff --git a/src/components/form/tests/CSelect.spec.js b/src/components/form/tests/CSelect.spec.js
@@ -68,17 +68,17 @@ describe(ComponentName, () => {
   it('emit update event', () => {
     const select = customWrapper.find('select')
     select.element.value = 'Option 2'
-    select.trigger('input')
+    select.trigger('change')
     expect(customWrapper.emitted()['update:value']).toBeTruthy()
     const select2 = customSimpleWrapper.find('select')
     select2.element.value = 'Option 3'
-    select2.trigger('input')
+    select2.trigger('change')
     expect(customSimpleWrapper.emitted()['update:value']).toBeTruthy()
   })
   it('not emit update event on multiple select', () => {
     const select = wrapperMultiple.find('select')
     select.element.value = 'Option 2'
-    select.trigger('input')
+    select.trigger('change')
     expect(wrapperMultiple.emitted()['update:value']).not.toBeTruthy()
   })
 })
