treehollow / treehollow-backend

T大树洞 的Golang后端
https://thuhole.com/
GNU Affero General Public License v3.0
167 stars 47 forks source link

前端黑屏bug #35

Closed rcy17 closed 3 years ago

rcy17 commented 4 years ago

我linux chrome83.0.4103.97,在点开 #4445 后会黑屏,看到 #4665 有相同的情况, #4665 下Alice说的强制刷新并未解决问题,怀疑是前端bug。 点开出错时,chrome 的console 有如下输出:

react-dom.production.min.js:4260 TypeError: Cannot read property 'setProperty' of undefined
    at mr (react-dom.production.min.js:2225)
    at zs (react-dom.production.min.js:5345)
    at Ys (react-dom.production.min.js:5103)
    at Bs (react-dom.production.min.js:4958)
    at Ms (react-dom.production.min.js:4817)
    at react-dom.production.min.js:2543
    at t.unstable_runWithPriority (scheduler.production.min.js:338)
    at ua (react-dom.production.min.js:2513)
    at ma (react-dom.production.min.js:2538)
    at pa (react-dom.production.min.js:2528)
Uncaught TypeError: Cannot read property 'setProperty' of undefined
    at mr (react-dom.production.min.js:2225)
    at zs (react-dom.production.min.js:5345)
    at Ys (react-dom.production.min.js:5103)
    at Bs (react-dom.production.min.js:4958)
    at Ms (react-dom.production.min.js:4817)
    at react-dom.production.min.js:2543
    at t.unstable_runWithPriority (scheduler.production.min.js:338)
    at ua (react-dom.production.min.js:2513)
    at ma (react-dom.production.min.js:2538)
    at pa (react-dom.production.min.js:2528)

望解决!

thuhole commented 4 years ago

@panda2134

panda2134 commented 4 years ago

估计是哪里渲染md没有返回null吧,这一时半会我也查不出来On Wed, Jun 24, 2020 at 00:10, thuhole notifications@github.com wrote:
@panda2134

—You are receiving this because you were mentioned.Reply to this email directly, view it on GitHub, or unsubscribe.

thuhole commented 4 years ago

@panda2134 我大概看了一下,好像确实没那么简单

thuhole commented 4 years ago

@panda2134 这里面没有空的回复or树洞

thuhole commented 4 years ago

@panda2134 复现方法:评论【$Eve$ test】

panda2134 commented 4 years ago

应该是在渲染markdown之后又嵌入了着色回复者名称造成的。不知道怎么修,要不您自己看看?On Wed, Jun 24, 2020 at 00:23, thuhole notifications@github.com wrote:
@panda2134 复现方法:评论【$Eve$ test】

—You are receiving this because you were mentioned.Reply to this email directly, view it on GitHub, or unsubscribe.

thuhole commented 4 years ago

@panda2134 你都不会修我就更不会修了。。。没事不着急明天再来看吧

thuhole commented 4 years ago

按照现在的代码,$Eve$会按什么顺序着色呢,先变成node然后regex匹配然后转化为ColoredSpan?那应该没问题吧

thuhole commented 4 years ago

刚刚测试了一下,搜索时的关键词高亮渲染也会出bug,比如搜索"3000"也会黑屏,原因是那条洞里

真诚求助,请问有没有GRE已经上岸的同学,想问问在刷题阶段用哪些材料比较好,我目前在用要你命3000系列,但是又听说考满分的机经很好🤨,所以不知道该选择哪个了,(目前备考时间还有2-3个月,受疫情影响可能更长)感谢各位🙏

出现了3000这个关键词,然后尝试渲染的时候就失败了。。。很奇怪 为什么评论区正常情况渲染名字着色高亮不会失败?

thuhole commented 4 years ago

但比如搜索“大家咋看”,得到这样一条洞

如何看待原清华本科生胡泽聪为了在CMU继续科研而让自己同学代替自己上课签到,体测,展示的问题? - 知乎\nhttps://www.zhihu.com/question/402247508\n\n## 大家咋看

就不会出bug,虽然搜索词的高亮渲染没有了。

thuhole commented 4 years ago

如果这个问题不能很好解决,可以考虑两个让步方案:

  1. regex检测到markdown之后停止所有高亮渲染
  2. 回退到支持markdown之前的版本
thuhole commented 4 years ago

刚刚测试了一下,搜索时的关键词高亮渲染也会出bug,比如搜索"3000"也会黑屏,原因是那条洞里

真诚求助,请问有没有GRE已经上岸的同学,想问问在刷题阶段用哪些材料比较好,我目前在用要你命3000系列,但是又听说考满分的机经很好🤨,所以不知道该选择哪个了,(目前备考时间还有2-3个月,受疫情影响可能更长)感谢各位🙏

出现了3000这个关键词,然后尝试渲染的时候就失败了。。。很奇怪 为什么评论区正常情况渲染名字着色高亮不会失败?

啊,原因好像不是这个,原因还是洞底下有人搞破坏

thuhole commented 4 years ago

@panda2134 我现在暂时禁用了评论高亮

thuhole commented 4 years ago

https://github.com/thuhole/webhole/commit/55f77f26b77c551e3a50232352fdb9b07fafd982 疑似修复

thuhole commented 4 years ago

虽然疑似修复,但还是指不定哪天出来什么bug。。。可能还是要多测试一下。