-
-
Notifications
You must be signed in to change notification settings - Fork 180
feat(date): adds date time picker #128
feat(date): adds date time picker #128
Conversation
dateFormatCalendar?: string | ||
/** Disable datepicker. */ | ||
disabled?: boolean | ||
dropdownMode: 'scroll' | 'select' |
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.
this prop is missing a description
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.
Done.
children?: ReactNode | ||
/** The format for parsed and displayed dates. */ | ||
dateFormat?: string | ||
dateFormatCalendar?: string |
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.
this prop is missing a description
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.
Done.
@martinrradev nice work! just a couple of missing prop descriptions to add. thanks for your contribution 😊 |
🎉 This PR is included in version 0.20.1 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Fixes #34
Changes proposed in this pull request:
Newly added dependencies with Bundlephobia links: