From 1b90415b716045ce6b975bcaca7c4f8fd82944f6 Mon Sep 17 00:00:00 2001
From: xidedix <xidedix@users.noreply.github.com>
Date: Fri, 5 Oct 2018 16:42:45 +0200
Subject: [PATCH 1/6] fix(Aside): deprecate `hidden` prop,  tune
 `displayBreakpoint` behaviour

---
 src/Aside.js        | 20 +++++---------------
 src/Aside.md        |  6 +++---
 tests/Aside.test.js |  9 ++-------
 3 files changed, 10 insertions(+), 25 deletions(-)

diff --git a/src/Aside.js b/src/Aside.js
index 669ae98b..aba1295e 100644
--- a/src/Aside.js
+++ b/src/Aside.js
@@ -1,14 +1,14 @@
 import React, { Component } from 'react';
 import PropTypes from 'prop-types';
 import classNames from 'classnames';
-import { asideMenuCssClasses } from './Shared';
+import { asideMenuCssClasses, checkBreakpoint, validBreakpoints } from './Shared';
+import toggleClasses from './Shared/toggle-classes';
 
 const propTypes = {
   children: PropTypes.node,
   className: PropTypes.string,
   display: PropTypes.string,
   fixed: PropTypes.bool,
-  hidden: PropTypes.bool,
   isOpen: PropTypes.bool,
   offCanvas: PropTypes.bool,
   tag: PropTypes.oneOfType([PropTypes.func, PropTypes.string])
@@ -18,7 +18,6 @@ const defaultProps = {
   tag: 'aside',
   display: '',
   fixed: false,
-  hidden: false,
   isOpen: false,
   offCanvas: true
 };
@@ -28,22 +27,16 @@ class AppAside extends Component {
     super(props);
 
     this.isFixed = this.isFixed.bind(this);
-    this.isHidden = this.isHidden.bind(this);
     this.isOffCanvas = this.isOffCanvas.bind(this);
     this.displayBreakpoint = this.displayBreakpoint.bind(this);
   }
 
   componentDidMount() {
     this.isFixed(this.props.fixed);
-    this.isHidden(this.props.hidden);
     this.isOffCanvas(this.props.offCanvas);
     this.displayBreakpoint(this.props.display);
   }
 
-  isHidden(hidden) {
-    if (hidden) { document.body.classList.add('aside-menu-hidden'); }
-  }
-
   isFixed(fixed) {
     if (fixed) { document.body.classList.add('aside-menu-fixed'); }
   }
@@ -53,12 +46,10 @@ class AppAside extends Component {
   }
 
   displayBreakpoint(display) {
-    const cssTemplate = `aside-menu-${display}-show`;
-    let [cssClass] = asideMenuCssClasses[0];
-    if (display && asideMenuCssClasses.indexOf(cssTemplate) > -1) {
-      cssClass = cssTemplate;
+    if (display && checkBreakpoint(display, validBreakpoints)) {
+      const cssClass = `aside-menu-${display}-show`
+      toggleClasses(cssClass, asideMenuCssClasses, true)
     }
-    document.body.classList.add(cssClass);
   }
 
   render() {
@@ -66,7 +57,6 @@ class AppAside extends Component {
 
     delete attributes.display
     delete attributes.fixed
-    delete attributes.hidden
     delete attributes.offCanvas
     delete attributes.isOpen
 
diff --git a/src/Aside.md b/src/Aside.md
index d8a78d1d..7f8fa2a4 100644
--- a/src/Aside.md
+++ b/src/Aside.md
@@ -2,11 +2,11 @@
 
 | prop | default |
 | ------- | ---------
-| children  | 
+| children  |
 | className | `aside-menu`
-| display   | `sm, md, lg, xl, ""` 
+| display   | `sm, md, lg, xl, ""`
 | fixed     | `false`
-| hidden    | `false`
+| hidden    | `false` *deprecated* in v2 as conflicting with HTML5 `hidden` attribute
 | isOpen    | `false`
 | offCanvas | `true`
 | tag       | `aside`
diff --git a/tests/Aside.test.js b/tests/Aside.test.js
index 15e32696..b7f3d226 100644
--- a/tests/Aside.test.js
+++ b/tests/Aside.test.js
@@ -12,20 +12,15 @@ configure({ adapter: new Adapter() });
 
 describe('AppAside', () => {
   it('renders aside with class="aside-menu"', () => {
-    expect(render(<AppAside fixed hidden offCanvas display="lg">aside</AppAside>))
+    expect(render(<AppAside fixed offCanvas display="lg">aside</AppAside>))
     .toContain('<aside class="aside-menu">aside</aside>')
   });
   it('calls componentDidMount', () => {
     spy(AppAside.prototype, 'componentDidMount');
 
-    const wrapper = mount(<AppAside fixed hidden display="lg" />);
+    const wrapper = mount(<AppAside fixed />);
     expect(AppAside.prototype.componentDidMount.calledOnce).toEqual(true);
   });
-  it('should call isHidden()', () => {
-    const isHidden = spy(AppAside.prototype, 'isHidden');
-    shallow(<AppAside />);
-    expect(isHidden.called).toBe(true);
-  });
   it('should call isOffCanvas()', () => {
     const isOffCanvas = spy(AppAside.prototype, 'isOffCanvas');
     shallow(<AppAside offCanvas={false}/>);

From aca561b3aa1704ce483c516dcbe5e39aa317450c Mon Sep 17 00:00:00 2001
From: xidedix <xidedix@users.noreply.github.com>
Date: Fri, 5 Oct 2018 16:48:29 +0200
Subject: [PATCH 2/6] refactor(AsideToggler): extract `toggle` method, add
 missing `defaultOpen` prop

---
 src/AsideToggler.js | 35 ++++++++++++++++++++---------------
 src/AsideToggler.md |  1 +
 2 files changed, 21 insertions(+), 15 deletions(-)

diff --git a/src/AsideToggler.js b/src/AsideToggler.js
index 28c7736c..38f4ffc3 100644
--- a/src/AsideToggler.js
+++ b/src/AsideToggler.js
@@ -1,12 +1,13 @@
 import React, { Component } from 'react';
 import PropTypes from 'prop-types';
 import classNames from 'classnames';
-import { asideMenuCssClasses } from './Shared/index';
+import { asideMenuCssClasses, validBreakpoints, checkBreakpoint } from './Shared/index';
 import toggleClasses from './Shared/toggle-classes';
 
 const propTypes = {
   children: PropTypes.node,
   className: PropTypes.string,
+  defaultOpen: PropTypes.bool,
   display: PropTypes.any,
   mobile: PropTypes.bool,
   tag: PropTypes.oneOfType([PropTypes.func, PropTypes.string]),
@@ -14,6 +15,7 @@ const propTypes = {
 };
 
 const defaultProps = {
+  defaultOpen: false,
   display: 'lg',
   mobile: false,
   tag: 'button',
@@ -28,27 +30,30 @@ class AppAsideToggler extends Component {
     this.state = {};
   }
 
-  asideToggle(e) {
-    e.preventDefault();
+  componentDidMount() {
+    this.toggle(this.props.defaultOpen)
+  }
 
-    if (this.props.mobile) {
-      document.body.classList.toggle('aside-menu-show');
-    } else {
-      const display = this.props.display;
-      const cssTemplate = `aside-menu-${display}-show`;
-      let [cssClass] = asideMenuCssClasses[0];
-      if (display && asideMenuCssClasses.indexOf(cssTemplate) > -1) {
-        cssClass = cssTemplate;
-      }
-      toggleClasses(cssClass, asideMenuCssClasses);
+  toggle (force) {
+    const [display, mobile] = [this.props.display, this.props.mobile];
+    let cssClass = asideMenuCssClasses[0];
+    if (!mobile && display && checkBreakpoint(display, validBreakpoints)) {
+      cssClass = `aside-menu-${display}-show`
     }
+    toggleClasses(cssClass, asideMenuCssClasses, force)
+  }
+
+  asideToggle(e) {
+    e.preventDefault();
+    this.toggle()
   }
 
   render() {
     const { className, children, type, tag: Tag, ...attributes } = this.props;
 
-    delete attributes.display
-    delete attributes.mobile
+    delete attributes.defaultOpen;
+    delete attributes.display;
+    delete attributes.mobile;
 
     const classes = classNames(className, 'navbar-toggler');
 
diff --git a/src/AsideToggler.md b/src/AsideToggler.md
index b342a5d9..d185c64c 100644
--- a/src/AsideToggler.md
+++ b/src/AsideToggler.md
@@ -5,6 +5,7 @@ prop | default
 --- | ---
 children  | `<span className="navbar-toggler-icon" />`
 className | `navbar-toggler`
+defaultOpen| `false`
 display   | `lg`
 mobile    | `false`
 tag       | `button`

From 24c0406a234a53637c80687e6c3b48a6c3733d90 Mon Sep 17 00:00:00 2001
From: xidedix <xidedix@users.noreply.github.com>
Date: Fri, 5 Oct 2018 16:49:13 +0200
Subject: [PATCH 3/6] refactor(SidebarMinimizer): extract `togglePs` method

---
 src/SidebarMinimizer.js | 21 +++++++++++++++++----
 1 file changed, 17 insertions(+), 4 deletions(-)

diff --git a/src/SidebarMinimizer.js b/src/SidebarMinimizer.js
index 2992c5bf..4b365663 100644
--- a/src/SidebarMinimizer.js
+++ b/src/SidebarMinimizer.js
@@ -20,14 +20,27 @@ class AppSidebarMinimizer extends Component {
 
     this.handleClick = this.handleClick.bind(this);
   }
+
+  componentDidMount() {
+    const isMinimized = document.body.classList.contains('sidebar-minimized');
+    this.togglePs(!isMinimized)
+  }
+
   sidebarMinimize() {
-    document.body.classList.toggle('sidebar-minimized');
+    const isMinimized = document.body.classList.toggle('sidebar-minimized');
+    this.togglePs(!isMinimized)
+  }
+
+  togglePs(toggle) {
     const sidebar = document.querySelector('.sidebar-nav')
     if (sidebar) {
-      const toggleOn = sidebar.classList.toggle('ps');
-      sidebar.classList.toggle('scrollbar-container', toggleOn);
-      sidebar.classList.toggle('ps--active-y', toggleOn);
+      if (toggle) {
+        sidebar.classList.add('ps', 'ps-container', 'ps--active-y')
+      } else {
+        sidebar.classList.remove('ps', 'ps-container', 'ps--active-y')
+      }
     }
+
   }
 
   brandMinimize() {

From c5c10411636358422cc5e9455c8f788b4fbee646 Mon Sep 17 00:00:00 2001
From: xidedix <xidedix@users.noreply.github.com>
Date: Fri, 5 Oct 2018 16:49:33 +0200
Subject: [PATCH 4/6] chore: update demo

---
 demo/src/containers/DefaultLayout/DefaultLayout.js |  2 +-
 demo/src/scss/style.css                            | 12 +++++++++++-
 2 files changed, 12 insertions(+), 2 deletions(-)

diff --git a/demo/src/containers/DefaultLayout/DefaultLayout.js b/demo/src/containers/DefaultLayout/DefaultLayout.js
index c6ede867..e4bb99df 100644
--- a/demo/src/containers/DefaultLayout/DefaultLayout.js
+++ b/demo/src/containers/DefaultLayout/DefaultLayout.js
@@ -84,7 +84,7 @@ class DefaultLayout extends Component {
               </Switch>
             </Container>
           </main>
-          <AppAside fixed hidden display="lg">
+          <AppAside fixed>
             Aside
           </AppAside>
         </div>
diff --git a/demo/src/scss/style.css b/demo/src/scss/style.css
index 64479931..614b2d0c 100644
--- a/demo/src/scss/style.css
+++ b/demo/src/scss/style.css
@@ -1,7 +1,7 @@
 @charset "UTF-8";
 /*!
  * CoreUI - Open Source Dashboard UI Kit
- * @version v2.0.12
+ * @version v2.0.14
  * @link https://coreui.io
  * Copyright (c) 2018 creativeLabs Łukasz Holeczek
  * Licensed under MIT (https://coreui.io/license)
@@ -13808,6 +13808,16 @@ button.bg-gray-dark:focus {
   background-color: #181b1e !important;
 }
 
+.bg-light-blue {
+  background-color: #63c2de !important;
+}
+
+a.bg-light-blue:hover, a.bg-light-blue:focus,
+button.bg-light-blue:hover,
+button.bg-light-blue:focus {
+  background-color: #39b2d5 !important;
+}
+
 .bg-gray-100 {
   background-color: #f0f3f5 !important;
 }

From fe9f942fdff1a6d5d4907b535c15a2cae9b93613 Mon Sep 17 00:00:00 2001
From: xidedix <xidedix@users.noreply.github.com>
Date: Fri, 5 Oct 2018 16:50:20 +0200
Subject: [PATCH 5/6] chore: dependencies update

- update `@coreui/coreui` to `^2.0.14`
- update `reactstrap` to `^6.5.0`
- update `enzyme` to `3.7.0`
- update `enzyme-adapter-react-16` to `1.6.0`
---
 package.json | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/package.json b/package.json
index 97ccb564..60611ef2 100644
--- a/package.json
+++ b/package.json
@@ -43,7 +43,7 @@
     "react-onclickout": "^2.0.8",
     "react-perfect-scrollbar": "^1.2.2",
     "react-router-dom": "^4.3.1",
-    "reactstrap": "^6.4.0"
+    "reactstrap": "^6.5.0"
   },
   "peerDependencies": {
     "@coreui/coreui": "^2.0.14",
@@ -51,8 +51,8 @@
   },
   "devDependencies": {
     "babel-eslint": "^10.0.1",
-    "enzyme": "^3.6.0",
-    "enzyme-adapter-react-16": "^1.5.0",
+    "enzyme": "^3.7.0",
+    "enzyme-adapter-react-16": "^1.6.0",
     "eslint": "^5.6.1",
     "eslint-plugin-import": "^2.14.0",
     "eslint-plugin-react": "^7.11.1",

From 46accce048f9d45b2f832128c102e3803c2a77ce Mon Sep 17 00:00:00 2001
From: xidedix <xidedix@users.noreply.github.com>
Date: Fri, 5 Oct 2018 16:50:42 +0200
Subject: [PATCH 6/6] Ship: v2.0.8

---
 CHANGELOG.md | 62 ++++++++++++++++++++++++++++++++++++++++++++++++++++
 package.json |  2 +-
 2 files changed, 63 insertions(+), 1 deletion(-)
 create mode 100644 CHANGELOG.md

diff --git a/CHANGELOG.md b/CHANGELOG.md
new file mode 100644
index 00000000..cc235d8e
--- /dev/null
+++ b/CHANGELOG.md
@@ -0,0 +1,62 @@
+### [@coreui/react](https://coreui.io/) changelog
+
+##### `v2.0.8`
+- refactor(SidebarMinimizer): extract `togglePs` method
+- refactor(SidebarMinimizer): for use `togglePs` method
+- fix(SidebarMinimizer): add `componentDidMount` with `togglePs`
+- refactor(AsideToggler): extract `toggle` method
+- fix(AsideToggler): add missing `defaultOpen` prop
+- fix(Aside): deprecate `hidden` prop in v2 as conflicting with HTML5 `hidden` attribute
+- fix(Aside): `displayBreakpoint` behaviour
+- update `@coreui/coreui` to `^2.0.14`
+- update `reactstrap` to `^6.5.0`
+- update `enzyme` to `3.7.0`
+- update `enzyme-adapter-react-16` to `1.6.0`
+
+##### `v2.0.7`
+- fix(SidebarNav): dirty fix for rtl ps scrollingX issue
+
+##### `v2.0.6`
+- update `@coreui/coreui` to `^2.0.12`
+- update `@coreui/icons` to `0.3.0`
+- update `react-perfect-scrollbar` to `^1.2.2`
+- update `reactstrap` to `^6.4.0`
+- update `babel-eslint` to `^10.0.1`
+- update `enzyme` to `^3.6.0`
+- update `enzyme-adapter-react-16` to `^1.5.0`
+- update `eslint` to `^5.6.1`
+- update `eslint-plugin-import` to `^2.14.0`
+- update `eslint-plugin-react` to `^7.11.1`
+- update `nwb` to `^0.23.0`
+- update `react` to `^16.5.2`
+- update `react-dom` to `^16.5.2`
+- update `sinon` to `^5.1.1`
+
+##### `v2.0.5`
+- feat: hide onclick outside mobile sidebar
+- refactor: toggle-classes force
+- refactor: Shared/classes.js *Breakpoints
+- refactor: element-closest IE polyfill added
+- chore: dependencies update
+
+##### `v2.0.4`
+- refactor(Breadcrumb): fix for dynamic url like `/path/:id`
+- chore: update `prop-types` to `^15.6.2
+
+##### `v2.0.3`
+- fix: rollback to @coreui/icons v0.2.0
+
+##### `v2.0.2`
+- fix: remove AppLayout export as not production ready
+- chore: dependencies update
+- fix(Switch): not updating on props change - thanks @IceOnFire
+
+##### `v2.0.1`
+- chore: dependencies update
+
+##### `v2.0.0`
+- chore: dependencies update
+- refactor(demo): `@coreui/icons`
+
+##### `v2.0.0-rc.1`
+-fix: minimized sidebar scrollbar issue
diff --git a/package.json b/package.json
index 60611ef2..a1817cca 100644
--- a/package.json
+++ b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@coreui/react",
-  "version": "2.0.7",
+  "version": "2.0.8",
   "description": "CoreUI React Bootstrap 4 components",
   "license": "MIT",
   "author": {