-
-
Notifications
You must be signed in to change notification settings - Fork 47.5k
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
feat: expose TimePicker[open], close: #5775 & #5829 #5913
Conversation
components/time-picker/demo/addon.md
Outdated
onOpen={this.toggleOpenStatus} | ||
onClose={this.toggleOpenStatus} | ||
addon={() => ( | ||
<Button size="small" type="primary" onClick={this.handleClose}> |
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.
Make this more React-like.
pending for react-component/collapse#67 |
ci broken |
@@ -25,6 +25,9 @@ import moment from 'moment'; | |||
|---------------------|-----|-----|-------| | |||
| defaultValue | to set default time | [moment](http://momentjs.com/) | - | | |||
| value | to set time | [moment](http://momentjs.com/) | - | | |||
| open | whether to popup panel | boolean | false | | |||
| onOpen | a callback function which will be called while panel opening | ({ open: boolean }): void | 无 | | |||
| onClose | a callback function which will be called while panel closing | ({ open: boolean }): void | 无 | |
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.
和 Menu 一样改成 onOpenChange 吧。
d05ef1c
to
68b4d83
Compare
68b4d83
to
03dea50
Compare
Updated. |
@@ -25,6 +25,8 @@ import moment from 'moment'; | |||
|---------------------|-----|-----|-------| | |||
| defaultValue | to set default time | [moment](http://momentjs.com/) | - | | |||
| value | to set time | [moment](http://momentjs.com/) | - | | |||
| open | whether to popup panel | boolean | false | | |||
| onOpenChange | a callback function which will be called while panel opening/closing | (status: boolean): void | 无 | |
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.
无 => -
format?: string; | ||
onChange?: (time: moment.Moment, timeString: string) => void; | ||
onOpenChange?: (status: boolean) => void; |
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.
status => open
后两个细节,我合并后再改。 |
This thread has been automatically locked because it has not had recent activity. Please open a new issue for related bugs and link to relevant comments in this thread. |
close: #5775 & #5829