From 168a112802cf07331331364f26e7e13afa881f88 Mon Sep 17 00:00:00 2001
From: rabbihossain <me@2hr.me>
Date: Fri, 6 Mar 2020 11:52:47 +0600
Subject: [PATCH] feat: making sidebar collapsible

this changes make the sidebar collapsible

"fix #1858"
---
 src/HospitalRun.tsx                       |  8 +++++-
 src/__tests__/HospitalRun.test.tsx        | 15 ++++++++++++
 src/__tests__/components/Sidebar.test.tsx | 30 +++++++++++++++--------
 src/components/Sidebar.tsx                | 27 +++++++++++++++++---
 src/components/component-slice.ts         | 28 +++++++++++++++++++++
 src/index.css                             |  2 +-
 src/store/index.ts                        |  2 ++
 7 files changed, 96 insertions(+), 16 deletions(-)
 create mode 100644 src/components/component-slice.ts

diff --git a/src/HospitalRun.tsx b/src/HospitalRun.tsx
index b54011daa6..30cd77acd4 100644
--- a/src/HospitalRun.tsx
+++ b/src/HospitalRun.tsx
@@ -23,6 +23,7 @@ import PrivateRoute from './components/PrivateRoute'
 const HospitalRun = () => {
   const { title } = useSelector((state: RootState) => state.title)
   const { permissions } = useSelector((state: RootState) => state.user)
+  const { sidebarCollapsed } = useSelector((state: RootState) => state.components)
 
   return (
     <div>
@@ -31,7 +32,12 @@ const HospitalRun = () => {
         <Sidebar />
         <ButtonBarProvider>
           <div className="row">
-            <main role="main" className="col-md-9 ml-sm-auto col-lg-10 px-4">
+            <main
+              role="main"
+              className={`${
+                sidebarCollapsed ? 'col-md-10 col-lg-11' : 'col-md-9 col-lg-10'
+              } ml-sm-auto px-4`}
+            >
               <div className="d-flex justify-content-between flex-wrap flex-md-nowrap align-items-center pt-3 pb-2 mb-3 border-bottom">
                 <h1 className="h2">{title}</h1>
                 <ButtonToolBar />
diff --git a/src/__tests__/HospitalRun.test.tsx b/src/__tests__/HospitalRun.test.tsx
index bf7572ae68..17541a3735 100644
--- a/src/__tests__/HospitalRun.test.tsx
+++ b/src/__tests__/HospitalRun.test.tsx
@@ -34,6 +34,7 @@ describe('HospitalRun', () => {
           title: 'test',
           user: { permissions: [Permissions.WritePatients] },
           breadcrumbs: { breadcrumbs: [] },
+          components: { sidebarCollapsed: false },
         })
 
         const wrapper = mount(
@@ -62,6 +63,7 @@ describe('HospitalRun', () => {
               title: 'test',
               user: { permissions: [] },
               breadcrumbs: { breadcrumbs: [] },
+              components: { sidebarCollapsed: false },
             })}
           >
             <MemoryRouter initialEntries={['/patients/new']}>
@@ -94,6 +96,7 @@ describe('HospitalRun', () => {
           user: { permissions: [Permissions.WritePatients, Permissions.ReadPatients] },
           patient: { patient },
           breadcrumbs: { breadcrumbs: [] },
+          components: { sidebarCollapsed: false },
         })
 
         const wrapper = mount(
@@ -123,6 +126,7 @@ describe('HospitalRun', () => {
               title: 'test',
               user: { permissions: [Permissions.WritePatients] },
               breadcrumbs: { breadcrumbs: [] },
+              components: { sidebarCollapsed: false },
             })}
           >
             <MemoryRouter initialEntries={['/patients/edit/123']}>
@@ -141,6 +145,7 @@ describe('HospitalRun', () => {
               title: 'test',
               user: { permissions: [Permissions.ReadPatients] },
               breadcrumbs: { breadcrumbs: [] },
+              components: { sidebarCollapsed: false },
             })}
           >
             <MemoryRouter initialEntries={['/patients/edit/123']}>
@@ -173,6 +178,7 @@ describe('HospitalRun', () => {
           user: { permissions: [Permissions.ReadPatients] },
           patient: { patient },
           breadcrumbs: { breadcrumbs: [] },
+          components: { sidebarCollapsed: false },
         })
 
         const wrapper = mount(
@@ -201,6 +207,7 @@ describe('HospitalRun', () => {
               title: 'test',
               user: { permissions: [] },
               breadcrumbs: { breadcrumbs: [] },
+              components: { sidebarCollapsed: false },
             })}
           >
             <MemoryRouter initialEntries={['/patients/123']}>
@@ -220,6 +227,7 @@ describe('HospitalRun', () => {
           user: { permissions: [Permissions.ReadAppointments] },
           appointments: { appointments: [] },
           breadcrumbs: { breadcrumbs: [] },
+          components: { sidebarCollapsed: false },
         })
 
         const wrapper = mount(
@@ -251,6 +259,7 @@ describe('HospitalRun', () => {
               title: 'test',
               user: { permissions: [] },
               breadcrumbs: { breadcrumbs: [] },
+              components: { sidebarCollapsed: false },
             })}
           >
             <MemoryRouter initialEntries={['/appointments']}>
@@ -269,6 +278,7 @@ describe('HospitalRun', () => {
           title: 'test',
           user: { permissions: [Permissions.WriteAppointments] },
           breadcrumbs: { breadcrumbs: [] },
+          components: { sidebarCollapsed: false },
         })
 
         const wrapper = mount(
@@ -298,6 +308,7 @@ describe('HospitalRun', () => {
               title: 'test',
               user: { permissions: [] },
               breadcrumbs: { breadcrumbs: [] },
+              components: { sidebarCollapsed: false },
             })}
           >
             <MemoryRouter initialEntries={['/appointments/new']}>
@@ -332,6 +343,7 @@ describe('HospitalRun', () => {
           user: { permissions: [Permissions.WriteAppointments, Permissions.ReadAppointments] },
           appointment: { appointment, patient: {} as Patient },
           breadcrumbs: { breadcrumbs: [] },
+          components: { sidebarCollapsed: false },
         })
 
         const wrapper = mount(
@@ -364,6 +376,7 @@ describe('HospitalRun', () => {
               title: 'test',
               user: { permissions: [Permissions.WriteAppointments] },
               breadcrumbs: { breadcrumbs: [] },
+              components: { sidebarCollapsed: false },
             })}
           >
             <MemoryRouter initialEntries={['/appointments/edit/123']}>
@@ -382,6 +395,7 @@ describe('HospitalRun', () => {
               title: 'test',
               user: { permissions: [Permissions.ReadAppointments] },
               breadcrumbs: { breadcrumbs: [] },
+              components: { sidebarCollapsed: false },
             })}
           >
             <MemoryRouter initialEntries={['/appointments/edit/123']}>
@@ -403,6 +417,7 @@ describe('HospitalRun', () => {
             title: 'test',
             user: { permissions: [Permissions.WritePatients] },
             breadcrumbs: { breadcrumbs: [] },
+            components: { sidebarCollapsed: false },
           })}
         >
           <MemoryRouter initialEntries={['/']}>
diff --git a/src/__tests__/components/Sidebar.test.tsx b/src/__tests__/components/Sidebar.test.tsx
index 3c37a9b090..c5136b3a8b 100644
--- a/src/__tests__/components/Sidebar.test.tsx
+++ b/src/__tests__/components/Sidebar.test.tsx
@@ -6,15 +6,25 @@ import Sidebar from 'components/Sidebar'
 import { Router } from 'react-router'
 import { ListItem } from '@hospitalrun/components'
 import { act } from '@testing-library/react'
+import configureMockStore from 'redux-mock-store'
+import thunk from 'redux-thunk'
+import { Provider } from 'react-redux'
+
+const mockStore = configureMockStore([thunk])
 
 describe('Sidebar', () => {
   let history = createMemoryHistory()
+  const store = mockStore({
+    components: { sidebarCollapsed: false },
+  })
   const setup = (location: string) => {
     history = createMemoryHistory()
     history.push(location)
     return mount(
       <Router history={history}>
-        <Sidebar />
+        <Provider store={store}>
+          <Sidebar />
+        </Provider>
       </Router>,
     )
   }
@@ -27,7 +37,7 @@ describe('Sidebar', () => {
 
       expect(
         listItems
-          .at(0)
+          .at(1)
           .text()
           .trim(),
       ).toEqual('dashboard.label')
@@ -38,7 +48,7 @@ describe('Sidebar', () => {
 
       const listItems = wrapper.find(ListItem)
 
-      expect(listItems.at(0).prop('active')).toBeTruthy()
+      expect(listItems.at(1).prop('active')).toBeTruthy()
     })
 
     it('should navigate to / when the dashboard link is clicked', () => {
@@ -47,7 +57,7 @@ describe('Sidebar', () => {
       const listItems = wrapper.find(ListItem)
 
       act(() => {
-        ;(listItems.at(0).prop('onClick') as any)()
+        ;(listItems.at(1).prop('onClick') as any)()
       })
 
       expect(history.location.pathname).toEqual('/')
@@ -62,7 +72,7 @@ describe('Sidebar', () => {
 
       expect(
         listItems
-          .at(1)
+          .at(2)
           .text()
           .trim(),
       ).toEqual('patients.label')
@@ -73,7 +83,7 @@ describe('Sidebar', () => {
 
       const listItems = wrapper.find(ListItem)
 
-      expect(listItems.at(1).prop('active')).toBeTruthy()
+      expect(listItems.at(2).prop('active')).toBeTruthy()
     })
 
     it('should navigate to /patients when the patients link is clicked', () => {
@@ -82,7 +92,7 @@ describe('Sidebar', () => {
       const listItems = wrapper.find(ListItem)
 
       act(() => {
-        ;(listItems.at(1).prop('onClick') as any)()
+        ;(listItems.at(2).prop('onClick') as any)()
       })
 
       expect(history.location.pathname).toEqual('/patients')
@@ -97,7 +107,7 @@ describe('Sidebar', () => {
 
       expect(
         listItems
-          .at(2)
+          .at(3)
           .text()
           .trim(),
       ).toEqual('scheduling.label')
@@ -108,7 +118,7 @@ describe('Sidebar', () => {
 
       const listItems = wrapper.find(ListItem)
 
-      expect(listItems.at(2).prop('active')).toBeTruthy()
+      expect(listItems.at(3).prop('active')).toBeTruthy()
     })
 
     it('should navigate to /appointments when the scheduling link is clicked', () => {
@@ -117,7 +127,7 @@ describe('Sidebar', () => {
       const listItems = wrapper.find(ListItem)
 
       act(() => {
-        ;(listItems.at(2).prop('onClick') as any)()
+        ;(listItems.at(3).prop('onClick') as any)()
       })
 
       expect(history.location.pathname).toEqual('/appointments')
diff --git a/src/components/Sidebar.tsx b/src/components/Sidebar.tsx
index 8957c07e9e..528357d23d 100644
--- a/src/components/Sidebar.tsx
+++ b/src/components/Sidebar.tsx
@@ -2,8 +2,14 @@ import React, { CSSProperties } from 'react'
 import { List, ListItem, Icon } from '@hospitalrun/components'
 import { useTranslation } from 'react-i18next'
 import { useLocation, useHistory } from 'react-router'
+import { useSelector, useDispatch } from 'react-redux'
+import { RootState } from '../store'
+import { updateSidebar } from './component-slice'
 
 const Sidebar = () => {
+  const dispatch = useDispatch()
+  const { sidebarCollapsed } = useSelector((state: RootState) => state.components)
+
   const { t } = useTranslation()
   const path = useLocation()
   const history = useHistory()
@@ -18,16 +24,29 @@ const Sidebar = () => {
   }
 
   return (
-    <nav className="col-md-2 d-none d-md-block bg-light sidebar">
+    <nav
+      className="col-md-2 d-none d-md-block bg-light sidebar"
+      style={{ width: sidebarCollapsed ? '56px' : '' }}
+    >
       <div className="sidebar-sticky">
         <List layout="flush" className="nav flex-column">
+          <ListItem
+            onClick={() => dispatch(updateSidebar())}
+            className="nav-item"
+            style={listItemStyle}
+          >
+            <Icon
+              style={{ float: sidebarCollapsed ? 'left' : 'right' }}
+              icon={sidebarCollapsed ? 'right-arrow' : 'left-arrow'}
+            />
+          </ListItem>
           <ListItem
             active={pathname === '/'}
             onClick={() => navigateTo('/')}
             className="nav-item"
             style={listItemStyle}
           >
-            <Icon icon="dashboard" /> {t('dashboard.label')}
+            <Icon icon="dashboard" /> {!sidebarCollapsed && t('dashboard.label')}
           </ListItem>
           <ListItem
             active={pathname.split('/')[1].includes('patient')}
@@ -35,7 +54,7 @@ const Sidebar = () => {
             className="nav-item"
             style={listItemStyle}
           >
-            <Icon icon="patients" /> {t('patients.label')}
+            <Icon icon="patients" /> {!sidebarCollapsed && t('patients.label')}
           </ListItem>
           <ListItem
             active={pathname.split('/')[1].includes('appointments')}
@@ -43,7 +62,7 @@ const Sidebar = () => {
             className="nav-item"
             style={listItemStyle}
           >
-            <Icon icon="appointment" /> {t('scheduling.label')}
+            <Icon icon="appointment" /> {!sidebarCollapsed && t('scheduling.label')}
           </ListItem>
         </List>
       </div>
diff --git a/src/components/component-slice.ts b/src/components/component-slice.ts
new file mode 100644
index 0000000000..62a6aeeefe
--- /dev/null
+++ b/src/components/component-slice.ts
@@ -0,0 +1,28 @@
+import { createSlice } from '@reduxjs/toolkit'
+import { AppThunk } from '../store'
+
+interface ComponentState {
+  sidebarCollapsed: boolean
+}
+
+const initialState: ComponentState = {
+  sidebarCollapsed: false,
+}
+
+const componentSlice = createSlice({
+  name: 'components',
+  initialState,
+  reducers: {
+    toggleSidebar(state) {
+      state.sidebarCollapsed = !state.sidebarCollapsed
+    },
+  },
+})
+
+export const { toggleSidebar } = componentSlice.actions
+
+export const updateSidebar = (): AppThunk => async (dispatch) => {
+  dispatch(toggleSidebar())
+}
+
+export default componentSlice.reducer
diff --git a/src/index.css b/src/index.css
index 0c52028d7a..a11f30cfa9 100644
--- a/src/index.css
+++ b/src/index.css
@@ -25,7 +25,7 @@ code {
   left: 0;
   z-index: 0; /* Behind the navbar */
   padding: 48px 0 0; /* Height of navbar */
-  box-shadow: inset -1px 0 0 rgba(0, 0, 0, .1);
+  border-right: 1px solid rgba(0, 0, 0, .1);
 }
 
 .sidebar-sticky {
diff --git a/src/store/index.ts b/src/store/index.ts
index b226815116..c381425dec 100644
--- a/src/store/index.ts
+++ b/src/store/index.ts
@@ -7,6 +7,7 @@ import appointments from '../scheduling/appointments/appointments-slice'
 import title from '../page-header/title-slice'
 import user from '../user/user-slice'
 import breadcrumbs from '../breadcrumbs/breadcrumbs-slice'
+import components from '../components/component-slice'
 
 const reducer = combineReducers({
   patient,
@@ -16,6 +17,7 @@ const reducer = combineReducers({
   appointment,
   appointments,
   breadcrumbs,
+  components,
 })
 
 const store = configureStore({