fix: if breadcrumbRender return false, breadcrumb will hidden (ant-de…

…sign#32738) * fix: if breadcrumbRender return false, breadcrumb will hidden * add testd
TranquilMarmot · Nov 10, 2021 · 01887b4 · 01887b4
1 parent 5f642cb
commit 01887b4
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/components/page-header/__tests__/index.test.js b/components/page-header/__tests__/index.test.js
@@ -73,6 +73,11 @@ describe('PageHeader', () => {
     expect(wrapper.find('.ant-page-header-back')).toHaveLength(0);
   });
 
+  it('pageHeader support breadcrumbRender return false', () => {
+    const wrapper = mount(<PageHeader title="Page Title" breadcrumbRender={() => false} />);
+    expect(wrapper.find('.ant-page-header-back')).toHaveLength(0);
+  });
+
   it('pageHeader do not has title', () => {
     const routes = [
       {

diff --git a/components/page-header/index.tsx b/components/page-header/index.tsx
@@ -159,7 +159,7 @@ const PageHeader: React.FC<PageHeaderProps> = props => {
         const isBreadcrumbComponent = breadcrumb && 'props' in breadcrumb;
         //  support breadcrumbRender function
         const breadcrumbRenderDomFromProps =
-          breadcrumbRender?.(props, defaultBreadcrumbDom) || defaultBreadcrumbDom;
+          breadcrumbRender?.(props, defaultBreadcrumbDom) ?? defaultBreadcrumbDom;
 
         const breadcrumbDom = isBreadcrumbComponent ? breadcrumb : breadcrumbRenderDomFromProps;