feat: enhance allowClear prop to support disabled (#160)

* feat: enhance allowClear prop to support disabled

* Update src/BaseInput.tsx

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>

* feat: add test for allowClear prop when disabled

---------

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>

Author: guoyunhe

src/BaseInput.tsx

@@ -74,7 +74,11 @@ const BaseInput = React.forwardRef<HolderRef, BaseInputProps>((props, ref) => {
     // ================== Clear Icon ================== //
     let clearIcon: ReactNode = null;
     if (allowClear) {
-      const needClear = !disabled && !readOnly && value;
+      const needClear =
+        !disabled &&
+        !readOnly &&
+        value &&
+        !(typeof allowClear === 'object' && allowClear.disabled);
       const clearIconCls = `${prefixCls}-clear-icon`;
       const iconNode =
         typeof allowClear === 'object' && allowClear?.clearIcon

src/interface.ts

@@ -33,7 +33,7 @@ export interface CommonInputProps {
     prefix?: CSSProperties;
     suffix?: CSSProperties;
   };
-  allowClear?: boolean | { clearIcon?: ReactNode };
+  allowClear?: boolean | { disabled?: boolean; clearIcon?: ReactNode };
 }
 
 type DataAttr = Record<`data-${string}`, string>;
@@ -86,7 +86,8 @@ export interface CountConfig {
 }
 
 export interface InputProps
-  extends CommonInputProps,
+  extends
+    CommonInputProps,
     Omit<
       InputHTMLAttributes<HTMLInputElement>,
       'size' | 'prefix' | 'type' | 'value'

tests/index.test.tsx

@@ -278,6 +278,17 @@ describe('Input allowClear', () => {
     });
   });
 
+  it('should not support allowClear when allowClear is disabled', () => {
+    const { container } = render(
+      <Input
+        prefixCls="rc-input"
+        allowClear={{ disabled: true }}
+        defaultValue="111"
+      />,
+    );
+    expect(container.querySelector('.rc-input-clear-icon-hidden')).toBeTruthy();
+  });
+
   it('classNames and styles should work', () => {
     const { container } = render(
       <>