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

主页屏蔽分区功能不起作用 #783

Open
4 tasks done
ghost opened this issue Oct 28, 2020 · 6 comments
Open
4 tasks done

主页屏蔽分区功能不起作用 #783

ghost opened this issue Oct 28, 2020 · 6 comments

Comments

@ghost
Copy link

ghost commented Oct 28, 2020

问题反馈

在您提交问题前, 请检查以下项目:

  • 您在报告一个 bug 或者提交一个功能请求. 如果您需要一般性的帮助, 请直接和我们联系或者在项目首页下留言.
  • 您的问题在与现有问题并不重复. 如果已有一个相同话题的问题, 请直接在其下留言.
  • 如果您在报告一个 bug 请确定它可以在最新的发行版或 master 分支下重现. 同时, 请详细说明重现问题的步骤 (如果有屏幕截图的话就更好了).
  • 您已填写了系统环境信息.

环境信息

请填写以下信息:

  1. 操作系统 (Windows/macOS/Linux):
    macOS 10.15.7 (19H2)
  2. 浏览器版本(您可以在 [菜单/帮助/关于] 下查看):
    Chrome 86.0.4240.111(正式版本) (x86_64)
  3. 扩展版本:
    1.2.25(正式版)

重现步骤/尝试的解决方案

请具体描述, 谢谢 :)
尝试勾选主页屏蔽分区,刷新bilibili主页后没有变化

@Cotch22
Copy link
Collaborator

Cotch22 commented Oct 29, 2020

目前仅支持侧边栏的屏蔽,这个功能是一个初步上线的版本,之后会逐步完善。
感谢你的期待。

@memorydream
Copy link
Contributor

这是一个成功复现的BUG,其原因在于Edge与其他浏览器中不一致的行为导致,与此同时有一个Edge Only的BUG,会一并修复

@ghost
Copy link
Author

ghost commented Oct 29, 2020

重复实验发现侧边栏屏蔽也无法生效

@memorydream
Copy link
Contributor

重复实验发现侧边栏屏蔽也无法生效

分区和侧边栏其实是一起的,屏蔽了分区侧边栏里会一并屏蔽,不然会出BUG
这个无法屏蔽的BUG是因为异步加载,导致屏蔽模块比主页分区先加载完成,因为在Edge里并没有碰到过这样的问题,而我开发调试都是在Edge下,故没有发现这个问题
pr里的那个分支修复了这个BUG,在我这边,Edge和Vivalid都正常屏蔽,Chrome的行为应该与Vivalid一致

@jjj201200
Copy link
Member

jjj201200 commented Nov 2, 2020

@memorydream 在现在dev的基础上,我建议使用动态注入和修改全局样式的方式来实现该功能,可以不用关心他是否是异步加载

因为id名都是固定且一一对应的,应该不难实现,而且逻辑更加简单,不需要查询,直接将为true的id生成相应css注入即可

参考chatFilter模块

@memorydream
Copy link
Contributor

@memorydream 在现在dev的基础上,我建议使用动态注入和修改全局样式的方式来实现该功能,可以不用关心他是否是异步加载

因为id名都是固定且一一对应的,应该不难实现,而且逻辑更加简单,不需要查询,直接将为true的id生成相应css注入即可

参考chatFilter模块

可以先搁置,因为在现在dev分支的代码在Vivaldi下正常工作,没有碰上异步加载的问题,可能是你添加了什么其他代码导致的

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants