Fix custom menu snippet in theming documentation

Closes marmelab#4416
akshah123 · Feb 14, 2020 · 265200b · 265200b
1 parent 21c63d8
commit 265200b
Showing 1 changed file with 9 additions and 4 deletions.
diff --git a/docs/Theming.md b/docs/Theming.md
@@ -719,7 +719,7 @@ import React, { createElement } from 'react';
 import { useSelector } from 'react-redux';
 import { useMediaQuery } from '@material-ui/core';
 import { MenuItemLink, getResources } from 'react-admin';
-import { withRouter } from 'react-router-dom';
+import DefaultIcon from '@material-ui/icons/ViewList';
 import LabelIcon from '@material-ui/icons/Label';
 
 const Menu = ({ onMenuClick, logout }) => {
@@ -732,8 +732,13 @@ const Menu = ({ onMenuClick, logout }) => {
                 <MenuItemLink
                     key={resource.name}
                     to={`/${resource.name}`}
-                    primaryText={resource.options && resource.options.label || resource.name}
-                    leftIcon={createElement(resource.icon)}
+                    primaryText={
+                        (resource.options && resource.options.label) ||
+                        resource.name
+                    }
+                    leftIcon={
+                        resource.icon ? <resource.icon /> : <DefaultIcon />
+                    }
                     onClick={onMenuClick}
                     sidebarIsOpen={open}
                 />
@@ -750,7 +755,7 @@ const Menu = ({ onMenuClick, logout }) => {
     );
 };
 
-export default withRouter(Menu);
+export default Menu;
 ```
 
 **Tip**: Note the `MenuItemLink` component. It must be used to avoid unwanted side effects in mobile views.