fix: Disabled should not display clear button

Author: zombieJ

src/OptionList.tsx

@@ -131,7 +131,6 @@ const OptionList: React.RefForwardingComponent<
   // ========================== Values ==========================
   const onSelectValue = (value: RawValueType) => {
     if (value !== null) {
-      console.log('-->', value, Array.from(values));
       onSelect(value, { selected: !values.has(value) });
     }
 

src/generate.tsx

@@ -796,7 +796,7 @@ export default function generateSelector<
       triggerSearch('', false);
     };
 
-    if (allowClear && (mergedRawValue.length || mergedSearchValue)) {
+    if (!disabled && allowClear && (mergedRawValue.length || mergedSearchValue)) {
       clearNode = (
         <TransBtn
           className={`${prefixCls}-clear`}

tests/Select.test.tsx

@@ -359,6 +359,13 @@ describe('Select.Basic', () => {
     );
 
     wrapper.find('input').simulate('change', { target: { value: 'foo' } });
+
+    // disabled
+    wrapper.setProps({ disabled: true });
+    expect(wrapper.find('.rc-select-clear')).toHaveLength(0);
+
+    // enabled
+    wrapper.setProps({ disabled: false });
     wrapper
       .find('.rc-select-clear')
       .last()

tests/__snapshots__/Select.test.tsx.snap

@@ -422,18 +422,6 @@ exports[`Select.Basic render renders disabled select correctly 1`] = `
       class="antd-arrow-icon"
     />
   </span>
-  <span
-    aria-hidden="true"
-    class="antd-clear"
-    style="user-select:none;-webkit-user-select:none"
-    unselectable="on"
-  >
-    <span
-      class="antd-clear-icon"
-    >
-      ×
-    </span>
-  </span>
 </div>
 `;