Skip to content

Commit f47a36b

Browse files
authored
fix: Picker input should not trigger form submit (#22)
* should not trigger default submit by enter * add test case
1 parent 3baac1e commit f47a36b

File tree

2 files changed

+13
-0
lines changed

2 files changed

+13
-0
lines changed

src/hooks/usePickerInput.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -49,6 +49,8 @@ export default function usePickerInput({
4949
onSubmit();
5050
setTyping(true);
5151
}
52+
53+
e.preventDefault();
5254
return;
5355
}
5456

tests/keyboard.spec.tsx

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -471,4 +471,15 @@ describe('Picker.Keyboard', () => {
471471
).toEqual('');
472472
});
473473
});
474+
475+
it('enter should prevent default to avoid form submit', () => {
476+
const wrapper = mount(<MomentPicker />);
477+
const preventDefault = jest.fn();
478+
wrapper.find('input').simulate('keyDown', {
479+
which: KeyCode.ENTER,
480+
preventDefault,
481+
});
482+
483+
expect(preventDefault).toHaveBeenCalled();
484+
});
474485
});

0 commit comments

Comments
 (0)