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

V1.4.0的一些问题 #254

Closed
Reqwey opened this issue Apr 5, 2020 · 26 comments
Closed

V1.4.0的一些问题 #254

Reqwey opened this issue Apr 5, 2020 · 26 comments

Comments

@Reqwey
Copy link

Reqwey commented Apr 5, 2020

好像最近你更新到了Valine V1.4.0(官网都没更🤣)

我发现一个UI的问题, 就是点击笑脸图案弹出Emoji选项后, 正常应该是再点一下收回去, 而这个版本没有实现, 点一下就不能再点了

还有一个, 就是验证码好像又不能用了

@xCss
Copy link
Owner

xCss commented Apr 5, 2020

官网的v1.4.0版本是属于内测版本,目前正在优化一些体验上的问题,后续将推出更新

Emoji只能点开不能收回是已知问题,本地已解决,感谢反馈。

验证码将在下个版本中取消,改由statusBar上显示类似于操作频繁的提示消息。

v1.4.0功能上有大更新,敬请期待。

(吐槽一下,最两天官网和个人博客都被人刷屏,好烦躁
image

@Reqwey
Copy link
Author

Reqwey commented Apr 5, 2020

所以错误代码 Code 140: code-140 跟这个有关吗

@Reqwey
Copy link
Author

Reqwey commented Apr 5, 2020

确实, 在你的个人博客看到一堆的undefined

@xCss
Copy link
Owner

xCss commented Apr 5, 2020

没错,code 140是提示今日 API 调用总次数已超过开发版限制.

哎,好烦躁啊,挤出个人时间来搞这个,还要被人搞
image

@Reqwey
Copy link
Author

Reqwey commented Apr 5, 2020

你可以弄垃圾评论检测(第三方的) https://github.com/DesertsP/Valine-Admin

@Reqwey
Copy link
Author

Reqwey commented Apr 5, 2020

至少重复的内容会被过滤掉

@xCss
Copy link
Owner

xCss commented Apr 5, 2020

哎,最终也逃不过真香警告~~
image

@Reqwey
Copy link
Author

Reqwey commented Apr 5, 2020

还有, 打个广告, 推荐一个改版, 可以禁掉匿名评论 https://github.com/xaoxuu/volantis

@xCss
Copy link
Owner

xCss commented Apr 5, 2020

哇哦w(゚Д゚)w,厉害哟image,貌似很多人用啊~

我看jsdelivr引用超过100w

@xaoxuu
Copy link
Contributor

xaoxuu commented Apr 5, 2020

我这改成必填项也只是勉强有一点点作用,有时候还是会收到昵称是 111 邮箱是 aaa 类似这样的评论。。。

@xaoxuu
Copy link
Contributor

xaoxuu commented Apr 5, 2020

@linhk1606 那个地址旧版的,不再更新了,因为主题用了这个名字,所以评论就放在主题的cdn仓库里面了。

@Reqwey
Copy link
Author

Reqwey commented Apr 6, 2020

还有一个, 就是如果页面有加载MathJax, 则Valine会加载不出来, 浏览器会报错

Uncaught TypeError: Cannot read property 'Typeset' of undefined
    at e.exports (valine.js:1)
    at valine.js:1
    at valine.js:1
    at HTMLScriptElement.i.onload.i.onreadystatechange (valine.js:1)

在这里

"undefined" != typeof MathJax && MathJax.Hub.Typeset(document.querySelector(".v"))

@xCss
Copy link
Owner

xCss commented Apr 6, 2020

@linhk1606 嗯,这个应该是我用了MathJax 2.x的渲染方式,而你加载的是3.x,我看看怎么处理这个问题~

@xCss
Copy link
Owner

xCss commented Apr 6, 2020

image
好骚啊,把这么多表情都整合进去了

哈哈,我看到你的博客上有个bug,你的所有的图片都是display:block,造成单个表情也是独占一行~

@Reqwey
Copy link
Author

Reqwey commented Apr 6, 2020

好像在GitHub上也占一行

@xCss
Copy link
Owner

xCss commented Apr 6, 2020

好像在GitHub上也占一行

github么有独占一行哦~

@Reqwey
Copy link
Author

Reqwey commented Apr 6, 2020

哇哦w(゚Д゚)w,厉害哟image,貌似很多人用啊~

我看jsdelivr引用超过100w

那是因为@xaoxuu有个美丽的hexo主题 , 有很多人用, 而 Valine 的这个改版是主题捆绑安装自带 的

@xCss
Copy link
Owner

xCss commented Apr 6, 2020

哇哦w(゚Д゚)w,厉害哟image,貌似很多人用啊~
我看jsdelivr引用超过100w

那是因为@xaoxuu有个美丽的hexo主题(我也用了) , 有很多人用, 而 Valine 的这个改版是主题捆绑安装自带 的

牛皮,牛皮,膜拜大佬 image

@xCss
Copy link
Owner

xCss commented Apr 6, 2020

@xCss

我在这个项目中配置math参数用以区分MathJax@2和MathJax@3.

或许你可以尝试一下

https://github.com/MiniValine/MiniValine

image

GitHub**MiniValine/MiniValine**A simple and minimalist comment system based on Leancloud. - MiniValine/MiniValine

没必要指定,可以直接判断MathJax版本号

@leirock
Copy link

leirock commented Apr 6, 2020

官网的v1.4.0版本是属于内测版本,目前正在优化一些体验上的问题,后续将推出更新

原来还是内测啊,默默地已经开始在我博客使用了 https://github.com/lei2rock/blog/blob/master/_config.yml#L457

@xaoxuu
Copy link
Contributor

xaoxuu commented Apr 6, 2020

@xCss 你好,我发现 1.4.0 之后,网页中拖拽一段文字或者一个按钮,就会把文字或者按钮标题填入评论框,1.3.10 中不存在这个问题。

@xCss
Copy link
Owner

xCss commented Apr 6, 2020

@xaoxuu 嗯,内测版难免会出现各种问题,而且,拖拽文字进评论框应该也不属于Bug吧?

@xCss
Copy link
Owner

xCss commented Apr 6, 2020

image.png

@xCss

inlineMath: [ ['$', '$'], ['\\(', '\\)'] ]

此处MathJax渲染和Markdown冲突

document.body.append(e)

这里不兼容IE

你可以参考我这里最新版本的代码
https://github.com/MiniValine/MiniValine/blob/master/src/utils/plugins/initMathJax2.js#L10
https://github.com/MiniValine/MiniValine/blob/master/src/utils/plugins/initMathJax2.js#L35

嗯,MathJax和Markdown冲突是已知问题,append在IE中的问题已收到~

@xaoxuu
Copy link
Contributor

xaoxuu commented Apr 6, 2020

@xCss 不是主动拖拽进评论框,而是只要发生了拖拽操作,就会跳到评论框。

例如: https://valine.js.org/ 随便按住一个按钮或者图片移动一点点再松开。

很影响体验的,我观看网页的时候选中了一段文字不小心多点了一下就跳到底下评论框了;一个按钮按下去又不想点了鼠标移走就又跳到评论框了。如果是 1.4.0 代码导致,的建议取消掉这个效果。

@xCss
Copy link
Owner

xCss commented Apr 7, 2020

@xCss 不是主动拖拽进评论框,而是只要发生了拖拽操作,就会跳到评论框。

例如: https://valine.js.org/ 随便按住一个按钮或者图片移动一点点再松开。

很影响体验的,我观看网页的时候选中了一段文字不小心多点了一下就跳到底下评论框了;一个按钮按下去又不想点了鼠标移走就又跳到评论框了。如果是 1.4.0 代码导致,的建议取消掉这个效果。

恩,了解了~

@xCss
Copy link
Owner

xCss commented Apr 10, 2020

v1.4.0已更新

@xCss xCss closed this as completed Apr 10, 2020
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

4 participants