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

配置uploadImgUrl后,点击图片上传按钮无反应 #1

Open
legendweny opened this issue Dec 25, 2015 · 7 comments
Open

配置uploadImgUrl后,点击图片上传按钮无反应 #1

legendweny opened this issue Dec 25, 2015 · 7 comments

Comments

@legendweny
Copy link

http://fork12.com/ckupload/test 测试站点

@wangfupeng1988
Copy link
Owner

我用手机扫描看了一下,发现了这个问题——点击上传图片按钮没反应。
但是我还发现了另外一个问题——编辑区域执行tap事件,菜单不容易出来。每次出来,都是和浏览器的默认菜单一起出来。再官网 http://wangeditor.github.io/m/ demo中就没有这个问题,你可以扫描一下看看。

这样,你吧你那个测试页面中的wangEditor-mobile.min.js文件替换成 https://github.com/wangfupeng1988/wangEditor-mobile/blob/master/dist/js/wangEditor-mobile.js,然后告诉我,我调试一下。

如果真有问题,我会立即改正!

@wangfupeng1988
Copy link
Owner

我刚才拿你的例子,对比了一下我的测试环境。发现,dom结构都是对的,但是我的测试环境下是OK的,你的例子不行。我也很想知道原因。碍于你给出的例子的wangEditor-mobile.min.js压缩版的,没法调试。

@legendweny
Copy link
Author

好了,我替换好了,你看看

@wangfupeng1988
Copy link
Owner

我看了,原因是在点击选择图片按钮的时候,执行 $inputFlie.trigger('click');没有弹出选择框,而我的测试环境是可以正常弹出来的——这是现象——也就说,选择文件的 input 根本就没有执行 click 事件。

我看了一下你的例子,发现你的inputjquery注入了两个click事件,而我的测试环境下面是没有这个的。可以通过浏览器检测一下

所以,我大胆猜测,是这个fork12.com中的js对input做了什么特殊处理,要不然怎么会平白无故增加click事件呢。

建议你针对这个问题,单独做一个测试。


另外,我不明白,你为何要再这个平台上做测试。你自己制作的demo,真的就没法选择文件?

@Natumsol
Copy link

@wangfupeng1988 你好,我也遇到过这个问题,经过排查发现是zepto.js版本兼容问题。1.0的zepto.js是可以正常显示的,我用的最新版的zepto.js(1.1.6版本)就会出现点击图片上传按钮无反应。

@wangfupeng1988
Copy link
Owner

@Natumsol 我测试的是跟浏览器兼容性有关系,只有在QQ浏览器的部分情况,才会出现这种情况。像比较稳定的 safari、chrome,我没测出来

@Natumsol
Copy link

@wangfupeng1988 你试着将zepto.js换为1.1.6,然后在UC浏览器或微信里进行测试。还有就是用在PC端可以正常的上传图片,但是在移动端(微信, UC浏览器)就会出现上传超时的现象,我用fiddler抓包后发现,上传请求实际上并没有发出。

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

No branches or pull requests

3 participants