From 77b351f68fdd2f26a11c111ad5023b9522e64d6d Mon Sep 17 00:00:00 2001
From: Fatpandac <i@fatpandac.com>
Date: Tue, 16 Jan 2024 14:02:31 +0800
Subject: [PATCH] fix: ok button of the DatePicker becomes compacted when used
 with Space.Compact (#46769)

* fix: ok button of the DatePicker becomes compacted when used with Space.Compact

* fix: break CSS selector

* fix: break the compact context on the panel

* chore: update test
---
 components/date-picker/PickerButton.tsx       |   2 +-
 .../generatePicker/generateRangePicker.tsx    | 112 +++++++++---------
 .../generatePicker/generateSinglePicker.tsx   | 108 ++++++++---------
 .../__snapshots__/demo-extend.test.ts.snap    |   6 +-
 components/statistic/index.ts                 |   2 +-
 components/table/InternalTable.tsx            |   2 +-
 6 files changed, 120 insertions(+), 112 deletions(-)

diff --git a/components/date-picker/PickerButton.tsx b/components/date-picker/PickerButton.tsx
index bf8b0f462b92..43a9e1f00257 100644
--- a/components/date-picker/PickerButton.tsx
+++ b/components/date-picker/PickerButton.tsx
@@ -1,6 +1,6 @@
 import * as React from 'react';
-import type { ButtonProps } from '../button';
 import Button from '../button';
+import type { ButtonProps } from '../button';
 
 export default function PickerButton(props: ButtonProps) {
   return <Button size="small" type="primary" {...props} />;
diff --git a/components/date-picker/generatePicker/generateRangePicker.tsx b/components/date-picker/generatePicker/generateRangePicker.tsx
index 04ae1acbb868..c7a260b67ec5 100644
--- a/components/date-picker/generatePicker/generateRangePicker.tsx
+++ b/components/date-picker/generatePicker/generateRangePicker.tsx
@@ -16,7 +16,7 @@ import DisabledContext from '../../config-provider/DisabledContext';
 import useSize from '../../config-provider/hooks/useSize';
 import { FormItemInputContext } from '../../form/context';
 import { useLocale } from '../../locale';
-import { useCompactItemContext } from '../../space/Compact';
+import { NoCompactStyle, useCompactItemContext } from '../../space/Compact';
 import enUS from '../locale/en_US';
 import useStyle from '../style';
 import {
@@ -124,58 +124,64 @@ export default function generateRangePicker<DateType>(generateConfig: GenerateCo
     const [zIndex] = useZIndex('DatePicker', props.popupStyle?.zIndex as number);
 
     return wrapCSSVar(
-      <RCRangePicker<DateType>
-        separator={
-          <span aria-label="to" className={`${prefixCls}-separator`}>
-            <SwapRightOutlined />
-          </span>
-        }
-        disabled={mergedDisabled}
-        ref={innerRef}
-        dropdownAlign={transPlacement2DropdownAlign(direction, placement)}
-        placeholder={getRangePlaceholder(locale, picker, placeholder)}
-        suffixIcon={suffixNode}
-        prevIcon={<span className={`${prefixCls}-prev-icon`} />}
-        nextIcon={<span className={`${prefixCls}-next-icon`} />}
-        superPrevIcon={<span className={`${prefixCls}-super-prev-icon`} />}
-        superNextIcon={<span className={`${prefixCls}-super-next-icon`} />}
-        transitionName={`${rootPrefixCls}-slide-up`}
-        {...restProps}
-        {...additionalOverrideProps}
-        className={classNames(
-          {
-            [`${prefixCls}-${mergedSize}`]: mergedSize,
-            [`${prefixCls}-${variant}`]: enableVariantCls,
-          },
-          getStatusClassNames(prefixCls, getMergedStatus(contextStatus, customStatus), hasFeedback),
-          hashId,
-          compactItemClassnames,
-          className,
-          rangePicker?.className,
-          cssVarCls,
-          rootCls,
-          rootClassName,
-        )}
-        style={{ ...rangePicker?.style, ...style }}
-        locale={locale.lang}
-        prefixCls={prefixCls}
-        getPopupContainer={customGetPopupContainer || getPopupContainer}
-        generateConfig={generateConfig}
-        components={Components}
-        direction={direction}
-        dropdownClassName={classNames(
-          hashId,
-          popupClassName || dropdownClassName,
-          cssVarCls,
-          rootCls,
-          rootClassName,
-        )}
-        popupStyle={{
-          ...props.popupStyle,
-          zIndex,
-        }}
-        allowClear={mergeAllowClear(allowClear, clearIcon, <CloseCircleFilled />)}
-      />,
+      <NoCompactStyle>
+        <RCRangePicker<DateType>
+          separator={
+            <span aria-label="to" className={`${prefixCls}-separator`}>
+              <SwapRightOutlined />
+            </span>
+          }
+          disabled={mergedDisabled}
+          ref={innerRef}
+          dropdownAlign={transPlacement2DropdownAlign(direction, placement)}
+          placeholder={getRangePlaceholder(locale, picker, placeholder)}
+          suffixIcon={suffixNode}
+          prevIcon={<span className={`${prefixCls}-prev-icon`} />}
+          nextIcon={<span className={`${prefixCls}-next-icon`} />}
+          superPrevIcon={<span className={`${prefixCls}-super-prev-icon`} />}
+          superNextIcon={<span className={`${prefixCls}-super-next-icon`} />}
+          transitionName={`${rootPrefixCls}-slide-up`}
+          {...restProps}
+          {...additionalOverrideProps}
+          className={classNames(
+            {
+              [`${prefixCls}-${mergedSize}`]: mergedSize,
+              [`${prefixCls}-${variant}`]: enableVariantCls,
+            },
+            getStatusClassNames(
+              prefixCls,
+              getMergedStatus(contextStatus, customStatus),
+              hasFeedback,
+            ),
+            hashId,
+            compactItemClassnames,
+            className,
+            rangePicker?.className,
+            cssVarCls,
+            rootCls,
+            rootClassName,
+          )}
+          style={{ ...rangePicker?.style, ...style }}
+          locale={locale.lang}
+          prefixCls={prefixCls}
+          getPopupContainer={customGetPopupContainer || getPopupContainer}
+          generateConfig={generateConfig}
+          components={Components}
+          direction={direction}
+          dropdownClassName={classNames(
+            hashId,
+            popupClassName || dropdownClassName,
+            cssVarCls,
+            rootCls,
+            rootClassName,
+          )}
+          popupStyle={{
+            ...props.popupStyle,
+            zIndex,
+          }}
+          allowClear={mergeAllowClear(allowClear, clearIcon, <CloseCircleFilled />)}
+        />
+      </NoCompactStyle>,
     );
   });
 
diff --git a/components/date-picker/generatePicker/generateSinglePicker.tsx b/components/date-picker/generatePicker/generateSinglePicker.tsx
index 8efd78478cf3..8fa3a2700fb7 100644
--- a/components/date-picker/generatePicker/generateSinglePicker.tsx
+++ b/components/date-picker/generatePicker/generateSinglePicker.tsx
@@ -17,7 +17,7 @@ import DisabledContext from '../../config-provider/DisabledContext';
 import useSize from '../../config-provider/hooks/useSize';
 import { FormItemInputContext } from '../../form/context';
 import { useLocale } from '../../locale';
-import { useCompactItemContext } from '../../space/Compact';
+import { NoCompactStyle, useCompactItemContext } from '../../space/Compact';
 import enUS from '../locale/en_US';
 import useStyle from '../style';
 import {
@@ -151,58 +151,60 @@ export default function generatePicker<DateType>(generateConfig: GenerateConfig<
         const [zIndex] = useZIndex('DatePicker', props.popupStyle?.zIndex as number);
 
         return wrapCSSVar(
-          <RCPicker<DateType>
-            ref={innerRef}
-            placeholder={getPlaceholder(locale, mergedPicker, placeholder)}
-            suffixIcon={suffixNode}
-            dropdownAlign={transPlacement2DropdownAlign(direction, placement)}
-            prevIcon={<span className={`${prefixCls}-prev-icon`} />}
-            nextIcon={<span className={`${prefixCls}-next-icon`} />}
-            superPrevIcon={<span className={`${prefixCls}-super-prev-icon`} />}
-            superNextIcon={<span className={`${prefixCls}-super-next-icon`} />}
-            transitionName={`${rootPrefixCls}-slide-up`}
-            {...additionalProps}
-            {...restProps}
-            {...additionalOverrideProps}
-            locale={locale!.lang}
-            className={classNames(
-              {
-                [`${prefixCls}-${mergedSize}`]: mergedSize,
-                [`${prefixCls}-${variant}`]: enableVariantCls,
-              },
-              getStatusClassNames(
-                prefixCls,
-                getMergedStatus(contextStatus, customStatus),
-                hasFeedback,
-              ),
-              hashId,
-              compactItemClassnames,
-              consumerStyle?.className,
-              className,
-              cssVarCls,
-              rootCls,
-              rootClassName,
-            )}
-            style={{ ...consumerStyle?.style, ...style }}
-            prefixCls={prefixCls}
-            getPopupContainer={customizeGetPopupContainer || getPopupContainer}
-            generateConfig={generateConfig}
-            components={Components}
-            direction={direction}
-            disabled={mergedDisabled}
-            dropdownClassName={classNames(
-              hashId,
-              cssVarCls,
-              rootCls,
-              rootClassName,
-              popupClassName || dropdownClassName,
-            )}
-            popupStyle={{
-              ...props.popupStyle,
-              zIndex,
-            }}
-            allowClear={mergeAllowClear(allowClear, clearIcon, <CloseCircleFilled />)}
-          />,
+          <NoCompactStyle>
+            <RCPicker<DateType>
+              ref={innerRef}
+              placeholder={getPlaceholder(locale, mergedPicker, placeholder)}
+              suffixIcon={suffixNode}
+              dropdownAlign={transPlacement2DropdownAlign(direction, placement)}
+              prevIcon={<span className={`${prefixCls}-prev-icon`} />}
+              nextIcon={<span className={`${prefixCls}-next-icon`} />}
+              superPrevIcon={<span className={`${prefixCls}-super-prev-icon`} />}
+              superNextIcon={<span className={`${prefixCls}-super-next-icon`} />}
+              transitionName={`${rootPrefixCls}-slide-up`}
+              {...additionalProps}
+              {...restProps}
+              {...additionalOverrideProps}
+              locale={locale!.lang}
+              className={classNames(
+                {
+                  [`${prefixCls}-${mergedSize}`]: mergedSize,
+                  [`${prefixCls}-${variant}`]: enableVariantCls,
+                },
+                getStatusClassNames(
+                  prefixCls,
+                  getMergedStatus(contextStatus, customStatus),
+                  hasFeedback,
+                ),
+                hashId,
+                compactItemClassnames,
+                consumerStyle?.className,
+                className,
+                cssVarCls,
+                rootCls,
+                rootClassName,
+              )}
+              style={{ ...consumerStyle?.style, ...style }}
+              prefixCls={prefixCls}
+              getPopupContainer={customizeGetPopupContainer || getPopupContainer}
+              generateConfig={generateConfig}
+              components={Components}
+              direction={direction}
+              disabled={mergedDisabled}
+              dropdownClassName={classNames(
+                hashId,
+                cssVarCls,
+                rootCls,
+                rootClassName,
+                popupClassName || dropdownClassName,
+              )}
+              popupStyle={{
+                ...props.popupStyle,
+                zIndex,
+              }}
+              allowClear={mergeAllowClear(allowClear, clearIcon, <CloseCircleFilled />)}
+            />
+          </NoCompactStyle>,
         );
       },
     );
diff --git a/components/space/__tests__/__snapshots__/demo-extend.test.ts.snap b/components/space/__tests__/__snapshots__/demo-extend.test.ts.snap
index e42a91449e26..baa6d455d393 100644
--- a/components/space/__tests__/__snapshots__/demo-extend.test.ts.snap
+++ b/components/space/__tests__/__snapshots__/demo-extend.test.ts.snap
@@ -6407,7 +6407,7 @@ exports[`renders components/space/demo/compact.tsx extend context correctly 1`]
                     class="ant-picker-ok"
                   >
                     <button
-                      class="ant-btn ant-btn-primary ant-btn-sm ant-btn-compact-item ant-btn-compact-first-item"
+                      class="ant-btn ant-btn-primary ant-btn-sm"
                       disabled=""
                       type="button"
                     >
@@ -8020,7 +8020,7 @@ exports[`renders components/space/demo/compact.tsx extend context correctly 1`]
                       class="ant-picker-ok"
                     >
                       <button
-                        class="ant-btn ant-btn-primary ant-btn-sm ant-btn-compact-item ant-btn-compact-first-item"
+                        class="ant-btn ant-btn-primary ant-btn-sm"
                         disabled=""
                         type="button"
                       >
@@ -15185,7 +15185,7 @@ Array [
                     class="ant-picker-ok"
                   >
                     <button
-                      class="ant-btn ant-btn-primary ant-btn-sm ant-btn-compact-item ant-btn-compact-first-item"
+                      class="ant-btn ant-btn-primary ant-btn-sm"
                       disabled=""
                       type="button"
                     >
diff --git a/components/statistic/index.ts b/components/statistic/index.ts
index dd83b57a8683..dfe3cd00ac13 100644
--- a/components/statistic/index.ts
+++ b/components/statistic/index.ts
@@ -13,4 +13,4 @@ export type CompoundedStatistic = typeof Statistic & CompoundedComponent;
 
 (Statistic as CompoundedStatistic).Countdown = Countdown;
 
-export default (Statistic as CompoundedStatistic);
+export default Statistic as CompoundedStatistic;
diff --git a/components/table/InternalTable.tsx b/components/table/InternalTable.tsx
index 1025d775e21b..4ae04c5ca360 100644
--- a/components/table/InternalTable.tsx
+++ b/components/table/InternalTable.tsx
@@ -627,4 +627,4 @@ const InternalTable = <RecordType extends AnyObject = AnyObject>(
   );
 };
 
-export default (React.forwardRef(InternalTable) as RefInternalTable);
+export default React.forwardRef(InternalTable) as RefInternalTable;