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.

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

fix #42 #43

Merged
merged 8 commits into from
Oct 8, 2014
Merged

fix #42 #43

merged 8 commits into from
Oct 8, 2014

Conversation

chestnutchen
Copy link
Member

No description provided.

@leeight
Copy link
Member

leeight commented Sep 11, 2014

Can one of the admins verify this patch?

@@ -33,6 +33,17 @@ define(

Uploader.prototype.type = 'Uploader';

var mimeTypes = {
Copy link
Member

Choose a reason for hiding this comment

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

这个不是 mimeType 吧,顶多算 extensionType


/**
* 富单选或富复选框组控件
* 和BoxGroup的区别:``````选项的文本内容可以含有子控件```````
Copy link
Member

Choose a reason for hiding this comment

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

**EMPHASIS**

Copy link
Member

Choose a reason for hiding this comment

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

看看能否改成直接继承 BoxGroup,否则感觉会有不少重复代码

@@ -528,15 +600,9 @@ define(
this.notifyFail(options.message);
Copy link
Member

Choose a reason for hiding this comment

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

这里要改一下,新的 NMP 接口不是返回 "success": true 的。

Copy link
Member Author

Choose a reason for hiding this comment

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

哦要把code加上。。。

Justineo added a commit that referenced this pull request Oct 8, 2014
@Justineo Justineo merged commit 5ee9911 into ecomfe:dev Oct 8, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants