Skip to content

New component: image preview#357

Merged
cpylua merged 13 commits intomasterfrom
hotfix/add_image_preview
Jul 4, 2017
Merged

New component: image preview#357
cpylua merged 13 commits intomasterfrom
hotfix/add_image_preview

Conversation

@ronghang
Copy link
Copy Markdown
Contributor

add image-preview to zent, add test and demo.

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage decreased (-0.04%) to 92.93% when pulling 546e57f on hotfix/add_image_preview into d0e18a7 on master.

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.1%) to 93.08% when pulling 3e1a9a7 on hotfix/add_image_preview into d0e18a7 on master.

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.1%) to 93.079% when pulling b3ffa59 on hotfix/add_image_preview into d0e18a7 on master.


static defaultProps = {
prefix: 'zent',
visible: true,
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这个 visible 感觉不必要暴露到props上去吧,默认就是打开,也没提供外部控制关闭的选项。

import ReactDOM from 'react-dom';
import Image from './Image';

export default function ImagePreview(options = {}) {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

函数名字首字母小些吧。previewImage?

@@ -0,0 +1,3 @@
import ImagePreview from './ImagePreview';

export default ImagePreview;
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这地方也是,是个函数

Copy link
Copy Markdown
Contributor

@cpylua cpylua left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

函数名字,className,文档修改一下

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.1%) to 93.079% when pulling 346af07 on hotfix/add_image_preview into d0e18a7 on master.

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.02%) to 93.08% when pulling 008f186 on hotfix/add_image_preview into ed1d2e1 on master.

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.02%) to 93.08% when pulling e30a8c1 on hotfix/add_image_preview into ed1d2e1 on master.

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.02%) to 93.08% when pulling 7733460 on hotfix/add_image_preview into ed1d2e1 on master.

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.02%) to 93.08% when pulling d4b879e on hotfix/add_image_preview into ed1d2e1 on master.

@cpylua cpylua merged commit e4b8d32 into master Jul 4, 2017
@cpylua cpylua deleted the hotfix/add_image_preview branch July 4, 2017 06:27
@cpylua cpylua changed the title add image preview New component: image preview Jul 4, 2017
@cpylua cpylua added this to the 3.6.0 milestone Jul 4, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants