Skip to content
This repository has been archived by the owner on Feb 1, 2024. It is now read-only.

DatePickerInput improvements #2460

Merged
merged 5 commits into from
Nov 22, 2022
Merged

DatePickerInput improvements #2460

merged 5 commits into from
Nov 22, 2022

Conversation

qubis741
Copy link
Contributor

@qubis741 qubis741 commented Nov 22, 2022

Changed

  • DatePickerInput: Used proper cursor and added ability to close DatePicker on ESC and ENTER key (@qubis741) in (#2460)

Manual check

  • typeable DatePicker has "typing" cursor when not focused
  • DatePickerInput is closable on ESC and ENTER

driesd
driesd previously approved these changes Nov 22, 2022
@driesd
Copy link
Contributor

driesd commented Nov 22, 2022

⌛ MC'ing

lorgan3
lorgan3 previously approved these changes Nov 22, 2022
CHANGELOG.md Outdated Show resolved Hide resolved
@qubis741 qubis741 dismissed stale reviews from lorgan3 and driesd via 27ece4e November 22, 2022 12:24
@driesd
Copy link
Contributor

driesd commented Nov 22, 2022

In the stories, the typable={false} should have been added to the story underneath it. See screenshot.

Screenshot 2022-11-22 at 13 24 12

@driesd
Copy link
Contributor

driesd commented Nov 22, 2022

Besides the story remark, MC works as advertised ✅

@qubis741 qubis741 merged commit 06d9338 into next-release Nov 22, 2022
@qubis741 qubis741 deleted the FRAF-1154-v5 branch November 22, 2022 12:44
@qubis741 qubis741 mentioned this pull request Nov 22, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants