refactor: CCollapse: remove toggler functionality

Author: woothu

src/components/Collapse/CCollapse.vue

@@ -15,8 +15,7 @@ const props = {
     default: 'ease-in-out'
   },
   show: Boolean,
-  navbar: Boolean,
-  toggler: String
+  navbar: Boolean
 }
 
 export default {
@@ -40,23 +39,11 @@ export default {
   },
   mounted () {
     this.$el.style.display = this.visible ?  '' : 'none'
-    this.$nextTick(() => {
-      this.el = document.getElementById(this.toggler)
-      if (this.el) {
-        this.el.addEventListener('click', this.collapse)
-      }
-    })
   },
   beforeDestroy () {
-    if (this.el) {
-      this.el.removeEventListener('click', this.collapse)
-    }
     clearTimeout(this.heightWatcher)
   },
   methods: {
-    collapse () {
-      this.visible = !this.visible
-    },
     collapseController (val) {
       if (this.collapsing === false) {
         val ? this.toggle(true) : this.toggle(false)

src/components/Collapse/test/CCollapse.spec.js

@@ -21,24 +21,12 @@ const togglerWrapper = mount(CCollapse, {
   propsData: {
     duration: 500,
     show: true,
-    navbar: true,
-    toggler: 'toggler_id'
+    navbar: true
   },
   slots: {
-    default: '<button id="toggler_id">button</button>'
+    default: 'Collapse content'
   }
 })
-// const App = Vue.extend({
-//   components: { CCollapse },
-//   render (h) {
-//     return h('div', {}, [
-//       h('CCollapse', { props: { toggler: 'toggler_id' }}, ['content']),
-//       h('button', { attrs: { id: 'toggler_id' }}, ['button']),
-//     ])
-//   }
-// })
-
-// const togglerWrapper = mount(App)
 
 describe(ComponentName, () => {
   it('has a name', () => {
@@ -76,15 +64,6 @@ describe(ComponentName, () => {
     }, 700)
     jest.runAllTimers()
   })
-  it('toggle on toggler click', () => {
-    togglerWrapper.find('button').trigger('click')
-    expect(togglerWrapper.vm.visible).toBe(false)
-  })
-  it('removes toggler event listener on destroy', () => {
-    const spy = jest.spyOn(togglerWrapper.vm.el, 'removeEventListener')
-    togglerWrapper.destroy()
-    expect(spy).toBeCalled()
-  })
   it('destroy with no errors', () => {
     try {
       customWrapper.destroy()

src/components/Collapse/test/__snapshots__/CCollapse.spec.js.snap

@@ -11,6 +11,7 @@ exports[`CCollapse renders correctly 2`] = `
 <div
   class="navbar-collapse"
   style="display: none;"
+  toggler="some_id"
 >
   CCollapse content
 </div>

src/components/index.d.ts

@@ -117,7 +117,6 @@ export declare class CCollapse extends Vue {
   transition: string
   show: boolean
   navbar: boolean
-  toggler: string
 }
 
 export declare class CDropdown extends Vue {