crimx / ext-saladict

🥗 All-in-one professional pop-up dictionary and page translator which supports multiple search modes, page translations, new word notebook and PDF selection searching.
https://saladict.crimx.com/
MIT License
11.81k stars 715 forks source link

与github对ipynb文件的渲染有冲突 #2093

Open Legend94rz opened 10 months ago

Legend94rz commented 10 months ago

Device info

Describe the bug

与github对ipynb文件的渲染有冲突,chrome里禁用该扩展后正常,只关闭取词无效。

To Reproduce

  1. 随便打开一个托管在github上一个ipynb文件,例如这个
  2. 在github页面上的文本部分(注意不是代码部分),随便拖动鼠标,选择一段文本
  3. github出现"Unable to render code block",并且ipynb的内容消失

Expected behavior

选择文本应该不影响ipynb的渲染才对。

Screenshots

image

crimx commented 10 months ago

排查了一下,我怀疑是因为它渲染在 iframe 中,沙拉查词在 iframe 划词时会通过 postMessage 向父 frame 传递划词信息,可能是它的渲染器也是通过 postMessage 交换信息,但没考虑到对信息进行校验,所以直接就炸了。

xin-fight commented 9 months ago

为了修复这个bug,需要用户本地进行什么更改吗?

crimx commented 9 months ago

用户没什么办法,这个由于沙拉查词的通讯机制导致的,只能等修复。(得先确认是这个问题)。

crimx commented 9 months ago

可以通过配置黑名单方式在特定网址禁用沙拉查词

AtwoodDuan commented 3 months ago

配置黑名单方式在特定网址禁用沙拉查词,依然无法避免点击ipynb后页面崩溃的问题

Nyte-BK201 commented 1 month ago

已复现该问题,没想到这个问题存在了这么久了。尝试过黑名单禁用,但是禁用的检查明显在滑动第一个词之后发生,所以依然导致了崩溃