fix: Cascader should not clear search input when press left/right key (…

…#417) close ant-design/ant-design#43401
react-component · Jul 6, 2023 · 47011b9 · 47011b9 · vercel · Jul 6, 2023
1 parent b8aa0ed
commit 47011b9
Show file tree

Hide file tree

Showing 3 changed files with 20 additions and 1 deletion.
diff --git a/examples/adjust-overflow.tsx b/examples/adjust-overflow.tsx
@@ -121,7 +121,7 @@ const placements = {
 
 function Demo() {
   return (
-    <div style={{ position: 'absolute', right: 10, top: 150 }}>
+    <div>
       <MyCascader />
       <br />
       <br />

diff --git a/src/OptionList/useKeyboard.ts b/src/OptionList/useKeyboard.ts
@@ -135,6 +135,9 @@ export default (
         }
 
         case KeyCode.LEFT: {
+          if (searchValue) {
+            break;
+          }
           if (rtl) {
             nextColumn();
           } else {
@@ -144,6 +147,9 @@ export default (
         }
 
         case KeyCode.RIGHT: {
+          if (searchValue) {
+            break;
+          }
           if (rtl) {
             prevColumn();
           } else {

diff --git a/tests/keyboard.spec.tsx b/tests/keyboard.spec.tsx
@@ -245,6 +245,19 @@ describe('Cascader.Keyboard', () => {
     ).toHaveLength(0);
   });
 
+  it('should not switch column when press left/right key in search input', () => {
+    wrapper = mount(<Cascader options={addressOptions} showSearch />);
+    wrapper.find('input').simulate('change', {
+      target: {
+        value: '123',
+      },
+    });
+    wrapper.find('input').simulate('keyDown', { which: KeyCode.LEFT });
+    expect(wrapper.isOpen()).toBeTruthy();
+    wrapper.find('input').simulate('keyDown', { which: KeyCode.RIGHT });
+    expect(wrapper.isOpen()).toBeTruthy();
+  });
+
   // TODO: This is strange that we need check on this
   it.skip('should not handle keyDown events when children specify the onKeyDown', () => {
     wrapper = mount(