From 29a314597519c171ad967920d91abeb4f28365fb Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=E9=BB=91=E9=9B=A8?= <wangning4567@163.com>
Date: Wed, 22 Feb 2023 09:31:27 +0800
Subject: [PATCH] fix: fix 'span' to 'li'

---
 components/breadcrumb/BreadcrumbSeparator.tsx     |  6 +++++-
 .../__snapshots__/Breadcrumb.test.tsx.snap        | 10 ++++++----
 .../__snapshots__/demo-extend.test.ts.snap        | 15 +++++++++------
 .../__tests__/__snapshots__/demo.test.ts.snap     | 15 +++++++++------
 4 files changed, 29 insertions(+), 17 deletions(-)

diff --git a/components/breadcrumb/BreadcrumbSeparator.tsx b/components/breadcrumb/BreadcrumbSeparator.tsx
index 1b21a51fa3de..ee1c0724c944 100644
--- a/components/breadcrumb/BreadcrumbSeparator.tsx
+++ b/components/breadcrumb/BreadcrumbSeparator.tsx
@@ -10,7 +10,11 @@ const BreadcrumbSeparator: CompoundedComponent = ({ children }) => {
   const { getPrefixCls } = React.useContext(ConfigContext);
   const prefixCls = getPrefixCls('breadcrumb');
 
-  return <span className={`${prefixCls}-separator`}>{children || '/'}</span>;
+  return (
+    <li className={`${prefixCls}-separator`} aria-hidden="true">
+      {children || '/'}
+    </li>
+  );
 };
 
 BreadcrumbSeparator.__ANT_BREADCRUMB_SEPARATOR = true;
diff --git a/components/breadcrumb/__tests__/__snapshots__/Breadcrumb.test.tsx.snap b/components/breadcrumb/__tests__/__snapshots__/Breadcrumb.test.tsx.snap
index afe350f4de4e..4ce96c4022e4 100644
--- a/components/breadcrumb/__tests__/__snapshots__/Breadcrumb.test.tsx.snap
+++ b/components/breadcrumb/__tests__/__snapshots__/Breadcrumb.test.tsx.snap
@@ -12,11 +12,12 @@ exports[`Breadcrumb filter React.Fragment 1`] = `
         Location
       </span>
     </li>
-    <span
+    <li
+      aria-hidden="true"
       class="ant-breadcrumb-separator"
     >
       :
-    </span>
+    </li>
     <li>
       <a
         class="ant-breadcrumb-link"
@@ -25,11 +26,12 @@ exports[`Breadcrumb filter React.Fragment 1`] = `
         Application Center
       </a>
     </li>
-    <span
+    <li
+      aria-hidden="true"
       class="ant-breadcrumb-separator"
     >
       /
-    </span>
+    </li>
   </ol>
 </nav>
 `;
diff --git a/components/breadcrumb/__tests__/__snapshots__/demo-extend.test.ts.snap b/components/breadcrumb/__tests__/__snapshots__/demo-extend.test.ts.snap
index 2d12c4ea5525..9b6292f13167 100644
--- a/components/breadcrumb/__tests__/__snapshots__/demo-extend.test.ts.snap
+++ b/components/breadcrumb/__tests__/__snapshots__/demo-extend.test.ts.snap
@@ -426,11 +426,12 @@ exports[`renders ./components/breadcrumb/demo/separator-component.tsx extend con
         Location
       </span>
     </li>
-    <span
+    <li
+      aria-hidden="true"
       class="ant-breadcrumb-separator"
     >
       :
-    </span>
+    </li>
     <li>
       <a
         class="ant-breadcrumb-link"
@@ -439,11 +440,12 @@ exports[`renders ./components/breadcrumb/demo/separator-component.tsx extend con
         Application Center
       </a>
     </li>
-    <span
+    <li
+      aria-hidden="true"
       class="ant-breadcrumb-separator"
     >
       /
-    </span>
+    </li>
     <li>
       <a
         class="ant-breadcrumb-link"
@@ -452,11 +454,12 @@ exports[`renders ./components/breadcrumb/demo/separator-component.tsx extend con
         Application List
       </a>
     </li>
-    <span
+    <li
+      aria-hidden="true"
       class="ant-breadcrumb-separator"
     >
       /
-    </span>
+    </li>
     <li>
       <span
         class="ant-breadcrumb-link"
diff --git a/components/breadcrumb/__tests__/__snapshots__/demo.test.ts.snap b/components/breadcrumb/__tests__/__snapshots__/demo.test.ts.snap
index 325ffedf94f4..e96338d10f4e 100644
--- a/components/breadcrumb/__tests__/__snapshots__/demo.test.ts.snap
+++ b/components/breadcrumb/__tests__/__snapshots__/demo.test.ts.snap
@@ -224,11 +224,12 @@ exports[`renders ./components/breadcrumb/demo/separator-component.tsx correctly
         Location
       </span>
     </li>
-    <span
+    <li
+      aria-hidden="true"
       class="ant-breadcrumb-separator"
     >
       :
-    </span>
+    </li>
     <li>
       <a
         class="ant-breadcrumb-link"
@@ -237,11 +238,12 @@ exports[`renders ./components/breadcrumb/demo/separator-component.tsx correctly
         Application Center
       </a>
     </li>
-    <span
+    <li
+      aria-hidden="true"
       class="ant-breadcrumb-separator"
     >
       /
-    </span>
+    </li>
     <li>
       <a
         class="ant-breadcrumb-link"
@@ -250,11 +252,12 @@ exports[`renders ./components/breadcrumb/demo/separator-component.tsx correctly
         Application List
       </a>
     </li>
-    <span
+    <li
+      aria-hidden="true"
       class="ant-breadcrumb-separator"
     >
       /
-    </span>
+    </li>
     <li>
       <span
         class="ant-breadcrumb-link"