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

Remove jquery #93

Merged
merged 8 commits into from
Aug 20, 2019
Merged

Remove jquery #93

merged 8 commits into from
Aug 20, 2019

Conversation

xiazeyu
Copy link
Collaborator

@xiazeyu xiazeyu commented Aug 11, 2019

Issue Fixed #18 #31 #72 #83 #89

Proposed Changes

  • 更好的文档格式
  • 最新的依赖
  • 妈妈再也不用担心我加载不出jQuery了

@MikeCoder

@xiazeyu xiazeyu requested a review from D0n9X1n August 11, 2019 07:48
@xiazeyu
Copy link
Collaborator Author

xiazeyu commented Aug 11, 2019

不过也有妥协
会有1.14%的过时浏览器用不了

https://caniuse.com/#search=innerHTML
https://caniuse.com/#search=addEventListener

@D0n9X1n
Copy link
Owner

D0n9X1n commented Aug 12, 2019

这边引入 jQuery 主要为了解决这个问题 - #21 ,删除 jQuery 之前应该先支持内容中包含额外的 js 文件。

lib/blog-encrypt.js Outdated Show resolved Hide resolved
@D0n9X1n D0n9X1n merged commit 811b53a into master Aug 20, 2019
@D0n9X1n
Copy link
Owner

D0n9X1n commented Aug 20, 2019

晚些时候我测试下,然后发新版本。

感谢!

@xiazeyu xiazeyu deleted the remove-jquery branch August 20, 2019 13:57
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.

输入密码按回车不知为何没反应。
2 participants