[Breadcrumbs] Remove private separator component (#19234)

mui · Jan 15, 2020 · 3c1d3a1 · 3c1d3a1
1 parent 46d4e6a
commit 3c1d3a1
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 60 deletions.
diff --git a/packages/material-ui/src/Breadcrumbs/BreadcrumbSeparator.js b/packages/material-ui/src/Breadcrumbs/BreadcrumbSeparator.js
diff --git a/packages/material-ui/src/Breadcrumbs/BreadcrumbSeparator.test.js b/packages/material-ui/src/Breadcrumbs/BreadcrumbSeparator.test.js
diff --git a/packages/material-ui/src/Breadcrumbs/Breadcrumbs.js b/packages/material-ui/src/Breadcrumbs/Breadcrumbs.js
@@ -5,7 +5,6 @@ import clsx from 'clsx';
 import withStyles from '../styles/withStyles';
 import Typography from '../Typography';
 import BreadcrumbCollapsed from './BreadcrumbCollapsed';
-import BreadcrumbSeparator from './BreadcrumbSeparator';
 
 export const styles = {
   /* Styles applied to the root element. */
@@ -23,17 +22,22 @@ export const styles = {
     listStyle: 'none',
   },
   /* Styles applied to the separator element. */
-  separator: {},
+  separator: {
+    display: 'flex',
+    userSelect: 'none',
+    marginLeft: 8,
+    marginRight: 8,
+  },
 };
 
 function insertSeparators(items, className, separator) {
   return items.reduce((acc, current, index) => {
     if (index < items.length - 1) {
       acc = acc.concat(
         current,
-        <BreadcrumbSeparator key={`separator-${index}`} className={className}>
+        <li aria-hidden key={`separator-${index}`} className={className}>
           {separator}
-        </BreadcrumbSeparator>,
+        </li>,
       );
     } else {
       acc.push(current);