fix(md-sidenav): enablle closeOnClick for screens > 992px, close #318

aurelia-ui-toolkits · Nov 20, 2016 · 4a86ab8 · 4a86ab8
1 parent 0a59ca7
commit 4a86ab8
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/src/sidenav/sidenav-collapse.js b/src/sidenav/sidenav-collapse.js
@@ -19,10 +19,13 @@ export class MdSidenavCollapse {
       // this.widthSubscription = this.observerLocator.getObserver(this.ref, 'mdWidth').subscribe(this.widthChanged.bind(this));
       // this.fixedSubscription = this.observerLocator.getObserver(this.ref, 'fixed').subscribe(this.fixedChanged.bind(this));
 
+      const closeOnClick = this.ref.mdFixed && window.innerWidth > 992 ? false : getBooleanFromAttributeValue(this.ref.mdCloseOnClick);
+
       this.element.setAttribute('data-activates', this.ref.controlId);
       let sideNavConfig = {
         edge: this.ref.mdEdge || 'left',
-        closeOnClick: (this.ref.mdFixed ? false : getBooleanFromAttributeValue(this.ref.mdCloseOnClick)),
+        // closeOnClick: (this.ref.mdFixed ? false : getBooleanFromAttributeValue(this.ref.mdCloseOnClick)),
+        closeOnClick,
         menuWidth: parseInt(this.ref.mdWidth, 10)
       };
       // this.log.debug('sideNavConfig:', sideNavConfig);