Merge pull request #14 from coreui/fix-hide-mobile-sidebar

xidedix · web-flow · commit d77504fd2290 · 2018-06-18T20:14:14.000+02:00
fix: hide mobile sidebar issue #13
diff --git a/projects/coreui/angular/src/lib/shared/layout/layout.directive.ts b/projects/coreui/angular/src/lib/shared/layout/layout.directive.ts
@@ -51,7 +51,7 @@ export class MobileSidebarToggleDirective {
   @HostListener('click', ['$event'])
   toggleOpen($event: any) {
     $event.preventDefault();
-    document.querySelector('body').classList.toggle('sidebar-mobile-show');
+    document.querySelector('body').classList.toggle('sidebar-show');
   }
 }
 
diff --git a/projects/coreui/angular/src/lib/sidebar/app-sidebar-nav.component.ts b/projects/coreui/angular/src/lib/sidebar/app-sidebar-nav.component.ts
@@ -147,8 +147,8 @@ export class AppSidebarNavLinkComponent implements OnInit {
   }
 
   public hideMobile() {
-    if (document.body.classList.contains('sidebar-mobile-show')) {
-      document.body.classList.toggle('sidebar-mobile-show');
+    if (document.body.classList.contains('sidebar-show')) {
+      document.body.classList.toggle('sidebar-show');
     }
   }
 

Original file line number	Diff line number	Diff line change
`@@ -51,7 +51,7 @@ export class MobileSidebarToggleDirective {`
`51`	`51`	`@HostListener('click', ['$event'])`
`52`	`52`	`toggleOpen($event: any) {`
`53`	`53`	`$event.preventDefault();`
`54`		`- document.querySelector('body').classList.toggle('sidebar-mobile-show');`
	`54`	`+ document.querySelector('body').classList.toggle('sidebar-show');`
`55`	`55`	`}`
`56`	`56`	`}`
`57`	`57`
Original file line number	Diff line number	Diff line change
`@@ -147,8 +147,8 @@ export class AppSidebarNavLinkComponent implements OnInit {`
`147`	`147`	`}`
`148`	`148`
`149`	`149`	`public hideMobile() {`
`150`		`- if (document.body.classList.contains('sidebar-mobile-show')) {`
`151`		`- document.body.classList.toggle('sidebar-mobile-show');`
	`150`	`+ if (document.body.classList.contains('sidebar-show')) {`
	`151`	`+ document.body.classList.toggle('sidebar-show');`
`152`	`152`	`}`
`153`	`153`	`}`
`154`	`154`