Closed IceTeaManiac closed 1 month ago
这个实现可能存在bug
videoController 的 danmakuOn 属性在其实例化时从 Gstorage 中获取。
videoController 的实例化发生在 IndexModule 。
这样的话,通过设置改变 Gstorage 中的属性时,直到程序下一次重启,videoController 中的 danamkuOn 属性不会出现变更。
是这样的,因此我在 SetSwitchItem
配置了 needReboot: true
来提示用户需要重启。
这个实现不好,重启应该尽量被避免。
这个地方没有重启的必要,这个PR需要修改,我的建议是在 videoPage 初始化时获取 Gstorage 中的 danmakuEnabledByDefault 属性,并移除 needReboot: true
重启确实会破坏用户体验。 这里我再熟悉一下代码,后面修改好了的话再提一下 PR。
在“弹幕设置”里面增加一个“默认开启”的选项,开启之后进入视频播放页面时会自动打开弹幕开关。