From 10fa4eb392c20ffa6269333eb84ce46f789d574e Mon Sep 17 00:00:00 2001
From: Troy Morehouse <troymore@nbnet.nb.ca>
Date: Mon, 11 Sep 2017 19:37:29 -0300
Subject: [PATCH 1/6] Update toggle.js

---
 lib/directives/toggle.js | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/lib/directives/toggle.js b/lib/directives/toggle.js
index 5b9a19ae03d..bfdc8f17576 100755
--- a/lib/directives/toggle.js
+++ b/lib/directives/toggle.js
@@ -6,9 +6,10 @@ const listen_types = {click: true};
 // Property key for handler
 const BVT = '__BV_toggle__';
 
-// Event nmmes
-const EVENT_TOGGLE = 'collapse::toggle';
-const EVENT_STATE = 'collapse::toggle::state';
+// Emitted Contreol Event for collapse (emitted to collapse)
+const EVENT_TOGGLE = 'bv::toggle::collapse';
+// Listen to event for toggle state update (Emited by collapse)
+const EVENT_STATE = 'bv::collapse::state';
 
 export default {
 

From 7bb18586001629b9ad482547747c3d4e7a33b0c3 Mon Sep 17 00:00:00 2001
From: Troy Morehouse <troymore@nbnet.nb.ca>
Date: Mon, 11 Sep 2017 20:00:04 -0300
Subject: [PATCH 2/6] [collapse] use new namespaced root events

---
 lib/components/collapse.vue | 25 +++++++++++++++++++------
 1 file changed, 19 insertions(+), 6 deletions(-)

diff --git a/lib/components/collapse.vue b/lib/components/collapse.vue
index 1e62917c489..0a1741f5ca0 100755
--- a/lib/components/collapse.vue
+++ b/lib/components/collapse.vue
@@ -22,6 +22,14 @@
 
 <script>
     import { listenOnRootMixin } from '../mixins';
+    import { hasClass } from '../utils/dom';
+
+    // Events we emit on $root
+    const EVENT_STATE = 'bv:collapse::state';
+    const EVENT_ACCORDION = 'bv::collapse::accordion';
+
+    // Events we listen to on $root
+    const EVENT_TOGGLE = 'bv::toggle::collapse';
 
     export default {
         mixins: [listenOnRootMixin],
@@ -87,6 +95,7 @@
                 this.reflow(el);
                 el.style.height = el.scrollHeight + 'px';
                 this.transitioning = true;
+                // This should be moved out so we can add cancellable events
                 this.$emit('show');
             },
             onAfterEnter(el) {
@@ -101,6 +110,7 @@
                 this.reflow(el);
                 this.transitioning = true;
                 el.style.height = 0;
+                // This should be moved out so we can add cancellable events
                 this.$emit('hide');
             },
             onAfterLeave(el) {
@@ -114,10 +124,11 @@
             },
             emitState() {
                 this.$emit('input', this.show);
-                this.$root.$emit('collapse::toggle::state', this.id, this.show);
+                // Let v-b-toggle know the state of this collapse
+                this.$root.$emit(EVENT_STATE, this.id, this.show);
                 if (this.accordion && this.show) {
                     // Tell the other collapses in this accordion to close
-                    this.$root.$emit('accordion::toggle', this.id, this.accordion);
+                    this.$root.$emit(EVENT_ACCORDION, this.id, this.accordion);
                 }
             },
             clickHandler(evt) {
@@ -126,7 +137,7 @@
                 if (!this.isNav || !el || getComputedStyle(this.$el).display !== 'block') {
                     return;
                 }
-                if (el.classList.contains('nav-link') || el.classList.contains('dropdown-item')) {
+                if (hasClass(el, 'nav-link') || hasClass(el, 'dropdown-item')) {
                     this.show = false;
                 }
             },
@@ -158,8 +169,10 @@
             },
         },
         created() {
-            this.listenOnRoot('collapse::toggle', this.handleToggleEvt);
-            this.listenOnRoot('accordion::toggle', this.handleAccordionEvt);
+            // Listen for toggle events to open/close us
+            this.listenOnRoot(EVENT_TOGGLE, this.handleToggleEvt);
+            // Listen to otehr collapses for accordion events
+            this.listenOnRoot(EVENT_ACCORDION, this.handleAccordionEvt);
         },
         mounted() {
             if (this.isNav && typeof document !== 'undefined') {
@@ -170,7 +183,7 @@
             }
             this.emitState();
         },
-        destroyed() {
+        beforeDestroy() {
             if (this.isNav && typeof document !== 'undefined') {
                 window.removeEventListener('resize', this.handleResize, false);
                 window.removeEventListener('orientationchange', this.handleResize, false);

From 7032ceaafb6478c10fe0b1df5b01aa01d70ef1e9 Mon Sep 17 00:00:00 2001
From: Troy Morehouse <troymore@nbnet.nb.ca>
Date: Mon, 11 Sep 2017 20:08:01 -0300
Subject: [PATCH 3/6] [v-b-toggle] use dom utils

---
 lib/directives/toggle.js | 25 +++++++++++++++----------
 1 file changed, 15 insertions(+), 10 deletions(-)

diff --git a/lib/directives/toggle.js b/lib/directives/toggle.js
index bfdc8f17576..1bd2466a4b4 100755
--- a/lib/directives/toggle.js
+++ b/lib/directives/toggle.js
@@ -1,14 +1,19 @@
+import target from './_target';
+import { setAttr, addClass, removeClass } from '../utils/dom';
+
+// Are we client side?
 const inBrowser = typeof window !== 'undefined';
 
-import target from './_target';
+// target listen types
 const listen_types = {click: true};
 
-// Property key for handler
+// Property key for handler storage
 const BVT = '__BV_toggle__';
 
-// Emitted Contreol Event for collapse (emitted to collapse)
+// Emitted Control Event for collapse (emitted to collapse)
 const EVENT_TOGGLE = 'bv::toggle::collapse';
-// Listen to event for toggle state update (Emited by collapse)
+
+// Listen to Event for toggle state update (Emited by collapse)
 const EVENT_STATE = 'bv::collapse::state';
 
 export default {
@@ -23,19 +28,19 @@ export default {
 
         if (inBrowser && vnode.context && targets.length > 0) {
             // Add aria attributes to element
-            el.setAttribute('aria-controls', targets.join(' '));
-            el.setAttribute('aria-expanded', 'false');
+            setAttr(el, 'aria-controls', targets.join(' '));
+            setAttr(el, 'aria-expanded', 'false');
 
             // Toggle state hadnler, stored on element
             el[BVT] = function toggleDirectiveHandler(id, state) {
                 if (targets.indexOf(id) !== -1) {
                     // Set aria-expanded state
-                    el.setAttribute('aria-expanded', state ? 'true' : 'false');
-                    // Set 'collapsed' class state
+                    setAttr(el, 'aria-expanded', state ? 'true' : 'false');
+                    // Set/Clear 'collapsed' class state
                     if (state) {
-                        el.classList.remove('collapsed');
+                        removeClass(el, 'collapsed');
                     } else {
-                        el.classList.add('collapsed');
+                        addClass(el, 'collapsed');
                     }
                 }
             };

From e222f2dbd47d48c09c6741ce28adf0afe4f91c97 Mon Sep 17 00:00:00 2001
From: Troy Morehouse <troymore@nbnet.nb.ca>
Date: Mon, 11 Sep 2017 20:16:14 -0300
Subject: [PATCH 4/6] Update collapse.vue

---
 lib/components/collapse.vue | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/components/collapse.vue b/lib/components/collapse.vue
index 0a1741f5ca0..992e8351123 100755
--- a/lib/components/collapse.vue
+++ b/lib/components/collapse.vue
@@ -25,7 +25,7 @@
     import { hasClass } from '../utils/dom';
 
     // Events we emit on $root
-    const EVENT_STATE = 'bv:collapse::state';
+    const EVENT_STATE = 'bv::collapse::state';
     const EVENT_ACCORDION = 'bv::collapse::accordion';
 
     // Events we listen to on $root

From 550bc2b0f8706d7c890f149ee54f68bf65b18852 Mon Sep 17 00:00:00 2001
From: Troy Morehouse <troymore@nbnet.nb.ca>
Date: Mon, 11 Sep 2017 20:20:34 -0300
Subject: [PATCH 5/6] Update meta.json

---
 docs/components/collapse/meta.json | 18 ++----------------
 1 file changed, 2 insertions(+), 16 deletions(-)

diff --git a/docs/components/collapse/meta.json b/docs/components/collapse/meta.json
index 0b7c336151c..2e663de64fd 100755
--- a/docs/components/collapse/meta.json
+++ b/docs/components/collapse/meta.json
@@ -19,28 +19,14 @@
       "description": "emitted when collaspe has finished closing"
     },
     {
-      "event": "collapse::toggle",
-      "description": "toggles visible state of collaspe when this event emits on $root",
+      "event": "bv::toggle::collapse",
+      "description": "toggles visible state of collaspe when this event is emitted on $root",
       "args": [
         {
           "arg": "id",
           "description": "collapse id to toggle"
         }
       ]
-    },
-    {
-      "event": "accordion::toggle",
-      "description": "toggles visible state of accordion when this event emits on $root",
-      "args": [
-        {
-          "arg": "id",
-          "description": "collapse id that should be open"
-        },
-        {
-          "arg": "accordion",
-          "description": "accordion group identifier"
-        }
-      ]
     }
   ]
 }

From 5599d944a70c7f5b32edc580d89ccad1c23fc884 Mon Sep 17 00:00:00 2001
From: Troy Morehouse <troymore@nbnet.nb.ca>
Date: Mon, 11 Sep 2017 20:24:44 -0300
Subject: [PATCH 6/6] Update README.md

---
 docs/components/collapse/README.md | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/docs/components/collapse/README.md b/docs/components/collapse/README.md
index 9ef9289a5cf..bd7e134aed6 100755
--- a/docs/components/collapse/README.md
+++ b/docs/components/collapse/README.md
@@ -43,7 +43,7 @@ Other elements can easily toggle `<b-collapse>` components using the `v-b-toggle
 <!-- collapse-2.vue -->
 ```
 
-### Initial visibility (start expanded)
+## Initial visibility (start expanded)
 To make the `<b-collapse>` show initially, set the `visible` prop:
 
 ```html
@@ -59,7 +59,7 @@ To make the `<b-collapse>` show initially, set the `visible` prop:
 <!-- collapse-3.vue -->
 ```
 
-### `v-model` support
+## `v-model` support
 The component's collapsed (visible) state can also be set with `v-model` which
 binds internally to the `visible` prop.
 
@@ -95,7 +95,7 @@ export default {
 <-- collapse-4.vue -->
 ```
 
-### Trigger multiple collapse elements
+## Trigger multiple collapse elements
 You can even collapse multiple `<b-collapse>` components via a single `v-b-toggle` by 
 providing multiple target IDs using modifers: