Hide unused items (#18)

Co-authored-by: Luis Felipe Furlan Cruz <lfcruz@cpqd.com.br> Co-authored-by: Mariane Previde <mprevide@cpqd.com.br>
dojot · Oct 14, 2020 · 2e5a25e · 2e5a25e
1 parent c154f3f
commit 2e5a25e
Show file tree

Hide file tree

Showing 5 changed files with 43 additions and 26 deletions.
diff --git a/.gitignore b/.gitignore
@@ -9,4 +9,5 @@ npm-debug.log
 .eslintcache
 /.env
 /coverage
+/test
 build
diff --git a/src/js/common/components/Drawer/Drawer.jsx b/src/js/common/components/Drawer/Drawer.jsx
@@ -43,34 +43,38 @@ const DrawerComponent = props => {
         <img src={logo} alt='dojot logo' />
       </div>
       <MenuList disablePadding>
-        {primaryItems.map(item => (
-          <Link to={item.path} className={classes.menuLink} key={item.label}>
-            <MenuItem
-              selected={activeRoute(item)}
-              classes={{ selected: classes.selected }}
-            >
-              <ListItemIcon>
-                <item.icon />
-              </ListItemIcon>
-              <ListItemText primary={item.label} />
-            </MenuItem>
-          </Link>
-        ))}
+        {primaryItems.map(item =>
+          item.visible ? (
+            <Link to={item.path} className={classes.menuLink} key={item.label}>
+              <MenuItem
+                selected={activeRoute(item)}
+                classes={{ selected: classes.selected }}
+              >
+                <ListItemIcon>
+                  <item.icon />
+                </ListItemIcon>
+                <ListItemText primary={item.label} />
+              </MenuItem>
+            </Link>
+          ) : null,
+        )}
       </MenuList>
       <MenuList className={classes.bottomList}>
-        {secondaryItems.map(item => (
-          <Link to={item.path} className={classes.menuLink} key={item.label}>
-            <MenuItem
-              selected={activeRoute(item)}
-              classes={{ selected: classes.selected }}
-            >
-              <ListItemIcon>
-                <item.icon />
-              </ListItemIcon>
-              <ListItemText primary={item.label} />
-            </MenuItem>
-          </Link>
-        ))}
+        {secondaryItems.map(item =>
+          item.visible ? (
+            <Link to={item.path} className={classes.menuLink} key={item.label}>
+              <MenuItem
+                selected={activeRoute(item)}
+                classes={{ selected: classes.selected }}
+              >
+                <ListItemIcon>
+                  <item.icon />
+                </ListItemIcon>
+                <ListItemText primary={item.label} />
+              </MenuItem>
+            </Link>
+          ) : null,
+        )}
       </MenuList>
     </Drawer>
   );

diff --git a/src/js/common/components/Drawer/Drawer.test.js b/src/js/common/components/Drawer/Drawer.test.js
@@ -17,23 +17,27 @@ const initialProps = {
   isOpen: false,
   primaryItems: [
     {
+      visible: true,
       label: 'Dispositivos',
       path: '/devices',
       icon: DevicesOther,
     },
     {
+      visible: true,
       label: 'Modelos',
       path: '/templates',
       icon: FilterNone,
     },
   ],
   secondaryItems: [
     {
+      visible: true,
       label: 'Ajuda',
       path: '/help',
       icon: HelpOutline,
     },
     {
+      visible: true,
       label: 'Sair',
       path: '/logout',
       icon: ExitToApp,

diff --git a/src/js/common/menu/primaryItems.js b/src/js/common/menu/primaryItems.js
@@ -10,18 +10,21 @@ import {
 
 export default [
   {
+    visible: false,
     i18n: 'devices',
     label: 'Dispositivos',
     path: '/devices',
     icon: DevicesOther,
   },
   {
+    visible: false,
     i18n: 'templates',
     label: 'Modelos',
     path: '/templates',
     icon: FilterNone,
   },
   {
+    visible: false,
     i18n: 'flows',
     label: 'Fluxos',
     path: '/flow',
@@ -34,18 +37,21 @@ export default [
     icon: NotificationImportant,
   },
   {
+    visible: false,
     i18n: 'users',
     label: 'Usuários',
     path: '/users',
     icon: People,
   },
   {
+    visible: false,
     i18n: 'profiles',
     label: 'Perfis',
     path: '/profiles',
     icon: SupervisedUserCircle,
   },
   {
+    visible: true,
     i18n: 'dashboard',
     label: 'Dashboard',
     path: '/dashboard',

diff --git a/src/js/common/menu/secondaryItems.js b/src/js/common/menu/secondaryItems.js
@@ -2,12 +2,14 @@ import { ExitToApp, HelpOutline } from '@material-ui/icons';
 
 export default [
   {
+    visible: false,
     i18n: 'help',
     label: 'Ajuda',
     path: '/help',
     icon: HelpOutline,
   },
   {
+    visible: true,
     i18n: 'exit',
     label: 'Sair',
     path: '/logout',