refactor toggleDropdown click handler (#992)

src/components/Select.vue

@@ -17,7 +17,7 @@
             <slot name="selected-option" v-bind="normalizeOptionForSlot(option)">
               {{ getOptionLabel(option) }}
             </slot>
-            <button v-if="multiple" :disabled="disabled" @click="deselect(option)" type="button" class="vs__deselect" aria-label="Deselect option">
+            <button v-if="multiple" :disabled="disabled" @click="deselect(option)" type="button" class="vs__deselect" aria-label="Deselect option" ref="deselectButtons">
               <component :is="childComponents.Deselect" />
             </button>
           </span>
@@ -36,6 +36,7 @@
           type="button"
           class="vs__clear"
           title="Clear selection"
+          ref="clearButton"
         >
           <component :is="childComponents.Deselect" />
         </button>
@@ -652,33 +653,23 @@
        * @param  {Event} e
        * @return {void}
        */
-      toggleDropdown (e) {
+      toggleDropdown ({target}) {
-        const target = e.target;
+        //  don't react to click on deselect/clear buttons,
-        const toggleTargets = [
+        //  they dropdown state will be set in their click handlers
-          this.$el,
+        const ignoredButtons = [
-          this.searchEl,
+          ...(this.$refs['deselectButtons'] || []),
-          this.$refs.toggle,
+          ...([this.$refs['clearButton']] || [])
-          this.$refs.actions,
-          this.$refs.selectedOptions,
         ];
 
-        if (typeof this.$refs.openIndicator !== 'undefined') {
+        if (ignoredButtons.some(ref => ref.contains(target) || ref === target)) {
-          toggleTargets.push(
+          return;
-            this.$refs.openIndicator.$el,
-            // the line below is a bit gross, but required to support IE11 without adding polyfills
-            ...Array.prototype.slice.call(this.$refs.openIndicator.$el.childNodes),
-          );
         }
 
-        if (toggleTargets.indexOf(target) > -1 || target.classList.contains('vs__selected')) {
+        if (this.open) {
-          if (this.open) {
+          this.searchEl.blur();
-            this.searchEl.blur(); // dropdown will close on blur
+        } else if (!this.disabled) {
-          } else {
+          this.open = true;
-            if (!this.disabled) {
+          this.searchEl.focus();
-              this.open = true;
-              this.searchEl.focus();
-            }
-          }
         }
       },
 

tests/unit/Slots.spec.js

@@ -10,32 +10,49 @@ describe('Scoped Slots', () => {
         },
       });
 
-    expect(Select.find({ ref: 'selectedOptions' }).text()).toEqual('one')
+    expect(Select.find({ref: 'selectedOptions'}).text()).toEqual('one');
   });
 
-  it('receives an option object to the selected-option slot', () => {
+  describe('Slot: selected-option', () => {
-    const Select = mountDefault(
+    it('receives an option object to the selected-option slot', () => {
-      {value: 'one'},
+      const Select = mountDefault(
-      {
+        {value: 'one'},
-        scopedSlots: {
+        {
-          'selected-option': `<span slot="selected-option" slot-scope="option">{{ option.label }}</span>`,
+          scopedSlots: {
-        },
+            'selected-option': `<span slot="selected-option" slot-scope="option">{{ option.label }}</span>`,
+          },
+        });
+
+      expect(Select.find('.vs__selected').text()).toEqual('one');
+    });
+
+    it('opens the dropdown when clicking an option in selected-option slot',
+      () => {
+        const Select = mountDefault(
+          {value: 'one'},
+          {
+            scopedSlots: {
+              'selected-option': `<span class="my-option" slot-scope="option">{{ option.label }}</span>`,
+            },
+          });
+
+        Select.find('.my-option').trigger('mousedown');
+        expect(Select.vm.open).toEqual(true);
       });
-
-    expect(Select.find('.vs__selected').text()).toEqual('one')
   });
 
-  it('receives an option object to the option slot in the dropdown menu', () => {
+  it('receives an option object to the option slot in the dropdown menu',
-    const Select = mountDefault(
+    () => {
-      {value: 'one'},
+      const Select = mountDefault(
-      {
+        {value: 'one'},
-        scopedSlots: {
+        {
-          'option': `<span slot="option" slot-scope="option">{{ option.label }}</span>`,
+          scopedSlots: {
-        },
+            'option': `<span slot="option" slot-scope="option">{{ option.label }}</span>`,
-      });
+          },
+        });
 
-    Select.vm.open = true;
+      Select.vm.open = true;
 
-    expect(Select.find({ref: 'dropdownMenu'}).text()).toEqual('onetwothree')
+      expect(Select.find({ref: 'dropdownMenu'}).text()).toEqual('onetwothree');
-  });
+    });
 });