-
Notifications
You must be signed in to change notification settings - Fork 44
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[36] Date Time Fixtures #76
Conversation
321595a
to
562ae70
Compare
562ae70
to
b3d6eb6
Compare
@@ -93,7 +93,7 @@ const TimePickerInput = React.forwardRef<HTMLInputElement, TimePickerInputProps> | |||
setDate(setDateByType(tempDate, newValue, picker, period)); | |||
} | |||
if (e.key >= '0' && e.key <= '9') { | |||
if (picker === '12hours') setPrevIntKey(e.key); | |||
if (picker === 'hours') setPrevIntKey(e.key); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what are possible values of picker other than hours
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Previously there are 2 options like 12hours format or 24hours format.
I removed 24hours related code as it was redundant.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good
Description
Preview
Preview