Skip to content

Conversation

@jerrybendy
Copy link
Contributor

在使用 jsfiddle 标签插件生成的 iframe 中的 src 地址默认是包含 http:// 前缀的,如 http://jsfiddle.net/xxxxxxx/embedded/result,css,html/light

这会导致在 HTTPS 的网站中出现非 https 的内容,导致抛出警告,并且地址栏中的小锁加黄色警告标识。 jsfiddle 官方的 emmbed 插件地址已经去掉了前面的 http 前缀,所以这里可以放心的改成 //jsfiddle.net/

@coveralls
Copy link

coveralls commented Aug 10, 2016

Coverage Status

Coverage remained the same at 95.947% when pulling cc7f6ed on jerrybendy:patch-1 into 5234c4a on hexojs:master.

@Xuanwo
Copy link
Contributor

Xuanwo commented Oct 9, 2016

顺便修改一下test吧,let test happy

@jerrybendy
Copy link
Contributor Author

@Xuanwo 这个 test case 的文件在哪里

@NoahDragon
Copy link
Member

@jerrybendy hexo/test/scripts/tags/jsfiddle.js

@jerrybendy
Copy link
Contributor Author

已修改 #2206

@jerrybendy
Copy link
Contributor Author

提交错分支了。。。

@jerrybendy
Copy link
Contributor Author

image

提交 jsfiddle 的 test case ,打错字了 😱

@coveralls
Copy link

coveralls commented Oct 10, 2016

Coverage Status

Coverage increased (+1.2%) to 97.127% when pulling eb43047 on jerrybendy:patch-1 into 5234c4a on hexojs:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage increased (+1.2%) to 97.127% when pulling eb43047 on jerrybendy:patch-1 into 5234c4a on hexojs:master.

@NoahDragon
Copy link
Member

LGTM

@NoahDragon NoahDragon merged commit afe0bb1 into hexojs:master Oct 20, 2016
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.

4 participants