Merge pull request #10 from coreui/fix-sidebar-ps-issue

xidedix · web-flow · commit fdb3310f50eb · 2018-05-09T13:06:25.000+02:00
fix: minimized sidebar scrollbar issue
diff --git a/src/SidebarMinimizer.js b/src/SidebarMinimizer.js
@@ -24,8 +24,9 @@ class AppSidebarMinimizer extends Component {
     document.body.classList.toggle('sidebar-minimized');
     const sidebar = document.querySelector('.sidebar-nav')
     if (sidebar) {
-      sidebar.classList.toggle('ps');
-      sidebar.classList.toggle('scrollbar-container');
+      const toggleOn = sidebar.classList.toggle('ps');
+      sidebar.classList.toggle('scrollbar-container', toggleOn);
+      sidebar.classList.toggle('ps--active-y', toggleOn);
     }
   }
 

Original file line number	Diff line number	Diff line change
`@@ -24,8 +24,9 @@ class AppSidebarMinimizer extends Component {`
`24`	`24`	`document.body.classList.toggle('sidebar-minimized');`
`25`	`25`	`const sidebar = document.querySelector('.sidebar-nav')`
`26`	`26`	`if (sidebar) {`
`27`		`- sidebar.classList.toggle('ps');`
`28`		`- sidebar.classList.toggle('scrollbar-container');`
	`27`	`+ const toggleOn = sidebar.classList.toggle('ps');`
	`28`	`+ sidebar.classList.toggle('scrollbar-container', toggleOn);`
	`29`	`+ sidebar.classList.toggle('ps--active-y', toggleOn);`
`29`	`30`	`}`
`30`	`31`	`}`
`31`	`32`