fix: disableCheckbox (#410)

* fix: disableCheckbox

* chore: add test case

* chore: add snapshots

---------

Co-authored-by: 洋 <[email protected]>

Author: BoyYangzai

src/OptionList/Column.tsx

@@ -159,6 +159,9 @@ export default function Column({
               data-path-key={fullPathKey}
               onClick={() => {
                 triggerOpenPath();
+                if (disableCheckbox) {
+                  return;
+                }
                 if (!multiple || isMergedLeaf) {
                   triggerSelect();
                 }
@@ -183,9 +186,12 @@ export default function Column({
                   prefixCls={`${prefixCls}-checkbox`}
                   checked={checked}
                   halfChecked={halfChecked}
-                  disabled={disabled}
+                  disabled={disabled || disableCheckbox}
                   disableCheckbox={disableCheckbox}
                   onClick={(e: React.MouseEvent<HTMLSpanElement>) => {
+                    if (disableCheckbox) {
+                      return;
+                    }
                     e.stopPropagation();
                     triggerSelect();
                   }}

tests/__snapshots__/index.spec.tsx.snap

@@ -63,7 +63,7 @@ exports[`Cascader.Basic should not show title when title is falsy 1`] = `
 exports[`Cascader.Basic should render custom dropdown correctly 1`] = `
 <div
   class="rc-cascader-dropdown rc-cascader-dropdown-placement-bottomLeft"
-  style="left: -1000vw; top: -1000vh; box-sizing: border-box; min-width: auto;"
+  style="--arrow-x: 0px; --arrow-y: 0px; left: -1000vw; top: -1000vh; box-sizing: border-box; min-width: auto;"
 >
   <div>
     <div

tests/__snapshots__/search.spec.tsx.snap

@@ -31,7 +31,7 @@ exports[`Cascader.Search should correct render Cascader with same field name of
   </div>
   <div
     class="rc-cascader-dropdown rc-cascader-dropdown-placement-bottomLeft"
-    style="left: -1000vw; top: -1000vh; box-sizing: border-box; min-width: 0;"
+    style="--arrow-x: 0px; --arrow-y: 0px; left: -1000vw; top: -1000vh; box-sizing: border-box; min-width: 0;"
   >
     <div>
       <div

tests/checkable.spec.tsx

@@ -208,8 +208,17 @@ describe('Cascader.Checkable', () => {
 
     // disabled className
     wrapper.find('.rc-cascader-menu-item').simulate('click');
+    expect(wrapper.find('.rc-cascader-menu-item')).toHaveLength(4);
     expect(wrapper.find('.rc-cascader-checkbox-disabled')).toHaveLength(1);
 
+    // click disableCkeckbox
+    wrapper.find('.rc-cascader-menu-item').at(1).simulate('click');
+    expect(wrapper.find('.rc-cascader-checkbox-checked')).toHaveLength(0);
+
+    // click disableMenuItem
+    wrapper.find('.rc-cascader-checkbox-disabled').simulate('click');
+    expect(wrapper.find('.rc-cascader-checkbox-checked')).toHaveLength(0);
+
     // Check all children except disableCheckbox When the parent checkbox is checked
     expect(wrapper.find('.rc-cascader-checkbox')).toHaveLength(4);
     wrapper.find('.rc-cascader-checkbox').first().simulate('click');