fix: style menu add density and scrolltime #132 #133
Conversation
|
我无法复现 #132 (comment) 提到的问题，这是 Linux 独有的吗？如果确定是 mpv 的 bug，我更倾向于打开 issue 让上游去修复，而不是优先在脚本中处理它 |
这个bug出现概率比较低，而且我尝试在日志中打印，options设置的值也是对的。这个bug有点玄学，而且目前似乎也只有我遇到，所以我之前就是选择无视的，目前对于这个问题为什么会产生我完全找不到头绪，只能描述为mpv抽风😂。 这个pr的目的倒也主要不是为了修复这个bug，主要是添加了density 和 scrolltime的实时修改，解决这个bug只是顺带的。当然重新加载弹幕确实增加了些许延迟，不过可视化弹幕源操作的时候也是重新加载了弹幕，我个人感觉延迟在可接受范围内。 |
只有 density 和 scrolltime 的实时修改需要重载弹幕，其他选项的实时修改原本是不需要的。所以这一无差别重载逻辑我个人不太认可，还是应该复现定位 bug 的原因，是 mpv 的问题应该尽量由上游修复，是脚本的问题才应该在脚本中修复 |
|
那这个pr改成只有修改density 和 scrolltime属性时才重载弹幕，其他时候维持修改option的原实现 至于option修改不生效的问题，我再想办法定位看看 这样如何呢？ |
我的想法就是这样，option 修改不生效暂不在此 pr 处理，等待进一步定位 |
好的，我改改 |
from #132