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

bugfixes & highlight #4

Merged
merged 4 commits into from
Jun 23, 2020
Merged

bugfixes & highlight #4

merged 4 commits into from
Jun 23, 2020

Conversation

panda2134
Copy link

No description provided.

@panda2134
Copy link
Author

可能需要刷一下cdn,对字体有改动

@panda2134
Copy link
Author

图片
这个主要是防止恶意模仿树洞内跳转链接欺骗点击

@panda2134
Copy link
Author

图片

高亮。可以直接以```开头,修了评论区由于开头的评论者身份带来的md渲染Bug

@thuhole
Copy link
Collaborator

thuhole commented Jun 23, 2020

这个渲染空树洞和只有空格的树洞,会有黑屏问题嘛

@panda2134
Copy link
Author

这个渲染空树洞和只有空格的树洞,会有黑屏问题嘛

现在应该是不会了,可能还是得测试下

我昨天晚上把环境清了,要不你来测吧

@thuhole
Copy link
Collaborator

thuhole commented Jun 23, 2020

好,我来测

const parser = new HtmlToReact.Parser()

return parser.parseWithInstructions(renderedMarkdown, node => node.type !== 'script', processInstructions)
if (props.author && props.text.match(/^(?:#+ |>|```|\t|\s*-|\s*\d+\.)/)) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

这个regex会不会出什么特殊情况之类

Copy link
Author

Choose a reason for hiding this comment

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

加了$$的判断,我重开一个pr?

Copy link
Collaborator

Choose a reason for hiding this comment

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

OK

@thuhole thuhole merged commit 3fe299f into treehollow:master Jun 23, 2020
@panda2134
Copy link
Author

panda2134 commented Jun 23, 2020 via email

@thuhole
Copy link
Collaborator

thuhole commented Jun 23, 2020

@panda2134 很奇怪,为什么外链本地测试可以,服务器暂时不行。。。我估计是CDN的问题

@panda2134
Copy link
Author

panda2134 commented Jun 23, 2020 via email

@thuhole
Copy link
Collaborator

thuhole commented Jun 23, 2020

已经确认了,是
https://cdn.jsdelivr.net/gh/thuhole/webhole@gh-pages/static/fonts_7/icomoon.css

https://raw.githubusercontent.com/thuhole/webhole/gh-pages/static/fonts_7/icomoon.css
不同步所导致的。
这个CDN的purge cache功能好像还没上线。如果想立刻生效,可以弄一个新css文件。但这个功能看起来不是很紧急,所以先这样吧。

@thuhole
Copy link
Collaborator

thuhole commented Jun 23, 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

Successfully merging this pull request may close these issues.

2 participants