Merge pull request #4241 from steurt/next

[RFR] Also pass the dense prop from the Menu to the DashboardMenuItem
marmelab · Jan 6, 2020 · 31f8aee · 31f8aee
2 parents ae3c2f7 + 2f0934d
commit 31f8aee
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/packages/ra-ui-materialui/src/layout/DashboardMenuItem.js b/packages/ra-ui-materialui/src/layout/DashboardMenuItem.js
@@ -14,7 +14,6 @@ const DashboardMenuItem = ({ locale, onClick, ...props }) => {
             primaryText={translate('ra.page.dashboard')}
             leftIcon={<DashboardIcon />}
             exact
-            dense
             {...props}
         />
     );

diff --git a/packages/ra-ui-materialui/src/layout/Menu.js b/packages/ra-ui-materialui/src/layout/Menu.js
@@ -52,7 +52,11 @@ const Menu = ({
     return (
         <div className={classnames(classes.main, className)} {...rest}>
             {hasDashboard && (
-                <DashboardMenuItem onClick={onMenuClick} sidebarIsOpen={open} />
+                <DashboardMenuItem
+                    onClick={onMenuClick}
+                    dense={dense}
+                    sidebarIsOpen={open}
+                />
             )}
             {resources
                 .filter(r => r.hasList)
-Original file line number
+Diff line change
@@ Expand Up @@
                 primaryText={translate('ra.page.dashboard')}
                 leftIcon={<DashboardIcon />}
                 exact
-                dense
                 {...props}
             />
         );
@@ Expand Down @@