Skip to content
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.

Sign up for GitHub

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

Jump to bottom

expose reset function #46

Closed
wants to merge 1 commit into from
Closed

expose reset function #46

wants to merge 1 commit into from

Conversation

ddcat1115
Copy link
Contributor

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.4%) to 42.786% when pulling f0777a5 on ddcat1115:master into 422eedf on react-component:master.

@afc163
Copy link
Member

afc163 commented Aug 4, 2016 •
edited
Loading

专门用于触发动作的属性，感觉不是好的实践。

@ddcat1115
Copy link
Contributor Author

看到这里不推荐ref...
https://github.com/react-component/react-component.github.io/blob/master/docs/zh-cn/component-design.md#避免使用-ref

@afc163
Copy link
Member

afc163 commented Aug 4, 2016 •
edited
Loading

暴露 disabled 属性给外界控制 upload 状态如何？

@ddcat1115
Copy link
Contributor Author

你是说用外部属性的变化直接控制upload内部的disabled？

@afc163
Copy link
Member

afc163 commented Aug 5, 2016

嗯。这样那段逻辑可以放到 disabled 的变化中去。

yiminghe added a commit that referenced this pull request Aug 10, 2016
@yiminghe
add abort api. Update #46
46e809b
@yiminghe
Copy link
Member

yiminghe commented Aug 10, 2016 •
edited
Loading

大家先看下 46e809b

得用 abort 了， antd upload 每次对比下，如果有正在上传的文件消失了就调用 abort

@yiminghe yiminghe closed this in 29e48b1 Aug 11, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
4 participants
@ddcat1115 @coveralls @afc163 @yiminghe